From 408dc9c4aed9e643022a0f465e9d32cb9e63b23b Mon Sep 17 00:00:00 2001 From: PGijsbers Date: Mon, 27 Jun 2022 14:51:21 +0200 Subject: [PATCH 1/2] Use GET instead of POST for flow exist --- openml/flows/functions.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/openml/flows/functions.py b/openml/flows/functions.py index 048fa92a4..28d49b691 100644 --- a/openml/flows/functions.py +++ b/openml/flows/functions.py @@ -253,7 +253,7 @@ def flow_exists(name: str, external_version: str) -> Union[int, bool]: raise ValueError("Argument 'version' should be a non-empty string") xml_response = openml._api_calls._perform_api_call( - "flow/exists", "post", data={"name": name, "external_version": external_version}, + "flow/exists", "get", data={"name": name, "external_version": external_version}, ) result_dict = xmltodict.parse(xml_response) From bebb1e5f0dd693db49f0b2c796f17a645df0e19e Mon Sep 17 00:00:00 2001 From: PGijsbers Date: Mon, 27 Jun 2022 14:55:11 +0200 Subject: [PATCH 2/2] Add flow_exist fix notice --- doc/progress.rst | 1 + 1 file changed, 1 insertion(+) diff --git a/doc/progress.rst b/doc/progress.rst index 286666767..02dd78086 100644 --- a/doc/progress.rst +++ b/doc/progress.rst @@ -10,6 +10,7 @@ Changelog ~~~~~~ * FIX#1030: ``pre-commit`` hooks now no longer should issue a warning. * FIX#1110: Make arguments to ``create_study`` and ``create_suite`` that are defined as optional by the OpenML XSD actually optional. + * FIX#1147: ``openml.flow.flow_exists`` no longer requires an API key. * MAIN#1088: Do CI for Windows on Github Actions instead of Appveyor. * ADD#1103: Add a ``predictions`` property to OpenMLRun for easy accessibility of prediction data.