-
Notifications
You must be signed in to change notification settings - Fork 70
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
Develop a function to update information #112
Comments
Currently the design is that a
I'll leave this open for comment, then migrate to new issues from 1, 2, and 3 as the discussion dictates. |
I am for option 2 and then have more time series structs where we would have a device field that we can use to point to the device for which this is relevant. |
@jd-lara is that to say that you are in support of the proposal that includes all of 1, 2, & 3? Because I think that 2 isn't strictly necessary, but would be a good idea to improve clarity. |
@claytonpbarrows yes, implement 1 but not only for forecasts also a new struct for just time series. |
@jd-lara, in some sense everything that we have enabled in PowerSimulations is optimizing schedules against a 'forecast', so until we do something different, I don't see the need for a new struct. |
This will be resolved with the implementation of the time-series update in PowerSimulations. |
I am closing this since now all the structs are mutable. |
No description provided.
The text was updated successfully, but these errors were encountered: