Skip to content

FEAT SLEP006 permutation_test_score to support metadata routing #29266

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.

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

Merged
merged 50 commits into from
Jul 11, 2024

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>
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
28 checks passed
@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