Skip to content

Test Fixes for v0.15.1 #1358

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
eddiebergman merged 9 commits into from
Oct 14, 2024
Merged

Test Fixes for v0.15.1 #1358

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
e42b45a
fix: make FakeObject be the correct standard and robustify usage
LennartPurucker Oct 14, 2024
0ec7dc7
Merge branch 'develop' of https://github.com/openml/openml-python int…
LennartPurucker Oct 14, 2024
3391725
fix: get none valued study object from server
LennartPurucker Oct 14, 2024
6a9e037
add: test for minio download failures
LennartPurucker Oct 14, 2024
0b1bcdd
fix: skip test for WSL as it is not supported
LennartPurucker Oct 14, 2024
eba2b2e
maint: rework if/else case workflow
LennartPurucker Oct 14, 2024
f6ec1bc
maint: ruff fix
LennartPurucker Oct 14, 2024
9b2f392
add/fix: log messages for no premission
LennartPurucker Oct 14, 2024
83c78eb
fix: make flow name unique and enable testing of avoiding duplicates
LennartPurucker Oct 14, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: get none valued study object from server
  • Loading branch information
@LennartPurucker
LennartPurucker committed Oct 14, 2024
commit 339172520271111b2d53264a1d045d27ca0de5e9
6 changes: 6 additions & 0 deletions openml/study/functions.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,12 @@ def _get_study(id_: int | str, entity_type: str) -> BaseStudy:
alias = result_dict.get("oml:alias", None)
main_entity_type = result_dict["oml:main_entity_type"]

# Parses edge cases where the server returns a string with a newline character for empty values.
none_value_indicator = "\n "
for key in result_dict:
if result_dict[key] == none_value_indicator:
result_dict[key] = None

if entity_type != main_entity_type:
raise ValueError(
f"Unexpected entity type '{main_entity_type}' reported by the server"
Expand Down