-
Notifications
You must be signed in to change notification settings - Fork 197
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
WaterHeater:Mixed not forward translated when not on loop #1675
Comments
Agreed that this should be supported. One thing I dislike about is that the "Peak Use Flow Rate", which is the condition to check if the WH isn't hooked to a PlantLoop to see if it is really meant standalone, was made required-field and is default-initialized in the ctor to 0.1 m3/s. That basically means ANY WaterHeaterMixed in your model will be translated no matter what. That could be confusing and could be hard to tell that it is actually happening. Perhaps that's ok though. @kbenne thoughts please? |
Given that Peak Use Flow Rate is required, it seems like we don't have the option to selectively translate this type anymore. My feeling is that we should just accept that it is always translated, and therefore do what @mbadams5 suggests and just add it to the list of always translated types. I'm assuming that making Peak Use Flow Rate optional would be a breaking api change, and I wouldn't changing the API. |
Fix #1675 - ForwardTranslate WaterHeater:Mixed when not on loop but has Peak Use Flow Rate (standalone operation)
If a WaterHeater:Mixed is not on a SWH PlantLoop then it is not forward translated.
A water heater can be standalone when PeakUseFlowRate and UseFlowRateFractionSchedule are set.
I came across this in a few prototype buildings where this is used.
This iddobjectype just need to be added to here
One option around this is to create a plantloop with a pump that has basically zero head pressure and a water use equipment that is the same as the PeakUseFlowRate and has a schedule the same as UseFlowRateFractionSchedule.
fyi @kbenne
The text was updated successfully, but these errors were encountered: