Skip to content

Improve error message for inconsistent fold counts in OpenMLSplit#1512

Closed
axif0 wants to merge 1 commit into
openml:mainfrom
axif0:error
Closed

Improve error message for inconsistent fold counts in OpenMLSplit#1512
axif0 wants to merge 1 commit into
openml:mainfrom
axif0:error

Conversation

@axif0
Copy link
Copy Markdown

@axif0 axif0 commented Nov 25, 2025

Replaced the generic any() check with an explicit loop to identify the mismatched repetition.
Implemented a descriptive error message that specifies which repetition has the incorrect number of folds and compares it to the expected count (from repetition 0).

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 56.42%. Comparing base (4b1bdf4) to head (91e5abf).

Files with missing lines Patch % Lines
openml/tasks/split.py 75.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           develop    #1512       +/-   ##
============================================
- Coverage    79.90%   56.42%   -23.48%     
============================================
  Files           36       36               
  Lines         4349     4351        +2     
============================================
- Hits          3475     2455     -1020     
- Misses         874     1896     +1022     

☔ 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.

@axif0 axif0 closed this by deleting the head repository May 24, 2026
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.

2 participants