-
-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
[jvm-packages] add format option when saving a model #7940
Conversation
@trivialfis please help to review it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the work on the file format. Could you please update the document: https://github.com/dmlc/xgboost/blob/master/doc/tutorials/saving_model.rst ?
Yeah, I will add it in the followup PR. Thx |
private[spark] abstract class XGBoostWriter extends MLWriter { | ||
|
||
/** Currently it's using the "deprecated" format as | ||
* default, which will be changed into `ubj` in future releases. */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changed into json
in future releases.
This PR adds a format option which allows the booster to be dumped into json or ubj.
For now, we still use "deprecated" format as the default when dumping the booster model, but we're able to choose the different booster formats like json, ubj to be dumped.
eg,
or