modify as pep8