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

Merging air-cleaning models into issue branch issue3720_air_cleaning. #3753

Open
wants to merge 14 commits into
base: issue3720_air_cleaning
Choose a base branch
from

Conversation

caryfaulknerPNNL
Copy link

This PR is to merge the air cleaning package into this issue branch for LBNL's review. This package includes three base models: DuctGUV.mo, PAC.mo, and RoomGUV.mo, as well as base classes and examples related to those three models.

mwetter and others added 8 commits March 21, 2024 15:13
* Added insulation to idf file and resized system

* Added insulation and resized heating system
* Corrected wrong annotation

* Updated copyright year

* Corrected user guide
This merges IBPSA, issue1855_aquifer_documentation
* Rename models
* Converted DHC user guide to model instead of package
* Added missing link to userguide

---------

Co-authored-by: David Blum <[email protected]>
Co-authored-by: Michael Wetter <[email protected]>
* Changed sensor configuration to avoid warning

For lbl-srg#3731
defaultComponentName="res",
Documentation(info="<html>
<p>Partial model for duct GUV.</p>
</html>", revisions="<html>
Copy link
Contributor

Choose a reason for hiding this comment

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

It should be a new revision record.

@@ -0,0 +1,649 @@
within Buildings.AirCleaning;
package BaseClasses
Copy link
Contributor

Choose a reason for hiding this comment

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

comment of the package

@JayHuLBL
Copy link
Contributor

Screenshot 2024-03-29 at 1 05 57 PM

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.

5 participants