New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[GSoC] Implement new naming convention and fix imports #222
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please, rebase to the current HEAD.
This patch implements a new naming scheme for modules. It also fixes a few typos and flake8 formatting errors. Signed-off-by: Aniruddha Karajgi <akarajgi0@gmail.com>
This patch adds the __str__ method to all classes. It also changes the import style to absolute for the non-pandas metrics. Signed-off-by: Aniruddha Karajgi <akarajgi0@gmail.com>
b35ca55
to
a046e0a
Compare
@jgbarah, I got travis to work, but there is a small hiccup: I keep having to add the project path to PYTHONPATH in the export PYTHONPATH=$PYTHONPATH:$(pwd) The tests and coverage run as expected locally, but for some reason, the I have fixed all imports, added the |
This patch fixes the import system in tests. It also updates the structure of the tests directory. Signed-off-by: Aniruddha Karajgi <akarajgi0@gmail.com>
This patch improves tests and the coverage report excluded lines section. Signed-off-by: Aniruddha Karajgi <akarajgi0@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks!
This pull request implements a new naming scheme
for modules. It also fixes a few typos and flake8
formatting errors.