-
Notifications
You must be signed in to change notification settings - Fork 68
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
Start up costs & min/max runtimes #16
Conversation
The review goes along with PR https://github.com/oemof/oemof/pull/446. |
Just to remain in the workflow.. |
Thanks for your contribution! Its a nice feature. I tested the examples and they look fine. Only three remarks/questions:
|
Yep. Thanks!
Yes, it is the latter.
The initial status is also set in the end as explained in the docstring:
Should we adapt the naming or is it o.k. like this!?
I just made that up quickly. Do you have suggestions!? Go ahead ;-) |
Thanks for your suggestions and review! |
I have now merged the functionality into the dev. @jnnr @uvchik : Feel free to change as you like and merge afterwards if you are fine! |
It is just an example and if it works and looks okay we can merge it. |
If @jnnr has naming improvements, he can change it directly! I am fine with it anyway.. |
This provides examples on how to use start up costs and min/max runtimes.