Skip to content

check_psd_eigenvalues : fixed issue with small_pos_ratio on mac os x with python >= 3.7#18270

Merged
glemaitre merged 1 commit intoscikit-learn:masterfrom
smarie:fix_issue_18146_2
Aug 27, 2020
Merged

check_psd_eigenvalues : fixed issue with small_pos_ratio on mac os x with python >= 3.7#18270
glemaitre merged 1 commit intoscikit-learn:masterfrom
smarie:fix_issue_18146_2

Conversation

@smarie
Copy link
Copy Markdown
Contributor

@smarie smarie commented Aug 27, 2020

Fixed issue with small_pos_ratio on mac os x with python >= 3.7 by slightly raising threshold as discussed in #18149.
Updated docstring accordingly.

Copy link
Copy Markdown
Member

@alfaro96 alfaro96 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @smarie, this solution should fix the issue!

Let us wait for the green in the CI.

@glemaitre
Copy link
Copy Markdown
Member

It looks good. I am really thinking that we should better handle these issues in tests with a global strategy.

@glemaitre glemaitre merged commit 0bd45b5 into scikit-learn:master Aug 27, 2020
@alfaro96
Copy link
Copy Markdown
Member

alfaro96 commented Aug 28, 2020

Happy to see that this solution works and the wheel builder is "building" again 😄 (see https://dev.azure.com/scikit-learn/scikit-learn/_build/results?buildId=21578&view=results).

Thank you @smarie for the PR and @glemaitre for the review!

@smarie smarie deleted the fix_issue_18146_2 branch August 28, 2020 12:29
jayzed82 pushed a commit to jayzed82/scikit-learn that referenced this pull request Oct 22, 2020
…ue to floating point precision issue (scikit-learn#18270)

Co-authored-by: Sylvain MARIE <sylvain.marie@se.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants