Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Templates • HW plant #3364

Draft
wants to merge 714 commits into
base: master
Choose a base branch
from
Draft

Conversation

AntoineGautier
Copy link
Contributor

@AntoineGautier AntoineGautier commented Apr 28, 2023

This shall NOT be merged before #2700 is merged.
This will close #3266.
This is a DRAFT PR to test the new CI workflow for the Templates package (see Buildings/Resources/Scripts/travis/templates/checkandrun.sh) and start the review process.

TODO before merging:

  • Check that Issue2180 boiler plant main controller oct 2021 #2700 (review) are addressed, pull the changes once issue2180_BoilerPlant_MainController_oct_2021 is merged into the master, replace Templates.HeatingPlants.HotWater.Components.Controls.Guideline36Plugin with the actual controller. (The branch issue3266_template_HW_plant has no dependency to issue2180_BoilerPlant_MainController_oct_2021 and uses a "temporary" class that is plug compatible with the current version of the controller from commit 1a5959f.)
  • Restore all tests in Buildings/Resources/Scripts/travis/templates/BoilerPlant.py and .travis.yml.
  • Open issue to remove final in the binding of fue for records within Buildings.Fluid.Boilers.Data.Lochinvar

Note that the template models are currently not supported by Dymola due to a bug in handling arrays of record. Tickets #SR01004314-01 #SR01022035-01 and #SR00955379-01 are in status "Wait for Correction Delivery" or "R&D Handling" at DS.

@AntoineGautier AntoineGautier force-pushed the issue3266_template_HW_plant branch 4 times, most recently from 176cd68 to 1cea147 Compare May 18, 2023 15:45
@AntoineGautier AntoineGautier force-pushed the issue3266_template_HW_plant branch from 1cea147 to 49c0f67 Compare May 18, 2023 16:08
@AntoineGautier AntoineGautier force-pushed the issue3266_template_HW_plant branch from c091e76 to a02e08e Compare September 11, 2023 13:14
@AntoineGautier AntoineGautier force-pushed the issue3266_template_HW_plant branch from b623288 to 18f0bf1 Compare September 20, 2023 09:31
@AntoineGautier AntoineGautier changed the title HW plant template Templates • HW plant Sep 27, 2023
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.

Develop HW plant template
2 participants