-
-
Notifications
You must be signed in to change notification settings - Fork 148
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.
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
OpenMLSplit.__init__ not documented or tested #332
Comments
Hey @amueller! I want to work on this issue, can you explain what exactly i need to do. |
@amueller basically mentions two issues:
Therefore, it would be great if you add some documentation to the class, maybe @amueller can comment on issue number two in the meantime. Unfortunately, the documentation on OpenML.org is a bit sparse ATM, but you can find some improvements made by Andy, including a description of what a split and a task is, here: openml/openml.org#210 If you have any questions please do not hesitate to ask. |
Sorry if I wasn't clear on the second issue. I guess in general that would be fine as a pattern, but I just found it pretty confusing. I guess we can't currently create tasks from the Python interface, so there is no reason why anyone would create a split. If we were to create tasks in Python, I would imagine that somehow would involve creating a split? |
Given the this is necessary to create new tasks I can see your confusion and why we need documentation on how the split class can be instantiated. |
Any instantiation of
OpenMLSplit
is done via a private function in the code. That seems really weird. And the class is not really documented.The text was updated successfully, but these errors were encountered: