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

added alignment for zero-based index for interaction_constraints #167

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

kainkad
Copy link
Contributor

@kainkad kainkad commented Nov 29, 2024

  • added interaction_constraints to the INDEXPARAM
  • updated type for interaction_constraints to accept now Vector{Vector{Int}} instead of String (default for CLI but supported as a list of list in other languages)
  • aligned julia's one-based indexing to the zero-based indexing similar to categorical_feature

@@ -1,5 +1,5 @@

const INDEXPARAMS = [:categorical_feature]
const INDEXPARAMS = [:categorical_feature, :interaction_constraints]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Extended this to interaction_constraints param which requires similar zero-based indexing handling to categorical_feature.
There are some dataset params such as label_column, weight_column, or group_column can take a single index indicating the column, or the column name and these are added in https://github.com/IQVIA-ML/LightGBM.jl/pull/164/files.

@kainkad kainkad merged commit b8e14dc into all-params Dec 2, 2024
21 checks passed
@kainkad kainkad deleted the zero-indexed-params branch December 2, 2024 15:56
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.

1 participant