Skip to content

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

Open
caryfaulknerPNNL wants to merge 14 commits intolbl-srg:issue3720_air_cleaningfrom
caryfaulknerPNNL:air-cleaning-CF
Open

Merging air-cleaning models into issue branch issue3720_air_cleaning.#3753
caryfaulknerPNNL wants to merge 14 commits intolbl-srg:issue3720_air_cleaningfrom
caryfaulknerPNNL:air-cleaning-CF

Conversation

@caryfaulknerPNNL
Copy link
Copy Markdown

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 <dhblum@lbl.gov>
Co-authored-by: Michael Wetter <mwetter@lbl.gov>
* Changed sensor configuration to avoid warning

For lbl-srg#3731
Comment thread Buildings/AirCleaning/BaseClasses.mo Outdated
defaultComponentName="res",
Documentation(info="<html>
<p>Partial model for duct GUV.</p>
</html>", revisions="<html>
Copy link
Copy Markdown
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.

Comment thread Buildings/AirCleaning/BaseClasses.mo Outdated
@@ -0,0 +1,649 @@
within Buildings.AirCleaning;
package BaseClasses
Copy link
Copy Markdown
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
Copy Markdown
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