Problem
For the battery environmental temperature option, the "Use weather file ambient temperature" option is unrealistic because batteries are typically installed in a temperature-controlled container or room, so the battery is not exposed to ambient temperatures. Having that option is potentially misleading for new users.
If someone really wanted to use the ambient temperature as the battery environment temperature, they could use the "Enter time series temperature" option.
Resolution
Remove "User weather file ambient temperature" option and add code in update script to handle importing old files with that option enabled: Warning message if weather file ambient temperature option is selected, and change value of batt_room_temperature_vector as needed for other options.
This will potentially break LK scripts and other code created in older versions because batt_room_temperature_vector = 0 is the weather file temperature option.
Problem
For the battery environmental temperature option, the "Use weather file ambient temperature" option is unrealistic because batteries are typically installed in a temperature-controlled container or room, so the battery is not exposed to ambient temperatures. Having that option is potentially misleading for new users.
If someone really wanted to use the ambient temperature as the battery environment temperature, they could use the "Enter time series temperature" option.
Resolution
Remove "User weather file ambient temperature" option and add code in update script to handle importing old files with that option enabled: Warning message if weather file ambient temperature option is selected, and change value of
batt_room_temperature_vectoras needed for other options.This will potentially break LK scripts and other code created in older versions because
batt_room_temperature_vector = 0is the weather file temperature option.