-
-
Notifications
You must be signed in to change notification settings - Fork 147
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
More extensive unit tests for run to / from xml #465
Comments
I added two unit tests:
the first one is deactivated, as the python api is indeed not ready for this yet (although it's not a bad thing, it doesn't seem to miss any obvious fields) |
I just wanted to have a look at this issue because getting this right is crucial to the success of OpenML. However, I can't find your tests. Could you please post a link to the tests you added so that I can extend them? |
Great catch. They are actually in the PR that will be deleted.
We should merge these two functions to the development branch. |
Make sure that the tests ensure that functions _create_description_xml() and _create_run_from_xml() are each others exact complements.
e.g.,
The text was updated successfully, but these errors were encountered: