urbanopt / geojson-modelica-translator / 6342965393

88%

push

github-actions

Add a quick fix for reading gfunction.csv from ghe_id subfolder (#589)

* add ghe to create system parameter function

* Add design method for borehole

* update create sys param and system_parameter.py

* update tests

* update district system type property

* reduce unnecessary indenting

* add negative sign to heat_flow to designate as cooling

* clean up test setup now that we have at least python 3.8

* update district_system_type property in system_parameters.py

* redo the un-indenting I accidentally committed

* update dependencies with poetry

* remove generated test sys-param file

* gitignore generated test sys-param file

* add sys param argument

* fix typo that prevented sys-param creation via cli

* remove nonsensical 5G district parameters

* clean up uo_des test comments a bit, and add intermediate assertions

* remove outdated spawn compilation instructions

* enable 5G models with the CLI

* add borehole length to template

* add gfunction csv file to cli test for ghe district tests

* more updates to gmt class to handle 5g districts from cli

* choose generations more explicitly in district.py

* use correct sys-param parameter name in borefield.py

* add start/stop/step times for model simulation to cli options

* add 5g cli integration test, update calls to simulate during the summer

* Quick fix to enlarge loop flow rate

* Fix for reading gfunction from ghe_id folder

* Resolve conflict

* Discard unwanted changes

* Fix broken test_uo_des.py

* Point to a different ghe_dir

---------

Co-authored-by: Tanushree <tanushree.charan@nrel.gov>
Co-authored-by: Nathan Moore <nathan.moore@nrel.gov>

1003 of 1210 branches covered (0.0%)

Branch coverage included in aggregate %.

2 of 2 new or added lines in 1 file covered. (100.0%)

2732 of 3067 relevant lines covered (89.08%)

1.78 hits per line