NREL / ThermalNetwork

BSD 3-Clause "New" or "Revised" License
1 stars 1 forks source link

Downselect geojson buildings to only include buildings from sys-params file #16

Closed vtnate closed 8 months ago

vtnate commented 9 months ago

Any background context you want to provide?

The uo-cli brought this issue to light. The scenario-file could have fewer buildings than in the geojson file, which in turn means the sys-param file has fewer buildings than the geojson file. This caused a problem for ThermalNetwork.

What does this PR accomplish?

Don't use buildings (by removing in code only, not in the file) that are not in the sys-param file.

How should this be manually tested?