[Prachi194agrawal]

Description

This PR fixes #1210 by improving the warning message when avoid_duplicate_runs=True is used without an API key.

Changes:

• Replaced the generic RuntimeWarning with a UserWarning.
• Updated the warning message to explicitly inform the user that duplicate runs cannot be checked server-side without authentication.
• Changed the condition check to explicitly look for openml.config.apikey is None.

Type of change

• Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

I verified this locally by mocking openml.runs.run_model_on_task with apikey=None and asserting that the new UserWarning is triggered correctly.

[Prachi194agrawal]

Hi @PGijsbers,

I have submitted this fix for #1210. I updated the warning to a UserWarning and added the specific check for the missing API key as discussed.

Could you please review this when you have a moment? It looks like the GitHub workflows (CI) are waiting for approval to run as well.

Thanks!

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

[codecov-commenter]

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 51.24%. Comparing base (496ac32) to head (377af26).

Files with missing lines	Patch %	Lines
openml/runs/functions.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1484      +/-   ##
==========================================
- Coverage   51.41%   51.24%   -0.17%     
==========================================
  Files          63       63              
  Lines        5565     5565              
==========================================
- Hits         2861     2852       -9     
- Misses       2704     2713       +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.

[PGijsbers]

Thanks for taking the time to contribute. We were flooded with PRs and unable to process them, so sorry for the delay in our response. The linked issue was supposed to be closed. However, I do think the clarification that the run proceeds with the risk of duplicates is a useful addition. Moreover, the UserWarning is probably more appropriate than the RuntimeWarning. So I am merging this now. Thanks again!