NREL / ThermalNetwork

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

Additional output file for GMT to build multi-ghe Modelica models #30

Closed vtnate closed 2 months ago

vtnate commented 5 months ago

Any background context you want to provide?

Multi-ghe districts have additional information that is needed to build a Modelica model appropriately

What does this PR accomplish?

Generates additional json file with loop order information for the GMT & REopt to consume

How should this be manually tested?

mitchute commented 3 months ago

What is the reason for removing the files in the tests? Furthermore, shouldn't we be testing something related to make sure they are accurate?

vtnate commented 3 months ago

What is the reason for removing the files in the tests? Furthermore, shouldn't we be testing something related to make sure they are accurate?

No real reason to delete the test files. I'm a big fan of testing, I'll think about that a bit more. I haven't yet come up with a way to test it that isn't identical to what the real code is doing. Thank you!

mitchute commented 3 months ago

I don't think we need to complicate it, but it would be good to assert some simple tests against these files. If needed, there's the tempfile library that provides a platform-agnostic way to create temporary directories where we can write files, test, and then forget them.