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

Move number of split quantiles from hardwired to node class member #54

Merged
merged 1 commit into from
May 14, 2024

Conversation

EgorKraevTransferwise
Copy link
Collaborator

Context

Just wanted to make this change before I forget about it, related to cerlymarco/linear-tree#41

Checklist

@EgorKraevTransferwise
Copy link
Collaborator Author

Merging can wait until we have some actually useful changes to deploy as well :)

@wise-github-bot-app
Copy link

🔴

Your CODEOWNERS file is broken, fix the following errors before merging the branch:

  • The CODEOWNERS file does not exist. Repository ownership will default to ENGINEERING until you add a CODEOWNERS file with a * rule that assigns it to a team

@AlxdrPolyakov AlxdrPolyakov merged commit 5cdd5a5 into main May 14, 2024
8 of 9 checks passed
@AlxdrPolyakov AlxdrPolyakov deleted the num_quantiles branch September 5, 2024 12:21
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.

2 participants