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

Request to add certain training parameters present in classic Lightgbm to Synapse ML #1514

Closed
alzio2607 opened this issue May 19, 2022 · 2 comments
Assignees

Comments

@alzio2607
Copy link

alzio2607 commented May 19, 2022

Is your feature request related to a problem? Please describe.
Some features present in the classic (non-spark) Lightgbm are not there in synapseML. These features are really needed for someone looking to switch from the classic way to Synapse . It'll be appreciated if the parameters are integrated since they help with more precise training too.

Describe the solution you'd like
Would like to request the addition of the following params
cat_l2
max_cat_threshold
cat_smooth
min_data_per_group
min_data_in_bin

Additional context
Found 2 open issues pertaining to similar kind of request. Mentioning them here too. Creating this new issue for latest update
#997
#789

AB#1799267

@alzio2607 alzio2607 changed the title Request to add certain training parameters present in classic Lightgbm Request to add certain training parameters present in classic Lightgbm to Synapse M May 19, 2022
@alzio2607 alzio2607 changed the title Request to add certain training parameters present in classic Lightgbm to Synapse M Request to add certain training parameters present in classic Lightgbm to Synapse ML May 19, 2022
@imatiach-msft
Copy link
Contributor

@alzio2607 I think this is somewhat resolved with PR:
#1444
which was merged recently, since

Add ability for LightBGM user to set custom args that aren't defined explicitly by SynapseML (as Vowpal Wabbit does now)

adding @svotaw
Ofcourse, we should add these parameters to the API directly as well, in addition to the custom args parameter.

@svotaw svotaw self-assigned this Jul 9, 2022
@svotaw
Copy link
Collaborator

svotaw commented Jul 9, 2022

You should be able to set those as well as any other manual parameter after the above changes.

@svotaw svotaw closed this as completed Jul 9, 2022
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

No branches or pull requests

3 participants