Uh oh!
There was an error while loading. Please reload this page.
Conversation
mfeurer
commented
May 28, 2019
(tests pass on appveyor, presumably due to a different scipy version used for Windows) |
Codecov Report
@@ Coverage Diff @@## develop #698 +/- ##
===========================================
- Coverage 90.35% 90.33% -0.03%
===========================================
Files 36 36 Lines 3785 3787 +2 ===========================================
+ Hits 3420 3421 +1 - Misses 365 366 +1
Continue to review full report at Codecov.
|
PGijsbers
commented
May 28, 2019
What's the reason for not merging in the latest version of dev (with mentioned fix)? |
mfeurer
commented
May 28, 2019
Actually none, except that I'm not a fan of merging. The PR checks are green now, which should give us enough confidence that this PR does what it's supposed to do. |
| openml.config.server = self.production_server | ||
| flow_id = 6794 | ||
| setups = openml.setups.list_setups(flow=flow_id, output_format='object', size=10) | ||
| print(setups) |
There was a problem hiding this comment.
Why is the print statement here?
This PR fixes two bugs which were 'introduced' recently by data changes on the test server. This does not include the fix included in #693 and will therefore not fix all unit tests.