Skip to content

[R-package] minor improvements on weight demo#5212

Merged
jameslamb merged 1 commit intomicrosoft:masterfrom
jmoralez:weight-demo
May 17, 2022
Merged

[R-package] minor improvements on weight demo#5212
jameslamb merged 1 commit intomicrosoft:masterfrom
jmoralez:weight-demo

Conversation

@jmoralez
Copy link
Collaborator

Removes some extra lines that aren't necessary after #5204 and makes minor modifications to the weight descriptions.

@StrikerRUS
Copy link
Collaborator

While we're on it, maybe transform this demo into a vignette as a part of #1944?

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.

Thanks for doing this! I agree with @StrikerRUS 's suggestion to consider converting this demo to a vignette, but I also think we should just merge this and that that can be a separate PR if you have time / interest.

@jmoralez
Copy link
Collaborator Author

Yeah I have #1944 on my todo list, mainly to get more familiar with the full R-package. But I'll probably pick that up later, I'm currently focusing on the items of #5153.

@jameslamb
Copy link
Collaborator

Thank you! Totally agree with that approach @jmoralez

@jameslamb jameslamb merged commit 53218c1 into microsoft:master May 17, 2022
@jmoralez jmoralez deleted the weight-demo branch May 17, 2022 02:27
@jameslamb jameslamb mentioned this pull request Oct 7, 2022
40 tasks
@github-actions
Copy link
Contributor

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 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants