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.

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

Enable loading model from <1.0.0 trained with objective='binary:logitraw' #6517

Merged
merged 3 commits into from
Dec 17, 2020

Conversation

hcho3
Copy link
Collaborator

@hcho3 hcho3 commented Dec 16, 2020

Closes #6509

  • Override ProbToMargin() for the binary:logitraw objective, since no transformation is necessary.
  • Augment model compatibility suite with binary:logitraw. I also updated the files xgboost_model_compatibility_test.zip and xgboost_r_model_compatibility_test.zip.

@hcho3 hcho3 mentioned this pull request Dec 16, 2020
5 tasks
Copy link
Member

@trivialfis trivialfis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM if all tests are green.

@hcho3 hcho3 merged commit ad1a527 into dmlc:master Dec 17, 2020
@hcho3 hcho3 deleted the fix_loading_old_model_logitraw branch December 17, 2020 00:53
trivialfis added a commit to trivialfis/xgboost that referenced this pull request Dec 17, 2020
…raw' (dmlc#6517)

* Enable loading model from <1.0.0 trained with objective='binary:logitraw'

* Add binary:logitraw in model compatibility testing suite

* Feedback from @trivialfis: Override ProbToMargin() for LogisticRaw

Co-authored-by: Jiaming Yuan <jm.yuan@outlook.com>
trivialfis added a commit that referenced this pull request Dec 17, 2020
…raw' (#6517) (#6524)

* Enable loading model from <1.0.0 trained with objective='binary:logitraw'

* Add binary:logitraw in model compatibility testing suite

* Feedback from @trivialfis: Override ProbToMargin() for LogisticRaw

Co-authored-by: Jiaming Yuan <jm.yuan@outlook.com>

Co-authored-by: Philip Hyunsu Cho <chohyu01@cs.washington.edu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

restoring xgboost 0.82.1 version models (in R) with xgboost 1.1.1.1 or 1.2.01 still failing ?
2 participants