You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This is in relation to this tolerance bug komiya-atsushi/xgboost-predictor-java#21 that affects correctness of results on complex models. We have since fixed it in our fork at Yelp and achieved complete parity of results with xgboost4j.
We're now wondering if you would be open to PRs so that we can contribute this upstream. Just asking before we do the work and open the PR :)
The text was updated successfully, but these errors were encountered:
I've been working on getting xgboost-predictor-java integrated into the MLeap project (here's the PR: combust/mleap#645).
It'd be great to apply the Yelp's tolerance bugfix for issue #21 (parity lost on complex models) to your project. Our alternative is to adopt the Yelp's fork but I'd like to avoid duplicating work and contribute the fix upstream.
Hi @komiya-atsushi,
This is in relation to this tolerance bug komiya-atsushi/xgboost-predictor-java#21 that affects correctness of results on complex models. We have since fixed it in our fork at Yelp and achieved complete parity of results with xgboost4j.
We're now wondering if you would be open to PRs so that we can contribute this upstream. Just asking before we do the work and open the PR :)
The text was updated successfully, but these errors were encountered: