Skip to content

Fix duplicate benchmark_suite parameter in create_study docs - #1748

Merged
L-Agbaria merged 2 commits into
openml:mainfrom
RugvedBane:fix-1744
Sep 25, 2026
Merged

L-Agbaria merged 2 commits into
openml:mainfrom
RugvedBane:fix-1744

Conversation

@RugvedBane

Copy link
Copy Markdown
Contributor

Metadata

Details

Removed the duplicate benchmark_suite parameter entry from the create_study docstring in openml/study/functions.py.

The parameter was incorrectly documented twice, with one occurrence appearing before name and the correct occurrence already present later in the parameter list.

This is a documentation-only fix and does not change the behavior of create_study.

@codecov-commenter

codecov-commenter commented Sep 24, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.32%. Comparing base (26e35aa) to head (7c8c82a).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1748      +/-   ##
==========================================
- Coverage   81.50%   81.32%   -0.18%     
==========================================
  Files          63       63              
  Lines        5238     5238              
==========================================
- Hits         4269     4260       -9     
- Misses        969      978       +9     

☔ View full report in Codecov by Harness.
📢 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.

@L-Agbaria

Copy link
Copy Markdown
Contributor

The failing test appear to be due to the test requesting a list of more than 1000 entries, which is blocked by a rate limit of 1000. Will look into this and get back to you soon, thank you for the help!

@L-Agbaria
L-Agbaria merged commit 4a25027 into openml:main Sep 25, 2026
23 of 27 checks passed
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.

[BUG] Documentation for study/functions/create_study has duplicate parameter

3 participants