Describe the bug
See https://sam.nrel.gov/forum/forum-general/3759 for a description, example file, and workaround solution.
To Reproduce
-
Create a default Standalone Battery / Commercial case.
-
Go to the Electric Load page, click Edit array, Change time step and type 15.
-
Run a simulation and note the simulation message.
-
On the Battery Cell and System page, under Battery Thermal, change the environment temp option to Enter single fixed temperature.
-
Run a simulation and note simulation message.
Expected behavior
A. The default environment temp option for all standalone battery configurations should probably be single fixed temperature.
B. SSC should only use the time series environment temp data when the environment temp option is set to enter time series data, and only generate that error based on the option. The UI always creates an array of battery environment temperatures. For ‘Enter single fixed temperature’, the array is sized then each index is written over with the fixed value. There is a bug where the length of that array is hard coded to 8760 for Standalone battery. Should be fixed to dynamically allocate based on either load or curtailment.
C. The error message should say "Environment temperature input length must equal number of weather file and/or electric load data records."
Operating System and Version
SAM 2021.12.02
Describe the bug
See https://sam.nrel.gov/forum/forum-general/3759 for a description, example file, and workaround solution.
To Reproduce
Create a default Standalone Battery / Commercial case.
Go to the Electric Load page, click Edit array, Change time step and type 15.
Run a simulation and note the simulation message.
On the Battery Cell and System page, under Battery Thermal, change the environment temp option to Enter single fixed temperature.
Run a simulation and note simulation message.
Expected behavior
A. The default environment temp option for all standalone battery configurations should probably be single fixed temperature.
B. SSC should only use the time series environment temp data when the environment temp option is set to enter time series data, and only generate that error based on the option. The UI always creates an array of battery environment temperatures. For ‘Enter single fixed temperature’, the array is sized then each index is written over with the fixed value. There is a bug where the length of that array is hard coded to 8760 for Standalone battery. Should be fixed to dynamically allocate based on either load or curtailment.
C. The error message should say "Environment temperature input length must equal number of weather file and/or electric load data records."
Operating System and Version
SAM 2021.12.02