fix: change building names in ep_multiple_buildings.py to upper case #66

Merged
g_gutierrez merged 1 commits from fix/ep-result-factory-naming-bug into main 2024-10-01 10:15:41 -04:00
Member

This branch fixes a naming issue in the EnergyPlus workflow result factory. When EnergyPlus runs a simulation, and the building name contains letters (i.e., it's not purely numeric), it converts the building name to uppercase. This causes the current search process to fail because it doesn't account for the uppercase conversion. I have modified the search process to accommodate uppercase building names.

This branch fixes a naming issue in the EnergyPlus workflow result factory. When EnergyPlus runs a simulation, and the building name contains letters (i.e., it's not purely numeric), it converts the building name to uppercase. This causes the current search process to fail because it doesn't account for the uppercase conversion. I have modified the search process to accommodate uppercase building names.
a_rezaei self-assigned this 2024-10-01 10:12:13 -04:00
g_gutierrez was assigned by a_rezaei 2024-10-01 10:12:13 -04:00
a_rezaei added 1 commit 2024-10-01 10:12:14 -04:00
a_rezaei requested review from s_ranjbar 2024-10-01 10:12:49 -04:00
g_gutierrez merged commit 70dd9f7c6a into main 2024-10-01 10:15:41 -04:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: CERC/hub#66
No description provided.