Skip to content

Conversation

@adam2392
Copy link
Member

Reference Issues/PRs

Towards: #22893

What does this implement/fix? Explain your changes.

  • adds metadata routing to the function permutation_test_score.

Any other comments?

adam2392 and others added 30 commits February 21, 2024 10:16
Signed-off-by: Adam Li <adam2392@gmail.com>
Signed-off-by: Adam Li <adam2392@gmail.com>
Signed-off-by: Adam Li <adam2392@gmail.com>
Signed-off-by: Adam Li <adam2392@gmail.com>
Signed-off-by: Adam Li <adam2392@gmail.com>
Signed-off-by: Adam Li <adam2392@gmail.com>
Signed-off-by: Adam Li <adam2392@gmail.com>
Signed-off-by: Adam Li <adam2392@gmail.com>
Signed-off-by: Adam Li <adam2392@gmail.com>
Co-authored-by: Adrin Jalali <adrin.jalali@gmail.com>
Signed-off-by: Adam Li <adam2392@gmail.com>
Signed-off-by: Adam Li <adam2392@gmail.com>
Signed-off-by: Adam Li <adam2392@gmail.com>
Signed-off-by: Adam Li <adam2392@gmail.com>
Signed-off-by: Adam Li <adam2392@gmail.com>
Signed-off-by: Adam Li <adam2392@gmail.com>
Signed-off-by: Adam Li <adam2392@gmail.com>
Signed-off-by: Adam Li <adam2392@gmail.com>
Signed-off-by: Adam Li <adam2392@gmail.com>
@github-actions
Copy link

github-actions bot commented Jun 14, 2024

✔️ Linting Passed

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

Generated for commit: 5ad2f91. Link to the linter CI: here

Signed-off-by: Adam Li <adam2392@gmail.com>
@adam2392 adam2392 marked this pull request as ready for review June 17, 2024 21:35
@adam2392
Copy link
Member Author

adam2392 commented Jun 17, 2024

Oops I think I based the branch off of #28494. But I suppose this one can get reviewed after that one is merged.

… unless that one has serious issues. Let me know :p

adam2392 added 4 commits June 17, 2024 21:13
Signed-off-by: Adam Li <adam2392@gmail.com>
Signed-off-by: Adam Li <adam2392@gmail.com>
Signed-off-by: Adam Li <adam2392@gmail.com>
Signed-off-by: Adam Li <adam2392@gmail.com>
@adam2392 adam2392 requested a review from adrinjalali July 9, 2024 11:26
Copy link
Member

@adrinjalali adrinjalali left a comment

Choose a reason for hiding this comment

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

Nice! Otherwise LGTM.

cc @OmarManzoor @glemaitre for a second review.

Copy link
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 @adam2392

@adrinjalali adrinjalali enabled auto-merge (squash) July 11, 2024 12:22
@adrinjalali adrinjalali disabled auto-merge July 11, 2024 12:22
@OmarManzoor OmarManzoor enabled auto-merge (squash) July 11, 2024 12:37
@OmarManzoor OmarManzoor merged commit afee65a into scikit-learn:main Jul 11, 2024
@adam2392 adam2392 deleted the slep006/permutation branch July 11, 2024 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants