Skip to content

TST Extend tests for scipy.sparse.*array in sklearn/cluster/tests/test_hierarchical.py#27101

Merged
ogrisel merged 10 commits intoscikit-learn:mainfrom
vvkrddy:branch1
Aug 28, 2023
Merged

TST Extend tests for scipy.sparse.*array in sklearn/cluster/tests/test_hierarchical.py#27101
ogrisel merged 10 commits intoscikit-learn:mainfrom
vvkrddy:branch1

Conversation

@vvkrddy
Copy link
Copy Markdown
Contributor

@vvkrddy vvkrddy commented Aug 18, 2023

Reference Issues/PRs

Towards #27090.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Aug 18, 2023

✔️ Linting Passed

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

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

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.

Thanks for the PR @vvkrddy. Here are a few comments.

@ogrisel
Copy link
Copy Markdown
Member

ogrisel commented Aug 25, 2023

@vvkrddy could you please fix the linting issues by following the instructions in the comments by the linting bot?

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!

@ogrisel ogrisel enabled auto-merge (squash) August 28, 2023 09:13
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 @vvkrddy

@ogrisel ogrisel merged commit 8e14bd0 into scikit-learn:main Aug 28, 2023
@vvkrddy vvkrddy deleted the branch1 branch August 28, 2023 09:44
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Aug 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants