Skip to content

TST Extend tests for scipy.sparse.*array in sklearn/utils/tests/test_seq_dataset.py#27413

Merged
jjerphan merged 7 commits intoscikit-learn:mainfrom
Tialo:tests/test_seq_dataset
Nov 2, 2023
Merged

TST Extend tests for scipy.sparse.*array in sklearn/utils/tests/test_seq_dataset.py#27413
jjerphan merged 7 commits intoscikit-learn:mainfrom
Tialo:tests/test_seq_dataset

Conversation

@Tialo
Copy link
Copy Markdown
Contributor

@Tialo Tialo commented Sep 18, 2023

Reference Issues/PRs

Towards #27090.

What does this implement/fix? Explain your changes.

Any other comments?

File was tricky. If you have some ideas how to make it more neat, I will be glad to hear.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Sep 18, 2023

✔️ Linting Passed

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

Generated for commit: a848336. Link to the linter CI: here

Copy link
Copy Markdown
Member

@glemaitre glemaitre left a comment

Choose a reason for hiding this comment

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

A couple of comments

@glemaitre glemaitre self-requested a review October 2, 2023 08:16
Copy link
Copy Markdown
Member

@glemaitre glemaitre left a comment

Choose a reason for hiding this comment

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

Otherwise LGTM on my side. Thanks @Tialo

@glemaitre glemaitre added the Waiting for Second Reviewer First reviewer is done, need a second one! label Oct 31, 2023
Tialo and others added 2 commits October 31, 2023 19:17
Co-authored-by: Guillaume Lemaitre <g.lemaitre58@gmail.com>
Co-authored-by: Guillaume Lemaitre <g.lemaitre58@gmail.com>
Copy link
Copy Markdown
Member

@jjerphan jjerphan left a comment

Choose a reason for hiding this comment

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

LGTM given two modifications.

@jjerphan jjerphan merged commit 9ee3754 into scikit-learn:main Nov 2, 2023
@Tialo Tialo deleted the tests/test_seq_dataset branch November 2, 2023 12:49
REDVM pushed a commit to REDVM/scikit-learn that referenced this pull request Nov 16, 2023
…st_seq_dataset.py` (scikit-learn#27413)

Co-authored-by: Guillaume Lemaitre <g.lemaitre58@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants