-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Allow TPOT to run for the shortest (or longest) between generations and max_time_mins #504
Comments
I am not sure what the best way to do this is. If either only When both When both Alternatively don't rename |
Thank you for your suggestion. I agree with the scenario when |
You're indeed right that the I think it's technically possible to achieve your goal by setting If we can figure out a good way for these parameters to interact, I'm not opposed to tweaking it slightly. But I'd very much prefer to avoid adding another parameter. Perhaps what we can do is have |
And for user specified number of generations, combined with But you are probably right in that specifying by generations is probably not that important in practice (compared to specifying by time). |
In my proposed solution, if the user specifies |
Okay, I understood that wrong then. |
That seems like a good idea to support both use cases. |
When I set-up an experiment, I often find myself wanting to execute a set number of generations, but cut off if it exceeds X minutes.
For example, I want to run TPOT for 100 generations, but only use at most 60 minutes of time.
I can also imagine someone would want to do the exact opposite; run TPOT for at least 100 generations, but also run for at least X minutes of time.
As is, you can only pick between a set maximum of generations or a certain time.
When both are provided, the set amount of generations actually does nothing, and TPOT is run for
max_time_mins
(this is explicit in the code).Besides the lack of above options, it also bothers me that
max_time_mins
does not so much specify a maximum time, but a set time for which the experiment will run (and, practically speaking, never shorter).The changes are very minor in code (I did this for my own versions), but I wanted to discuss if this is a good addition, and how this should change the constructor parameters.
The text was updated successfully, but these errors were encountered: