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

[feature request] faster lambdarank #2701

Closed
guolinke opened this issue Jan 20, 2020 · 4 comments
Closed

[feature request] faster lambdarank #2701

guolinke opened this issue Jan 20, 2020 · 4 comments

Comments

@guolinke
Copy link
Collaborator

Summary

An efficient lambadrank implementation.

Motivation

As #2699 reduce the time cost on ConstructHistogram, the time cost of Boosting (get gradients and hessians) becomes relatively larger, especially for lambdarank. For example, the time cost of lambdarank objective is about 60 seconds in MSLTR, while the total time is about 180 seconds.

Description

Code is at https://github.com/microsoft/LightGBM/blob/master/src/objective/rank_objective.hpp

There is an O(n^2) loop inside for each query, to calculate the losses over all pairs.
we may need a good way to approximate the loss.

@StrikerRUS
Copy link
Collaborator

Closed in favor of being in #2302. We decided to keep all feature requests in one place.

Welcome to contribute this feature! Please re-open this issue (or post a comment if you are not a topic starter) if you are actively working on implementing this feature.

@guolinke
Copy link
Collaborator Author

refer to #2801 for faster rank_xendgc.

@guolinke
Copy link
Collaborator Author

I think #3425 already largely improves the speed of lambadrank.

@StrikerRUS
Copy link
Collaborator

@guolinke

I think #3425 already largely improves the speed of lambadrank.

Great! Then I'm removing this issue from top priorities section of the roadmap.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants