Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@OmarManzoor
Copy link
Contributor

Reference Issues/PRs

Closes #20943

What does this implement/fix? Explain your changes.

  • Adds the D2 Brier score which is the D2 score for brier_score_loss

Any other comments?

CC: @lorentzenchr

@github-actions
Copy link

github-actions bot commented May 7, 2024

✔️ Linting Passed

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

Generated for commit: 8b79912. Link to the linter CI: here

@OmarManzoor OmarManzoor changed the title D2 Brier Score FEA D2 Brier Score May 7, 2024
@lorentzenchr
Copy link
Member

#22046 seems like a blocker

@OmarManzoor
Copy link
Contributor Author

@lorentzenchr @ogrisel I updated this PR

Copy link
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.

Thanks for the PR. Here is a first pass of feedback.

Note: once this PR is merged, feel free to follow-up with a PR to add array API support.

@lorentzenchr lorentzenchr added the Waiting for Second Reviewer First reviewer is done, need a second one! label Jul 18, 2025
Copy link
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, thank you very much @OmarManzoor!

@ogrisel ogrisel enabled auto-merge (squash) July 31, 2025 13:47
@ogrisel ogrisel merged commit 810b920 into scikit-learn:main Jul 31, 2025
14 of 17 checks passed
@ogrisel
Copy link
Member

ogrisel commented Jul 31, 2025

GitHub auto-merged with a blue tick without actually re-running the full test suite... Hopefully that won't break main.

@OmarManzoor
Copy link
Contributor Author

Thank you for the reviews @lorentzenchr and @ogrisel

@OmarManzoor OmarManzoor deleted the d2_brier_score branch August 1, 2025 05:41
lucyleeow pushed a commit to lucyleeow/scikit-learn that referenced this pull request Aug 22, 2025
@jeremiedbb jeremiedbb mentioned this pull request Sep 3, 2025
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module:metrics Waiting for Second Reviewer First reviewer is done, need a second one!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add more D2 scores

3 participants