-
Couldn't load subscription status.
- Fork 7
Add accumulator examples. #571
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
Conversation
|
Please take a look and mention any suggestions for improvement. |
doc/src/records/accumulator.md
Outdated
|
|
||
| REPORT rpType=UDT rpFreq=Month rpDayBeg=Jan 1 rpDayEnd=Dec 31 | ||
| REPORTCOL colHead="Month" colVal=$Month colWid= 3 | ||
| REPORTCOL colHead="Monthly Total" colVal=@Accumulator["Zone Heating Set Point"].M.acmSum colDec=0 colWid=10 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure anyone would ever want the sum of a temperature output. Maybe we can find a different probe to use as an example? Maybe something like unmet load hours?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I meant to do average.
I'll find a different probe.
doc/src/records/accumulator.md
Outdated
| REPORTCOL colHead="Month" colVal=$Month colWid= 3 | ||
| REPORTCOL colHead="Day" colVal=$Day colWid= 3 | ||
| REPORTCOL colHead="Hour" colVal=$Hour colWid= 3 | ||
| REPORTCOL colHead="Hourly Window Transmitted Solar Gain" colVal=@Accumulator["Window Transmitted Solar Gain"].H.acmSum colDec=0 colWid=10 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Similar to above, I'm not sure if this is going to produce a meaningful output by adding something that has power dimensions (e.g., Btu/h). I can see where you might want to accumulate total window transmitted energy, but that would also require you to account for the subhour time step size.
|
I updated the probes. Let me know what you think. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One of us needs to dig a bit deeper to make sure the interpretations here are correct.
| REPORTCOL colHead="Total" colVal=@Accumulator[ 1].M.acmSum colDec=0 colWid=10 | ||
| REPORTCOL colHead="Average" colVal=@Accumulator[ 1].M.acmMean colDec=0 colWid=10 | ||
| ACCUMULATOR "Zone Sensible Cooling Rate [Btu/h]" | ||
| acmValue = @znRes[ 1].H.qscHvac // Hourly Output |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this the average Btu/h over the hour or the total Btu over the hour? The answer will impact the accuracy of the report values below.
| REPORTCOL colHead="Monthly Min Hourly Cooling Rate [Btu/h]" colVal=@Accumulator["Zone Sensible Cooling Rate [Btu/h]"].M.acmMin colDec=0 colWid=10 | ||
|
|
||
| ACCUMULATOR "Zone Solar Gain [Btu]" | ||
| acmValue = @znRes[ 1].S.qSlr // Subhourly Output |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Similar question here. I'm not sure you are using the correct units / interpretation here.
Description
I added two examples for how to use accumulators in the documentation.