You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
The text was updated successfully, but these errors were encountered:
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.The text was updated successfully, but these errors were encountered: