Add debug output #860
Add debug output #860
Conversation
| def _check_run(self, run): | ||
| self.assertIsInstance(run, dict) | ||
| self.assertEqual(len(run), 7) | ||
| assert len(run) == 7, str(run) |
There was a problem hiding this comment.
can you please explain in comments why 7? is not clear from the (direct) context
There was a problem hiding this comment.
That's the amount of entries per run returned by the API: https://openml.org/api/v1/xml/run/list/flow/1154
There was a problem hiding this comment.
I just pushed a commit explaining this. I will merge once the tests are through.
There was a problem hiding this comment.
good. I am not sure if we should really check this, as this is flexible for extension. But in those cases we can adjust the unit tests as well
Codecov Report
@@ Coverage Diff @@
## develop #860 +/- ##
===========================================
+ Coverage 88.36% 88.43% +0.07%
===========================================
Files 37 37
Lines 4272 4272
===========================================
+ Hits 3775 3778 +3
+ Misses 497 494 -3
Continue to review full report at Codecov.
|
to find the cause of previously failing unit tests.