Skip to content

TST Extend tests for scipy.sparse.*arrayin sklearn/ensemble/tests/test_bagging.py#27170

Merged
ogrisel merged 6 commits intoscikit-learn:mainfrom
work-mohit:fix/ensemble/tests/test_bagging.py
Sep 11, 2023
Merged

TST Extend tests for scipy.sparse.*arrayin sklearn/ensemble/tests/test_bagging.py#27170
ogrisel merged 6 commits intoscikit-learn:mainfrom
work-mohit:fix/ensemble/tests/test_bagging.py

Conversation

@work-mohit
Copy link
Copy Markdown
Contributor

Towards #27090

@github-actions
Copy link
Copy Markdown

github-actions bot commented Aug 25, 2023

✔️ Linting Passed

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

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

@work-mohit work-mohit changed the title TST Extend tests for scipy.sparse.*arrayin sklearn\ensemble\tests\test_polynomial.py TST Extend tests for scipy.sparse.*arrayin sklearn\ensemble\tests\test_bagging.py Aug 25, 2023
@jjerphan jjerphan changed the title TST Extend tests for scipy.sparse.*arrayin sklearn\ensemble\tests\test_bagging.py TST Extend tests for scipy.sparse.*arrayin sklearn/ensemble/tests/test_bagging.py Aug 27, 2023
Copy link
Copy Markdown
Contributor

@OmarManzoor OmarManzoor left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks @work-mohit

Copy link
Copy Markdown
Member

@ogrisel ogrisel left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the PR!

@ogrisel ogrisel merged commit 358a1c9 into scikit-learn:main Sep 11, 2023
@work-mohit work-mohit deleted the fix/ensemble/tests/test_bagging.py branch September 11, 2023 15:19
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants