Skip to content

Remove battery option to use weather file temperature #632

@cpaulgilman

Description

@cpaulgilman

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.

Metadata

Metadata

Assignees

Labels

Type

No type
No fields configured for issues without a type.

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions