-
Notifications
You must be signed in to change notification settings - Fork 85
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
Improve handling of sub-hourly timesteps for front of meter stand alone battery #994
Comments
See also this discussion on SAM forum: https://sam.nrel.gov/forum/forum-general/4307 |
Digging into the SSC code a bit, the front of meter models key off of the number of steps under "grid curtailment." It sounds like we should be able to fix this in lk. |
The behavior of the PPA/merchant plant models is inconsistent for PV+Battery cases, so it's hard to determine whether this should be fixed in lk or ssc. For example: pv+battery merchant plant gracefully handles a 15 minute time series with hourly gen, but single owner throws an error. Determine whether the PPA models should be more flexible, and if not what the scope of any lk fix should be. Punting to the patch since this is patchable. |
Decision from SAM meeting 5/29: For batteries, these subhourly price series matter, and subhourly weather data is available. Enforce consistent timesteps as much as possible. |
Decision from SAM meeting 6/5: Add stuff to help for the patch. Want to add a simulation timestep variable for stand alone batteries. Add a new UI form with one input variable that sets this. Given that this is a new variable, punt to the release. |
Describe the bug
The stand alone battery models should work for sub-hourly runs (and do, behind the meter) but the front of meter models don't have a well defined key for what determines the timestep. Setting up 15 minute PPA and dispatch data throws an error.
To Reproduce
Steps to reproduce the behavior:
Attached SAM file is configured to reproduce the error:
standalone_battery_timestep_error.zip
Expected behavior
Stand alone battery should run with sub-hourly data in all configurations.
Screenshots
Desktop (please complete the following information):
Note that it is easy to work around this issue in PySAM/the SDK, since the length of gen can be specified by the user. The workaround in the GUI is to specify a 15-minute weather file in a PV-Battery case, and then set the DC losses to 100%.
The text was updated successfully, but these errors were encountered: