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

Issue3288 dx coil heating #69

Open
wants to merge 111 commits into
base: issue3327_SPAWN_PTHPdevelopment
Choose a base branch
from

Conversation

karthikeyad-pnnl
Copy link
Owner

No description provided.

mwetter and others added 30 commits March 11, 2023 08:07
* Improved memory allocation

* corrected html syntax
* Fix reverse acting

* Update reference results

* Update revisions and release notes
* Corrected CDL conformance

For #3301

* Redirected documentation of MSL classes to doc.modelica.org

* Redirected documentation of MSL classes to doc.modelica.org

* Updated url to https
Added package Fluid.HydronicConfigurations

---------

Co-authored-by: David Blum <[email protected]>
Co-authored-by: Michael Wetter <[email protected]>
* improved validation models

* changed the initial value to avoid difference results between dymola and omc

* Tightened tolerance of reference results, enabled result verification

This enables result verification with OpenModelica for the OBC.CDL package

* Tightened tolerance

---------

Co-authored-by: Michael Wetter <[email protected]>
* Added example for air heat and mass balance.
* corrected implementation and updated references

---------

Co-authored-by: Michael Wetter <[email protected]>
* copied Obsolete.Fluid.Movers from IBPSA

* corrected constrainedby clause for OCT warning

* deleted Nrpm component in HydronicConfigurations

* renamed speed in enum
* Renamed AirCooled to AirSource, and same for WaterCooled

For #3288

* Restructure the DX coil package to implement the DX heating coil

* Modify the other models that are impacted by the structure change
* correcting the nominal conditions

* corrected assert condition
* added chiller plant controller namings, #819 [ci skip]
* moved PID controller instance to public section

* improved instance names
* adding proof block
* Renamed CDL.Continuous.Sources.Sine to Sin

For #3339

* Updated revision notes and image path
This sets the start value for NTUAirSta
mwetter and others added 30 commits May 31, 2023 17:56
X is not available on the weather data bus
This allows the record to be replaced by models that extend this model
This separates the base classes to those used for heating, cooling, or both
* fixed links in Buildings.Fluid.Geothermal

* Fluid.HydronicConfigurations.ActiveNetworks

Buildings/Fluid/HydronicConfigurations/ActiveNetworks/Examples$ sed -i 's/Buildings.Fluid.HydronicConfigurations.Examples.BaseClasses.Load/Buildings.Fluid.HydronicConfigurations.ActiveNetworks.Examples.BaseClasses.Load/g' $(find . -iname '*.mo')

* removed links

The links could not be resolved because they lead to constants instead of classes.

* Obsolete.Fluid.Movers.Data
This integrates IBPSA 1762.
This corrects the variability as assigning an input to a parameter is not allowed
This improves the documentation
* Moved data record for heating coil validation from cooling subpackage to heating subpackage

* Updated documentation with calculations from EPlus engineering reference. Updated parameter comments

* Created individual user guides for heating and cooling DX coils from common user guide

* Updated documentation with reference to heating coil

* Updated parameter on input interface

* Updated package.order files

* Updated EPlus files from version 22.2 to 23.1

* Removed unused parameter assignment
This avoids a warning in Optimica as the heat port may not be present in all plug compatible models
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.

7 participants