urbanopt / geojson-modelica-translator / 11131693904 / 1

88%

develop: 88%

push

github

Simplify sys params to closer match reality (#659)

* 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)

952 of 1171 branches covered (81.3%)

Branch coverage included in aggregate %.

2842 of 3135 relevant lines covered (90.65%)

0.91 hits per line