Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[ENH] V1 → V2 API Migration - core structure#1576

Merged
PGijsbers merged 225 commits intoopenml:mainopenml/openml-python:mainfrom
geetu040:migrationgeetu040/openml-python:migrationCopy head branch name to clipboard
Mar 23, 2026
Merged

[ENH] V1 → V2 API Migration - core structure#1576
PGijsbers merged 225 commits intoopenml:mainopenml/openml-python:mainfrom
geetu040:migrationgeetu040/openml-python:migrationCopy head branch name to clipboard

Conversation

@geetu040
Copy link
Collaborator

Towards #1575

This PR sets up the core folder and file structure along with base scaffolding for the API v1 → v2 migration.

It includes:

  • Skeleton for the HTTP client, backend, and API context
  • Abstract resource interfaces and versioned stubs (*V1, *V2)
  • Minimal wiring to allow future version switching and fallback support

No functional endpoints are migrated yet. This PR establishes a stable foundation for subsequent migration and refactor work.

@geetu040 geetu040 mentioned this pull request Dec 30, 2025
18 tasks
@codecov-commenter
Copy link

codecov-commenter commented Dec 31, 2025

Codecov Report

❌ Patch coverage is 51.58850% with 320 lines in your changes missing coverage. Please review.
✅ Project coverage is 52.41%. Comparing base (8a5532f) to head (f926092).

Files with missing lines Patch % Lines
openml/_api/clients/http.py 23.11% 163 Missing ⚠️
openml/_api/resources/base/versions.py 31.16% 53 Missing ⚠️
openml/_api/resources/base/fallback.py 26.31% 28 Missing ⚠️
openml/_api/setup/builder.py 44.44% 25 Missing ⚠️
openml/_api/setup/backend.py 66.66% 19 Missing ⚠️
openml/_api/resources/base/base.py 57.14% 15 Missing ⚠️
openml/_config.py 80.64% 12 Missing ⚠️
openml/cli.py 0.00% 4 Missing ⚠️
openml/_api/clients/minio.py 85.71% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1576      +/-   ##
==========================================
- Coverage   52.73%   52.41%   -0.33%     
==========================================
  Files          37       61      +24     
  Lines        4399     5037     +638     
==========================================
+ Hits         2320     2640     +320     
- Misses       2079     2397     +318     

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

@geetu040
Copy link
Collaborator Author

geetu040 commented Mar 13, 2026

FYI: @JATAYU000 @satvshr @omkar-334 @rohansen856 @EmanAbdelhaleem

Please sync with this PR. The CI tests should now be passing in all the stacked PRs (except some sporadic failures), since now they are using the local docker services created in the CI which provides endpoints for v2-api as well.

You can run these tests locally as well, with the following steps:

Run docker services locally:

git clone --depth 1 https://github.com/openml/services.git
cd ./services        
chmod -R a+rw ./data
chmod -R a+rw ./logs
docker compose --profile rest-api --profile minio --profile evaluation-engine up -d

Run pytest using the local services (add the env var):

OPENML_USE_LOCAL_SERVICES="true" pytest tests/

openml/_config.py Show resolved Hide resolved
openml/_config.py Outdated Show resolved Hide resolved
@geetu040 geetu040 requested a review from PGijsbers March 17, 2026 14:29
Copy link
Collaborator

@PGijsbers PGijsbers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

Copy link
Collaborator

@fkiraly fkiraly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am happy whenever @PGijsbers is happy.

@PGijsbers PGijsbers merged commit 8185ccc into openml:main Mar 23, 2026
50 of 51 checks passed
PGijsbers added a commit that referenced this pull request Mar 24, 2026
Fixes #1623 
Depends on #1576
Related to #1575

#### Details 
This PR implements `Evaluations` resource, and refactor its existing
functions

---------

Co-authored-by: Satvik Mishra <112589278+satvshr@users.noreply.github.com>
Co-authored-by: geetu040 <raoarmaghanshakir040@gmail.com>
Co-authored-by: Franz Király <fkiraly@gcos.ai>
Co-authored-by: Simon Blanke <simon.blanke@yahoo.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Matthias Feurer <lists@matthiasfeurer.de>
Co-authored-by: Pieter Gijsbers <p.gijsbers@tue.nl>
PGijsbers added a commit that referenced this pull request Mar 25, 2026
Fixes #1593
Depends on #1576
Related to #1575

#### Details 
This PR implements `EvaluationMeasures` resource, and refactor its
existing functions

---------

Co-authored-by: Satvik Mishra <112589278+satvshr@users.noreply.github.com>
Co-authored-by: geetu040 <raoarmaghanshakir040@gmail.com>
Co-authored-by: Franz Király <fkiraly@gcos.ai>
Co-authored-by: Simon Blanke <simon.blanke@yahoo.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Matthias Feurer <lists@matthiasfeurer.de>
Co-authored-by: Pieter Gijsbers <p.gijsbers@tue.nl>
PGijsbers added a commit that referenced this pull request Mar 25, 2026
Fixes #1625 
Depends on #1576
Related to #1575

Details
This PR implements `Setups` resource, and refactor its existing
functions

---------

Co-authored-by: Franz Király <fkiraly@gcos.ai>
Co-authored-by: Satvik Mishra <112589278+satvshr@users.noreply.github.com>
Co-authored-by: geetu040 <raoarmaghanshakir040@gmail.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Simon Blanke <simon.blanke@yahoo.com>
Co-authored-by: Matthias Feurer <lists@matthiasfeurer.de>
Co-authored-by: Pieter Gijsbers <p.gijsbers@tue.nl>
PGijsbers added a commit that referenced this pull request Mar 26, 2026
Fixes #1622
Depends on #1576 
Related to: #1575 

#### Details
This PR implements `EstimationProcedures` resource, and refactor its
existing functions

---------

Co-authored-by: Satvik Mishra <112589278+satvshr@users.noreply.github.com>
Co-authored-by: geetu040 <raoarmaghanshakir040@gmail.com>
Co-authored-by: Franz Király <fkiraly@gcos.ai>
Co-authored-by: Simon Blanke <simon.blanke@yahoo.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Matthias Feurer <lists@matthiasfeurer.de>
Co-authored-by: Pieter Gijsbers <p.gijsbers@tue.nl>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.