-
Notifications
You must be signed in to change notification settings - Fork 28.6k
[SPARK-3443][MLLIB] update default values of tree: #2322
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
Conversation
1. maxMemoryInMB: 128 -> 256 2. maxBins: 100 -> 32 3. maxDepth: 4 -> 5 (in some example code)
Jenkins, test this please. |
@mengxr I could only find a few items missed (in comments): Look in DecisionTree for "(suggested value:" |
test this please |
QA tests have started for PR 2322 at commit
|
@mengxr LGTM |
QA tests have finished for PR 2322 at commit
|
QA tests have started for PR 2322 at commit
|
QA tests have finished for PR 2322 at commit
|
test this please |
QA tests have started for PR 2322 at commit
|
QA tests have finished for PR 2322 at commit
|
I've merged this into master. Thanks @jkbradley for review! |
Adjust the default values of decision tree, based on the memory requirement discussed in #2125 :
@jkbradley