Skip to content

Conversation

@PGijsbers
Copy link
Collaborator

No description provided.

LennartPurucker and others added 22 commits April 18, 2023 15:17
… Homepage; More documentation for `components` (#1243)

* fix task ID for Iris task

* update comment on homepage

* added additional documentation specific to the `components` parameter.

* add change to progress.rst

* Fix dataframe append being deprecated by replacing it with (backwards-compatible) pd.concat

* fix logging example and add new changes to progress.rst

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* fix comment too long

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/psf/black: 22.6.0 → 23.3.0](psf/black@22.6.0...23.3.0)
- [github.com/pre-commit/mirrors-mypy: v0.961 → v1.2.0](pre-commit/mirrors-mypy@v0.961...v1.2.0)
- [github.com/pycqa/flake8: 4.0.1 → 6.0.0](PyCQA/flake8@4.0.1...6.0.0)

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Fix mypy errors: made implicit optional typing to be explicit

* Drop duplicate flake8 config

* Fix a few flake8 issues

* Update python version for pre-commit workflow

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Lennart Purucker <[email protected]>
Co-authored-by: Matthias Feurer <[email protected]>
updates:
- [github.com/pre-commit/mirrors-mypy: v1.2.0 → v1.3.0](pre-commit/mirrors-mypy@v1.2.0...v1.3.0)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…cation (#1246)

* Update runs/functions.py

Added warning to run_model_on_task to avoid duplicates if no authentication

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Update functions.py

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* Fix #1124

Make variable `openml.config.cache_directory` private so that
there is no confusion on how to retrieve the cache directory
(since this should be done via `openml.config.get_cache_directory`)

* Improve docstrings and method names

* Rename base_ to root_

* Update based on Pieter's feedback
* made dataset features optional

* fix check for qualities

* add lazy loading for dataset metadata and add option to refresh cache

* adjust progress.rst

* minor fixes

* break line to keep link and respect line length

* [no ci] changes for pull request review

* refactor and add cache usage to load_metadata

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* fix precommit

* [no ci] adjust task loading to new dataset loading

* [no ci] add actual lazy loading based on properties and adjusted test on how to use it

* switch deprecation to future warning, adjusted deprecation cycle to version 0.15.0, update documentation.

* Update openml/tasks/functions.py

Co-authored-by: Matthias Feurer <[email protected]>

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Matthias Feurer <[email protected]>
)

* made dataset features optional

* fix check for qualities

* add lazy loading for dataset metadata and add option to refresh cache

* adjust progress.rst

* minor fixes

* break line to keep link and respect line length

* [no ci] changes for pull request review

* refactor and add cache usage to load_metadata

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* fix precommit

* [no ci] adjust task loading to new dataset loading

* [no ci] add actual lazy loading based on properties and adjusted test on how to use it

* switch deprecation to future warning, adjusted deprecation cycle to version 0.15.0, update documentation.

* Update openml/tasks/functions.py

Co-authored-by: Matthias Feurer <[email protected]>

* changes based on pr review feedback

* fix test w.r.t. server state

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Matthias Feurer <[email protected]>
* Add mypy annotations for _api_calls.py

This commit adds mypy annotations for _api_calls.py to make
sure that we have no untyped classes and functions.

* Include Pieters suggestions
* Add deprecation warning for retrieving  dict

* Refactor check_datasets_active to work with dataframe

* Update unit tests to use list_datasets with output_format dataframe

* Move list_datasets test to proper file

* Remove list_datasets test, duplicate in test_datasets_functions

duplicate of tests/test_datasets/test_dataset_functions.py::test_list_datasets

* Update list_flows calls to use output_format='dataframe'

* Update list_runs calls to require dataframe output

* Update list_setup calls for deprecation

* Update list_study calls

* Update list_tasks to specify output_format dataframe

* Add `output_format` to `list_datasets` call

* Add TODO markers for removing `dict` support of `list_*` functions

* Make status check less strict, call list_dataset with output_format

* Change index on id to did, since thats the dataset id's column name

* Update test to reflect new error message

* Fix bug introduced by refactor

Must check results are (somewhat) complete before processing results

* Fix minor oversights of refactoring

* Rename variables to reflect they are no longer lists

* Fix unsafe indexing on dataframe and remaining unit tests

* Perform safer check for integer dtypes
)

It is not allowed to attach a run to a study which is already
associated with the study. This leads to a misleading error:
`1045: Problem attaching entities. Please ensure to only attach entities that exist - None`
updates:
- [github.com/pre-commit/mirrors-mypy: v1.3.0 → v1.4.1](pre-commit/mirrors-mypy@v1.3.0...v1.4.1)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* Add future warning for more user-facing functions that return arrays

* Use dataframe instead of array, as array will be deprecated

* Update for 0.15 release

* Update for 0.15.0 release that phases out arrays

* Fix mistakes introduced by switching to default dataframe
* Bump version number and add changelog

* Incorporate feedback from Pieter

* Fix unit test

* Make assert less strict

* Update release notes

* Fix indent
which conflicts with internals of scikit-learn 0.24.2
…1272)

* Allow fallback to ARFF on ServerError and make explicit in warning

* Remove accidental changelog header duplication
Co-authored-by: Matthias Feurer <[email protected]>
@PGijsbers PGijsbers closed this Jul 20, 2023
@PGijsbers PGijsbers reopened this Jul 20, 2023
@PGijsbers PGijsbers merged commit 0639882 into main Jul 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants