Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update XGBoost wrapper #14

Open
arogozhnikov opened this issue Jul 17, 2015 · 0 comments
Open

Update XGBoost wrapper #14

arogozhnikov opened this issue Jul 17, 2015 · 0 comments
Labels
question
Milestone
0.6.8

Comments

@arogozhnikov
Copy link
Contributor

XGBoost wrapper was significantly improved.

New wrapper is almost sklearn-compatible, but there are still failures of copy and clone, important field classes_ is not set.

So there are two ways:

  1. We leave things as they are, since our wrapper works fine.
  2. or I fix these bugs in original wrapper and we delete special wrapper for XGBoost. Less code to support, but no guarantee about proper support of that wrapper.
@arogozhnikov arogozhnikov added this to the 0.6.3 milestone Jul 17, 2015
@arogozhnikov arogozhnikov modified the milestones: 0.6.5, 0.6.4 Nov 21, 2015
@tlikhomanenko tlikhomanenko modified the milestones: 0.6.8, 0.6.6 Nov 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
question
Projects
None yet
Milestone
0.6.8
Development

No branches or pull requests
2 participants
@arogozhnikov @tlikhomanenko