Skip to content

Conversation

wanhanlong1130
Copy link
Collaborator

Just for review purpose.
All changes are in applications/ashrae205

@wanhanlong1130 wanhanlong1130 self-assigned this Sep 16, 2025
Copy link
Collaborator

@lymereJ lymereJ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@wanhanlong1130 - Thanks! Could you please convert the ipynb to py and integrate the whole process into one script so everything can be done at once. Also, could you please add the maps created from the curves?

wanhanlong1130 and others added 18 commits September 22, 2025 14:16
* fixing interpolation bug with updated library

* Add iteration loop.

* updated library.json with some of York curves removed

* removing multi_stage.json

* remove new_lib.py not needed

* unitarydirectexpansion_curves.json with updated library

* deleted library.json and multi_stage.json

* added speed attribute to each curve modifier

* =Add unit test for MS unitary DX coil generation.

---------

Co-authored-by: Lerond <[email protected]>
Co-authored-by: lymereJ <[email protected]>
* fixing interpolation bug with updated library

* Add iteration loop.

* updated library.json with some of York curves removed

* removing multi_stage.json

* remove new_lib.py not needed

* unitarydirectexpansion_curves.json with updated library

* deleted library.json and multi_stage.json

* added speed attribute to each curve modifier

* =Add unit test for MS unitary DX coil generation.

* Fix return for multi-speed fan configurations.

---------

Co-authored-by: Rahman, Aowabin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants