-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add generic interface tests from MLJTestIntegration #22
Conversation
Codecov Report
@@ Coverage Diff @@
## master #22 +/- ##
==========================================
+ Coverage 89.74% 93.58% +3.84%
==========================================
Files 1 1
Lines 156 156
==========================================
+ Hits 140 146 +6
+ Misses 16 10 -6
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Thanks, I'll add these in my PR because that'll just be easier. |
Marking as draft as #21 to incorporate. |
Yeah, sorry, I was distracted by my XGBoost.jl PR. I will return my attention to this when that is done. |
I'm very stumped as to what's going on here... it seems to be complaining that it is getting the wrong So if you don't mind I think I'd prefer to see if my PR to XGBoost.jl has a chance of being merged sometime soon before picking up the pieces of whatever happened here. |
Yes, as I recall, the existing XBoost.jl flattens the output of probability arrays and special cases binary probabilities (only one instead of two, per observation). This required some logic to sort out, and it is quite possible your refactor has broken this. |
Close as added in another merged PR |
No description provided.