Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

MAINT Add @fails_if_pypy to test_nonnegative_hashing_vectorizer_result_indices #19173

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

Merged

Conversation

ogrisel
Copy link
Member

@ogrisel ogrisel commented Jan 14, 2021

We got a nightly build failure on circle ci for the PyPy job because of a new test for HashingVectorizer that raises NotImplementedError.

Details:

https://app.circleci.com/pipelines/github/scikit-learn/scikit-learn/10754/workflows/229b09b0-27fd-41dc-840f-c96c910a8c07/jobs/129460

@ogrisel ogrisel added this to the 0.24.1 milestone Jan 14, 2021
@ogrisel ogrisel added the To backport PR merged in master that need a backport to a release branch defined based on the milestone. label Jan 14, 2021
@ogrisel
Copy link
Member Author

ogrisel commented Jan 14, 2021

We need a backport in the 0.24.X branch to make it possible to release the PyPy artifacts for 0.24.1 on conda-forge.

Copy link
Member

@thomasjpfan thomasjpfan left a comment

Choose a reason for hiding this comment

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

LGTM

@ogrisel ogrisel merged commit d626d77 into scikit-learn:master Jan 14, 2021
@ogrisel ogrisel deleted the pypy-hashingvectorizer-test-skip branch January 14, 2021 15:36
@ogrisel
Copy link
Member Author

ogrisel commented Jan 14, 2021

There is no commit message marker to try it on the circle ci pypy job as part of this PR, so merging directly.

@glemaitre glemaitre mentioned this pull request Apr 22, 2021
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug module:test-suite everything related to our tests 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.

2 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.