[ENH] V1 -> V2 Migration : Runs by Omswastik-11 · Pull Request #1616 · openml/openml-python

Omswastik-11 · 2026-01-15T09:09:13Z

Metadata

Reference Issue:
New Tests Added:
Documentation Updated:
Change Log Entry:

Details

codecov-commenter · 2026-01-15T20:58:38Z

Codecov Report

❌ Patch coverage is 56.25000% with 56 lines in your changes missing coverage. Please review.
✅ Project coverage is 54.06%. Comparing base (e653ef6) to head (74eb7ae).

Files with missing lines	Patch %	Lines
openml/_api/resources/run.py	74.39%	21 Missing ⚠️
openml/_api/clients/http.py	16.66%	20 Missing ⚠️
openml/runs/run.py	31.57%	13 Missing ⚠️
openml/runs/functions.py	33.33%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1616      +/-   ##
==========================================
- Coverage   54.67%   54.06%   -0.62%     
==========================================
  Files          63       63              
  Lines        5108     5159      +51     
==========================================
- Hits         2793     2789       -4     
- Misses       2315     2370      +55

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

…u040/openml-python into runs-migration-stacked # Conflicts: # openml/_api/resources/__init__.py # openml/_api/runtime/core.py

…into runs-migration-stacked

geetu040

sync with base pr
sdk code look good so far, please take a look at #1575 (comment) and make changes accordingly where needed.
all tests (existing and new) should pass to make sure we are retaining the original functionality of the sdk

Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>

…into runs-migration-stacked

Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>

…-11/openml-python into runs-migration-stacked

Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>

geetu040 · 2026-01-30T14:10:51Z

+    use_cache = not ignore_cache
+    reset_cache = ignore_cache
+    return api_context.backend.runs.get(
+        run_id,
+        use_cache=use_cache,
+        reset_cache=reset_cache,
+    )


use_cache should be true since the method always supports caching
reset_cache should rely on ignore_cache

Copilot

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

Copilot

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

geetu040 · 2026-04-03T11:51:34Z

you can remove this function now, since it's not used anywhere

geetu040 · 2026-04-03T11:51:40Z

same as above

Omswastik-11 · 2026-04-26T21:30:37Z

Hi @geetu040 . As per discussion in final mentoring meeting isn't it ready to merge ?

geetu040

overall this is close to getting merged, just a few review comments

geetu040 · 2026-05-01T07:07:20Z

following on the last unresolved review comment https://github.com/openml/openml-python/pull/1616/changes#r2988760874
what are these changes? is it a merge fault?

This was because of this https://github.com/openml/openml-python/actions/runs/25209014637/job/73915466617?pr=1616 . I had updated it . Should I look for a alternate solution ?

Hi @geetu040

8da79d3

I have override the publish to pass the tests . Is it good or should I come up with better solution ?

Co-authored-by: Armaghan Shakir <raoarmaghanshakir040@gmail.com>

Copilot

Pull request overview

Copilot reviewed 9 out of 10 changed files in this pull request and generated 3 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>

Copilot

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 5 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>

…-11/openml-python into runs-migration-stacked

…rver Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>

Omswastik-11 · 2026-05-02T13:35:00Z

Hi @geetu040 !! Can you check the replies to reviews and the failing tests and let me know if any changes needed ?

geetu040 mentioned this pull request Jan 15, 2026

[ENH] V1 → V2 API Migration #1575

Open

18 tasks

geetu040 assigned Omswastik-11 Jan 19, 2026

geetu040 and others added 11 commits January 27, 2026 14:17

implement tag/untag

54a3151

implement fallback

2b6fe65

Merge branches 'migration' and 'migration' of https://github.com/geet…

29fd985

…u040/openml-python into runs-migration-stacked # Conflicts: # openml/_api/resources/__init__.py # openml/_api/runtime/core.py

add test_http.py

fa53f8d

add uses_test_server marker

2b2db96

Merge branch 'migration' of https://github.com/geetu040/openml-python …

0135ba2

…into runs-migration-stacked

implement reset_cache

c9617f9

fixes with publish/delete

5bc37b8

fix cache_key in tests

08d9916

Merge branch 'migration' of https://github.com/geetu040/openml-python …

37a468f

…into runs-migration-stacked

update _not_supported

8caba11

geetu040 suggested changes Jan 30, 2026

View reviewed changes

Comment thread openml/_api/resources/runs.py Outdated

Comment thread openml/_api/resources/runs.py Outdated

Comment thread openml/_api/resources/runs.py Outdated

Comment thread openml/runs/functions.py Outdated

Omswastik-11 added 3 commits January 30, 2026 14:56

tests:added tests for migration

