-
-
Notifications
You must be signed in to change notification settings - Fork 25.2k
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
MNT fix cd_fast.pyx warning issues #13397
Conversation
This also makes the tests in #12682 pass :) |
can we test this some way? maybe making sure that something doesn't raise a warning? |
I'll try to add some tests. |
otherwise looks good btw |
Added some tests, #12682 also include some tests checking for presence and absence of convergence warnings. |
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.
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.
Sorry for messing up last week ...
I suppose I can merge then :) |
Good job!!! |
Sweet, thanks! |
* fix warning issues * add comments * add some tests * pep8
This reverts commit 7bcc701.
This reverts commit 7bcc701.
* fix warning issues * add comments * add some tests * pep8
Fixes #13394
This fixes issues with the "Objective did not converge...." warnings.
There were some indentation and missing
else
issues.