[ENH] V1 → V2 API Migration - studies

[rohansen856]

Metadata

• Reference Issue: [ENH] V1 → V2 API Migration - studies #1594 (towards [ENH] V1 → V2 API Migration #1575)
• New Tests Added: No
• Documentation Updated: No

Details

Stackend PR, Depends on #1576

This PR adds Studies v2 migration.

A question:
Due to the pre commit hook i could not put 6 arguments in a function, so i had to workaround that with this instead:
openml_api\resources\studies.py (line 10-15)

        limit = kwargs.get("limit")
        offset = kwargs.get("offset")
        status = kwargs.get("status")
        main_entity_type = kwargs.get("main_entity_type")
        uploader = kwargs.get("uploader")
        benchmark_suite = kwargs.get("benchmark_suite")

I would like to confirm if this approach is correct or not. Raising a draft PR for now.

[codecov-commenter]

Codecov Report

❌ Patch coverage is 78.94737% with 52 lines in your changes missing coverage. Please review.
✅ Project coverage is 54.20%. Comparing base (c5f68bf) to head (88077a7).

Files with missing lines	Patch %	Lines
openml/_api/resources/studies.py	30.43%	16 Missing ⚠️
openml/_api/http/client.py	82.60%	12 Missing ⚠️
openml/_api/resources/tasks.py	87.23%	6 Missing ⚠️
openml/_api/runtime/fallback.py	0.00%	6 Missing ⚠️
openml/_api/runtime/core.py	82.14%	5 Missing ⚠️
openml/_api/resources/datasets.py	77.77%	2 Missing ⚠️
openml/study/functions.py	66.66%	2 Missing ⚠️
openml/_api/__init__.py	75.00%	1 Missing ⚠️
openml/_api/config.py	96.87%	1 Missing ⚠️
openml/tasks/functions.py	87.50%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1610      +/-   ##
==========================================
+ Coverage   53.02%   54.20%   +1.17%     
==========================================
  Files          36       47      +11     
  Lines        4326     4555     +229     
==========================================
+ Hits         2294     2469     +175     
- Misses       2032     2086      +54     

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

[rohansen856]

Implementing noqa instead of the kwargs following example from here: openml\testing.py:

    def _check_fold_timing_evaluations(  # noqa: PLR0913
        self,
        fold_evaluations: dict[str, dict[int, dict[int, float]]],
        num_repeats: int,
        num_folds: int,
        *,
        max_time_allowed: float = 60000.0,
        task_type: TaskType = TaskType.SUPERVISED_CLASSIFICATION,
        check_scores: bool = True,
    ) -> None:

Final function signature:

    def list(  # noqa: PLR0913
        self,
        limit: int | None = None,
        offset: int | None = None,
        status: str | None = None,
        main_entity_type: str | None = None,
        uploader: list[int] | None = None,
        benchmark_suite: int | None = None,
    ) -> Any:

[geetu040]

Good work. Just use the listing as suggested in #1575 (comment) which is already similar to what you have done.