Skip to content

Conversation

R-Palazzo
Copy link
Contributor

Resolve #734
CU-86b437kec

@R-Palazzo R-Palazzo requested review from frances-h and fealho March 18, 2025 16:10
@R-Palazzo R-Palazzo self-assigned this Mar 18, 2025
@R-Palazzo R-Palazzo requested a review from a team as a code owner March 18, 2025 16:10
@sdv-team
Copy link
Contributor

@R-Palazzo R-Palazzo removed the request for review from a team March 18, 2025 16:10
Copy link

codecov bot commented Mar 18, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.43%. Comparing base (e915eb8) to head (03ca135).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #749   +/-   ##
=======================================
  Coverage   95.43%   95.43%           
=======================================
  Files         112      112           
  Lines        4379     4379           
=======================================
  Hits         4179     4179           
  Misses        200      200           
Flag Coverage Δ
integration 79.72% <ø> (ø)
unit 83.39% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Base automatically changed from xgboost_version_pin to main March 18, 2025 18:01
@R-Palazzo R-Palazzo force-pushed the issue-734-default-values-binary-classifier branch from 2d9ce8e to 03ca135 Compare March 19, 2025 09:25
@R-Palazzo R-Palazzo merged commit bff173f into main Mar 19, 2025
55 checks passed
@R-Palazzo R-Palazzo deleted the issue-734-default-values-binary-classifier branch March 19, 2025 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wrong default values for the classifier parameter of the BinaryClassifierPrecisionEfficacy
4 participants