Skip to content

Conversation

@mfeurer
Copy link
Collaborator

@mfeurer mfeurer commented May 20, 2021

This PR is for releasing OpenML-Python 0.12.2.

PGijsbers and others added 16 commits April 20, 2021 09:48
* Add a non-functional entry point

* Allow setting of API key through CLI

 - Add function to set any field in the configuration file
 - Add function to read out the configuration file
 - Towards full configurability from CLI

* Remove autocomplete promise, use _defaults

Autocomplete seems to be incompatible with `choices`, so I'll ignore
that for now. We also use `config._defaults` instead of an explicit
list to avoid duplication.

* Add server configuration

* Allow fields to be set directly non-interactively

With the `openml configure FIELD VALUE` command.

* Combine error and check functionalities

Otherwise you have to duplicate all checks in the error message
function.

* Share logic about setting/collecting the value

* Complete CLI for other fields.

Max_retries is excluded because it should not be user configurable, and
will most likely be removed.
Verbosity is configurable but is currently not actually used.

* Bring back sanitizing user input

And extend it to the bool inputs.

* Add small bit of info about the command line tool

* Add API key configuration note in the introduction

* Add to progress log

* Refactor flow of wait_until_valid_input
* Speed up dataset unit tests

by only loading necessary datasets

* Revert "Speed up dataset unit tests"

This reverts commit 861b52d.

* address suggestions from Pieter
* fix warnings, make sphinx fail on warnings

* fix a few links

* fix a bunch of links

* fix more links

* fix all remaining links

* and finally add the link checker

* debug workflow

* more debug

* undo debug

* Add to changelog

* fix new warning

* clean up more errors

* Fix link after rebase

* Apply suggestions from code review

Co-authored-by: PGijsbers <[email protected]>

Co-authored-by: PGijsbers <[email protected]>
* Convert sparse labels to pandas series

* Handling sparse labels as Series

* Handling sparse targets when dataset as arrays

* Revamping sparse dataset tests

* Removing redundant unit test

* Cleaning target column formatting

* Minor comment edit
* Adding warnings to all examples switching to a test server

* Creating warnings in new text cells

* Fixing a bug

* Debugging doc build failures

* Update openml/config.py

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

* Fixing GUI commit bug

* Using a common warning message for docs

* Renaming warning message file

* Editing the non-edited file

Co-authored-by: Matthias Feurer <[email protected]>
OpenML code 500 indicates no results for a flow query, and was likely
confused with the HTTP code 500 for internal server error.
…nment (#1075)

* Initial structure

* Add doc and test functionality for mounted repo

* Add branch support and safeguards

* Update docker usage and name, add structure

* Improved formatting

* Add reference to docker image from main docs

* Add Workflow to build and push docker image

* Use environment variable directly

* Try other formatting for SHA tag

* Try format as string

* Only push latest

* Explicitly make context relative

* Checkout repository

* Install wheel and setuptools before other packages

* Rename master to main

* Add information about Docker PR

* Make 'note' italtics instead of content

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

Co-authored-by: Matthias Feurer <[email protected]>
Create a second configurable retry policy. The configuration now allows for a `human` and `robot` retry policy, intended for interactive use and scripts, respectively.
Because the repository at the time of building the docker image is not
aware of branches that are created afterwards, which means otherwise
those are only accessible through the openml#branch syntax.
* doc update

* fixes
* rename master to main

* update changelog

* fix documentation building script

* rename master to main in all remaining docs

* drop badges

Co-authored-by: PGijsbers <[email protected]>
* started working on additional information for extension

* extended documentation

* final pass over extensions

* Update doc/extensions.rst

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

* Update doc/extensions.rst

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

* changes suggested by MF

* Update doc/extensions.rst

Co-authored-by: PGijsbers <[email protected]>

* Update doc/extensions.rst

Co-authored-by: PGijsbers <[email protected]>

* Update doc/extensions.rst

Co-authored-by: PGijsbers <[email protected]>

* added info to optional method

* fix documentation building

* updated doc

Co-authored-by: Matthias Feurer <[email protected]>
Co-authored-by: PGijsbers <[email protected]>
The Parquet file is optional, and failing to reach it (and download it)
should not prevent the usage of the other cached/downloaded files.
* Allow tasks to be downloaded without dataqualities

Previously ``download_qualities`` would be left at the default of True
with no way to overwrite it.

* Deprecate the use of strings for identifying tasks
@mfeurer mfeurer requested a review from PGijsbers May 20, 2021 13:55
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.

Test fails (so far) seem to be caused by the server.

@mfeurer mfeurer merged commit d2ccfe9 into main May 20, 2021
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.

6 participants