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

[SW-1768] Get Default Values of AutoML Parameters Synchronized with H2O-3 #1675

Merged
merged 2 commits into from
Dec 6, 2019

Conversation

mn-mikke
Copy link
Collaborator

@mn-mikke mn-mikke commented Dec 6, 2019

I was also thinking about referencing H2O-3 default values directly to have them specified only once. But since they must be specified also in Python API, I just changed values on the SW site.

@mn-mikke mn-mikke changed the title [SW-1768] Get Default Values of AutoML Parameters synchronized with H2O-3 [SW-1768] Get Default Values of AutoML Parameters Synchronized with H2O-3 Dec 6, 2019
Copy link
Contributor

@jakubhava jakubhava left a comment

Choose a reason for hiding this comment

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

Thank you @mn-mikke. Just question -> what does it mean for the user that maxRuntimeSecs has changed to 0 by default? Does it mean take as much time as you need?

@mn-mikke
Copy link
Collaborator Author

mn-mikke commented Dec 6, 2019

@jakubhava According what I remember from the discussion with @seb-h2o, it's exactly as you described. maxModels -> 0 also means an unlimited number of models. I will add note to a migration guide

@seb-h2o please correct me if I'm wrong.

@jakubhava
Copy link
Contributor

Thanks for explanation!

@sebhrusen
Copy link

to clarify:

  • maxRuntimeSecs = 0 means unlimited time.
  • maxModels = 0 means unlimited amount of models.
  • if both are set to 0, backend will automatically change the default for maxRuntimeSecs to 1h.
  • if both are set to value != 0, both are applied and the first limit reached will prevail.

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.

None yet

3 participants