source review for docstrings, pep8