8000 TST Extend tests for scipy.sparse.*array in sklearn/datasets/tests/test_svmlight_format.py by akeemlh · Pull Request #27248 · scikit-learn/scikit-learn · GitHub
[go: up one dir, main page]
Skip to content

TST Extend tests for scipy.sparse.*array in sklearn/datasets/tests/test_svmlight_format.py #27248

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

TST Extend tests for scipy.sparse.*array in sklearn/datasets/tests/test_svmlight_format.py #27248

wants to merge 2 commits into from

Conversation

akeemlh
Copy link
@akeemlh akeemlh commented Aug 31, 2023

Reference Issues/PRs

Contributes to #27090

What does this implement/fix? Explain your changes.

Any other comments?

@github-actions
Copy link
github-actions bot commented Aug 31, 2023 •
edited
Loading

✔️ Linting Passed

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

Generated for commit: 0bea275. Link to the linter CI: here

@jjerphan jjerphan mentioned this pull request Aug 31, 2023
Closed
@Charlie-XIAO
Copy link
Contributor

@akeemlh Seems that there is already #27220 adding tests for this file which is earlier?

@akeemlh
Copy link
Author
akeemlh commented Sep 3, 2023

@akeemlh Seems that there is already #27220 adding tests for this file which is earlier?

Looks like you're right, I didn't catch that

@glemaitre
Copy link
Member

Closing in favor of @27220

@glemaitre glemaitre closed this Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
module:datasets
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@akeemlh @Charlie-XIAO @glemaitre
0