dmlc / XGBoost.jl

XGBoost Julia Package
Other
288 stars 110 forks source link

Sorting of importance #109

Closed markub3327 closed 1 year ago

markub3327 commented 2 years ago

Hello @aviks,

the users can be able to set the sorting of importance by gain, cover, freq or name. This is passed as an argument of importance() method.

Thanks.

ablaom commented 2 years ago

@OkonSamuel I know you are working on better exposure for feature importances in MLJXGBoostInterface. You may want to watch this PR.

ExpandingMan commented 1 year ago

Thanks. This is no longer needed as of #111 which sorts by default. If you feel it's not flexible enough (right now there are no options for how or whether to sort, though there are separate calls for different metrics) please feel free to open an issue or make another PR.