[ENH] V1 → V2 API Migration - core structure#1576
[ENH] V1 → V2 API Migration - core structure#1576PGijsbers merged 225 commits intoopenml:mainopenml/openml-python:mainfrom
Conversation
…into issue1564
Codecov Report❌ Patch coverage is 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. 🚀 New features to boost your workflow:
|
This reverts commit 93155ee.
|
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 -dRun pytest using the local services (add the env var): OPENML_USE_LOCAL_SERVICES="true" pytest tests/ |
fkiraly
left a comment
There was a problem hiding this comment.
I am happy whenever @PGijsbers is happy.
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>
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>
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>
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>
Towards #1575
This PR sets up the core folder and file structure along with base scaffolding for the API v1 → v2 migration.
It includes:
*V1,*V2)No functional endpoints are migrated yet. This PR establishes a stable foundation for subsequent migration and refactor work.