urbanopt / geojson-modelica-translator / 11042210834

88%

Build Type

push

github

Committed by web-flow

Commit Message

Support multiple GHEs in a single district (#644)

* update borefield.py to accept a ghe_id when instantiated

* add debug logging to get_param_by_id method

* pass the list of ghes to borefield when instantiating in single_ghe test

* WIP new test for multi-ghe

* improve error message for missing g-function file

* additional g-function files to match the test sys-param files

* update borefield test to work in the multi-ghe paradigm

* improve help text in CLI

* more explicitly process different types of district sys-params

* clean up generation call, load loop order, load multiple ghes

* use new parameter for district_type argument

* load ghe loop order in the correct place

* move soil parameters out of ghe-specific and into all 5G parameters

* use new location of soil parameters

* don't require soil in the incorrect place, duh

* use new fifth_generation tag properly in 5G tests

* fix test sys-param files for 5G districts

* add graceful error message when sizing output from TN is not found

* remove outdated TODO message

* remove unused test sys-param file

* comment noisy debug log lines

* update geojson and sys_param paths

* add project folder for testing multi-ghe

* update building load profiles

* modify paths pointing to mos files and ghe_dir

* add ghe sizing results to ghe_dir

* move soil and add horizontal piping parameters

* pre-commit formatting

* read the loop order file for later use

* py file for new distribution template

* new distribution template file

* new distribution template instance file

* correct name

* correct name

* take out dis and temp sensors

* move dis and sensor connections elsewhere

* coupling no longer needed

* new distribution couplings

* read loop_order in district.py for use in couplings

* remove accidental `endraw` tag in jinja template

* new placeholder coupling templates

* WIP: first steps in reading ... (continued)

Run Details

1010 of 1226 branches covered (82.38%)

Branch coverage included in aggregate %.

140 of 152 new or added lines in 10 files covered. (92.11%)

3 existing lines in 1 file now uncovered.

2778 of 3061 relevant lines covered (90.75%)

1.82 hits per line