Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Rename arguments of list_evaluations to indicate they expect lists of ids #804

Copy link
Copy link
@PGijsbers

Description

@PGijsbers
Issue body actions

See also #796. It was a step forward, but the argument names still don't convey that we are looking for a list (i.e. argument name should be plural) or ids as opposed to OpenML objects (e.g. by a suffix _id).

I would argue that we also should update the type hints (to Optional[List[Union[str, int]]]) and add parameter descriptions in the docstring.

Reactions are currently unavailable

Metadata

Metadata

Assignees

Labels

DocumentationGood First IssueIssues suitable for people new to contributing to openml-python!Issues suitable for people new to contributing to openml-python!

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions

    Morty Proxy This is a proxified and sanitized view of the page, visit original site.