a70a33f

Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>

Merge branch 'migration' of https://github.com/geetu040/openml-python …

d08b1fe

…into runs-migration-stacked

tests:added tests for migration

aaf9d4b

Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>

Omswastik-11 requested a review from geetu040 January 30, 2026 09:50

Merge branch 'main' into runs-migration-stacked

84d43a9

Omswastik-11 marked this pull request as ready for review January 30, 2026 09:50

Omswastik-11 added 6 commits January 30, 2026 18:29

tests:modified old chache tests to use new caching

761ef9e

Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>

Merge branch 'runs-migration-stacked' of https://github.com/Omswastik…

cf37a75

…-11/openml-python into runs-migration-stacked

tests:modified old chache tests to use new caching

2eb7c7a

Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>

tests:modified old chache tests to use new caching

d9476c8

Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>

tests:skip the production related tests

f4718c1

Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>

tests:modify skip messgaes

4560b6b

Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>

geetu040 suggested changes Jan 30, 2026

View reviewed changes

SimonBlanke added 2 commits January 30, 2026 19:27

add 'get_api_config' skeleton method

1913c10

remove 'APISettings'

7681949

Copilot AI review requested due to automatic review settings April 3, 2026 11:07

Copilot AI reviewed Apr 3, 2026

View reviewed changes

geetu040 added 2 commits April 3, 2026 16:17

update run_id

f5f0df1

use cached run in test_api/test_run.py

9accf24

Copilot AI review requested due to automatic review settings April 3, 2026 11:40

Copilot AI reviewed Apr 3, 2026

View reviewed changes

geetu040 suggested changes Apr 3, 2026

View reviewed changes

Omswastik-11 added 2 commits April 3, 2026 17:58

remove unused func

229592f

remove old api calls from OpenMLRun classes

f6e6516

Omswastik-11 requested a review from geetu040 April 3, 2026 13:19

geetu040 suggested changes May 1, 2026

View reviewed changes

Update tests/test_api/test_run.py

15d3403

Co-authored-by: Armaghan Shakir <raoarmaghanshakir040@gmail.com>

Copilot AI review requested due to automatic review settings May 1, 2026 08:44

Copilot started reviewing on behalf of Omswastik-11 May 1, 2026 08:44 View session

Copilot AI reviewed May 1, 2026

View reviewed changes

Comment thread openml/runs/run.py

Comment thread openml/_api/clients/http.py

Comment thread openml/_api/resources/run.py

Omswastik-11 added 2 commits May 1, 2026 14:30

remove old apicalls

6da5428

Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>

fix the merge issues on versions file

91ef60c

Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>

Copilot AI review requested due to automatic review settings May 1, 2026 09:04

Copilot started reviewing on behalf of Omswastik-11 May 1, 2026 09:05 View session

Copilot AI reviewed May 1, 2026

View reviewed changes

Comment thread openml/runs/run.py

Comment thread tests/test_api/test_run.py

Comment thread openml/_api/clients/http.py

Comment thread openml/_api/clients/http.py

Comment thread openml/_api/resources/base/resources.py

Omswastik-11 added 2 commits May 1, 2026 15:41

override publish and add runid falllback

8da79d3

Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>

Merge branch 'runs-migration-stacked' of https://github.com/Omswastik…

1bad874

…-11/openml-python into runs-migration-stacked

Omswastik-11 requested a review from geetu040 May 1, 2026 10:17

modified the test to use runid 25 which is available on local test se…

ce66423

…rver Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>

Copilot AI review requested due to automatic review settings May 1, 2026 11:29

Copilot started reviewing on behalf of Omswastik-11 May 1, 2026 11:30 View session

Copilot AI reviewed May 1, 2026

View reviewed changes

add fallback for unavailable runs on local and test server

74eb7ae

Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>

Uh oh!

Conversation

Omswastik-11 commented Jan 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Metadata

Details

Uh oh!

codecov-commenter commented Jan 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

geetu040 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

geetu040 Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

geetu040 Apr 3, 2026

Choose a reason for hiding this comment

Uh oh!

geetu040 Apr 3, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Omswastik-11 commented Apr 26, 2026

Uh oh!

geetu040 left a comment

Choose a reason for hiding this comment

Uh oh!

geetu040 May 1, 2026

Choose a reason for hiding this comment

Uh oh!

Omswastik-11 May 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Omswastik-11 May 1, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Omswastik-11 commented May 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Omswastik-11 commented Jan 15, 2026 •

edited

Loading

codecov-commenter commented Jan 15, 2026 •

edited

Loading

Omswastik-11 May 1, 2026 •

edited

Loading

Omswastik-11 commented May 2, 2026 •

edited

Loading