Skip to content

Conversation

@Neeratyoy
Copy link
Contributor

Reference Issue

Addresses #75.

What does this PR implement/fix? Explain your changes.

Adds a repr for each of the top-level classes in OpenML.

How should this PR be tested?

In Ipython:

  • d = openml.datasets.get_dataset(523); d
  • f = openml.flows.get_flow(3740); f
  • r = openml.runs.get_run(10000); r
  • t = openml.tasks.get_task(31); t
  • s = openml.study.get_study(16); s
  • b = openml.study.get_suite(218); b
  • s = openml.setups.get_setup(10); s
    • s.parameters[190]
  • e = openml.evaluations.list_evaluations(function='predictive_accuracy', task=[31], size=10)
    • e[51]

@Neeratyoy Neeratyoy requested review from amueller and mfeurer July 17, 2019 17:53
@Neeratyoy
Copy link
Contributor Author

Looking to address the failing clustering_task unit test in #735.

@amueller
Copy link
Contributor

I think the easier / nicer thing would be to just not implement __str__ and just implement __repr__. So basically just do a search and replace from __str__ to __repr__.

@mfeurer
Copy link
Collaborator

mfeurer commented Jul 24, 2019

Superseeded by #743.

@mfeurer mfeurer closed this Jul 24, 2019
@mfeurer mfeurer deleted the add_repr branch November 12, 2019 10:09
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.

4 participants