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.

Sign up for GitHub

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

Jump to bottom

[CUDA] Add binary logloss metric for new CUDA version #5635

Merged
merged 7 commits into from
Dec 29, 2022
Merged

[CUDA] Add binary logloss metric for new CUDA version #5635

merged 7 commits into from
Dec 29, 2022

Conversation

shiyu1994
Copy link
Collaborator

@shiyu1994 shiyu1994 commented Dec 13, 2022 •
edited by jameslamb
Loading

Add binary logloss metric for new CUDA version.

Contributes to #5163.

@shiyu1994 shiyu1994 requested a review from guolinke as a code owner December 13, 2022 05:59
@jameslamb
Copy link
Collaborator

I added "Contributes to #5163" to the description. Please do that in the future, so people finding those issues from GitHub search can easily see the other issues and PRs related to it.

You can see those automatically-created links at #5163 for example.

image

jameslamb
jameslamb approved these changes Dec 29, 2022
View reviewed changes
Copy link
Collaborator

@jameslamb jameslamb left a comment

Choose a reason for hiding this comment

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

excellent, thank you! Glad to see we're making progress towards implementing all of these metrics in the cuda_exp version.

@jameslamb jameslamb merged commit 7353166 into master Dec 29, 2022
@jameslamb jameslamb deleted the cuda/metric-binary branch December 29, 2022 06:43
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity since it was closed.
To start a new related discussion, open a new issue at https://github.com/microsoft/LightGBM/issues
including a reference to this.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jameslamb jameslamb jameslamb approved these changes

@guolinke guolinke guolinke approved these changes

@jmoralez jmoralez Awaiting requested review from jmoralez jmoralez is a code owner

@StrikerRUS StrikerRUS Awaiting requested review from StrikerRUS StrikerRUS is a code owner

Assignees
No one assigned
Labels
feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@shiyu1994 @jameslamb @guolinke