Skip to content

Conversation

thomasjpfan
Copy link
Member

Conda-forge is failing with CPython tests, because of the strictness of the test_linear_regression_sample_weight_consistency test. Here is an example of a failing test.

@jeremiedbb

@github-actions
Copy link

Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

@jeremiedbb jeremiedbb added No Changelog Needed To backport PR merged in master that need a backport to a release branch defined based on the milestone. labels Jun 20, 2023
Copy link
Member

@jeremiedbb jeremiedbb left a comment

Choose a reason for hiding this comment

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

LGTM.

@betatim
Copy link
Member

betatim commented Jun 20, 2023

Did it always (randomly) fail or is it a new failure? If it is new it could be interesting to understand why it started. The current tolerance doesn't look too crazy(?)

@thomasjpfan
Copy link
Member Author

thomasjpfan commented Jun 20, 2023

It was not random. It failed on another conda-forge PR for PyPy on the Aarch64 + Linux + CPython job.

Copy link
Member

@betatim betatim left a comment

Choose a reason for hiding this comment

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

Seems weird that it would start failing, but I think we have bigger fish to fry.

@ogrisel
Copy link
Member

ogrisel commented Jun 20, 2023

The circle ci failure is unrelated and probably transient ("Downloaded bytes did not match Content-Length" for libcups. Merging.

@ogrisel ogrisel merged commit 4986838 into scikit-learn:main Jun 20, 2023
thomasjpfan added a commit to thomasjpfan/scikit-learn that referenced this pull request Jun 20, 2023
thomasjpfan added a commit to thomasjpfan/scikit-learn that referenced this pull request Jun 20, 2023
jeremiedbb pushed a commit to jeremiedbb/scikit-learn that referenced this pull request Jun 29, 2023
REDVM pushed a commit to REDVM/scikit-learn that referenced this pull request Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build / CI module:linear_model No Changelog Needed To backport PR merged in master that need a backport to a release branch defined based on the milestone.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants