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

Add rank:ndcg and rank:map as valid objectives #479

Merged
merged 1 commit into from
Dec 13, 2023

Conversation

styrmis
Copy link
Contributor

@styrmis styrmis commented Dec 8, 2023

When attempting to use rank:ndcg as an objective model deployment failed unexpectedly, stating that the objective was invalid.

As per the XGBoost docs rank:ndcg and rank:map are valid objectives, but are currently rejected by the parser.

This change resolves this by extending the list of objectives which map to the no-op normalizer, allowing the models to be parsed and used by the plugin.

As per the XGBoost docs [1] `rank:ndcg` and `rank:map` are valid
objectives, but are currently rejected by the parser.

This commits resolves this by extending the list of objectives which map
to the no-op normalizer, allowing the models to be parsed and used by
the plugin.

[1] https://github.com/dmlc/xgboost/blob/1094d6015df7be6f5829e53a4c257f3481b43ee5/doc/parameter.rst#parameters-for-learning-to-rank-rank-ndcg-rank-map-rank-pairwise
@styrmis styrmis marked this pull request as ready for review December 8, 2023 11:21
@wrigleyDan wrigleyDan merged commit b03f570 into o19s:main Dec 13, 2023
1 check passed
@wrigleyDan
Copy link
Contributor

LGTM, merging with main.

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.

2 participants