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
Fix nose2 warnings #410
Fix nose2 warnings #410
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.
This is great, thank you!
@@ -460,7 +461,7 @@ def while_loop_body(iteration, mean, variance, sq_dist): | |||
maximum_iterations=n_max_iterations) | |||
|
|||
if last_iteration == n_max_iterations: | |||
print('Maximum number of iterations {} reached.' | |||
warnings.warn('Maximum number of iterations {} reached.' |
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.
The continuous integration tests fail because of this line (flake8 complains). Could you pep8 the style? Thanks!
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.
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.
forgot to run flake8 one last time before submitting PR. Just fixed it
Codecov Report
@@ Coverage Diff @@
## master #410 +/- ##
=========================================
Coverage ? 93.45%
=========================================
Files ? 90
Lines ? 10486
Branches ? 0
=========================================
Hits ? 9800
Misses ? 686
Partials ? 0
Continue to review full report at Codecov.
|
fixes issue #380
ignores UserWarnings on Numpy and PyTorch backends when running the command
nose2 tests
. Only Sklearn deprecated FutureWarning is left in (can be ignored upon request).Unfortunately I'm unable to see the results using the Tensorflow backend due to the following issue and my computer being a bit older:
github.com/tensorflow/tensorflow/issues/17411