-
Notifications
You must be signed in to change notification settings - Fork 389
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
EIO transition changes for simple glazing #9179
Conversation
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.
@xuanluo113 A couple of minor comments and a request to fix some existing formatting issues, please, thanks, just for the window construction section, no need to work through other sections.
@@ -2346,7 +2346,7 @@ \subsubsection{Description of the Windows Construction Report}\label{description | |||
|
|||
\subsubsection{Fields in Part 1 -- Window Construction Report}\label{fields-in-part-1-window-construction-report} | |||
|
|||
\textbf{Line 1:} ! \textless{}WindowConstruction\textgreater{},Construction Name,\#Layers,Roughness,Conductance \{W/m2-K\},SHGC,SolarTransmittanceNormalIncid,VisibleTransmittanceNormalIncid | |||
\textbf{Line 1:} ! \textless{}WindowConstruction\textgreater{},Construction Name,\#Layers,Roughness,Conductance \{W/m2-K\},Conductance (Before Adjusted) \{W/m2-K\},Convection Coefficient Adjustment Ratio,SHGC,SolarTransmittanceNormalIncid,VisibleTransmittanceNormalIncid |
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.
The section below here include a description of each "field". Please add descriptions for the 2 new fields.
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.
Also, since you're working in this file already, please update the old names "Material:WindowGlass" etc. to "WindowMaterial:Glazing" etc.
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.
And it would be nice to use \begin{lstlisting}
style for the sample eio lines (e.g. Chiller Standard Rating Information).
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.
@mjwitte I added the field descriptions and modified the list to \begin{lstlisting}
. Can you please take a look if that's what you mean?
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.
For the old "Material:WindowGlass" parts, I'm not following. Does it mean the Material:WindowGlass section should be deleted as Window Constructions section is a replacement?
Besides, part 2 fields in Window Constructions seem to be duplicated with part 1.
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.
No, part 2 is for the material layers which follow for each WindowConstruction line. All I was asking for here was to substitute 3 instances of "Material:WindowGlass" (which is from long long ago) to "WindowMaterial:Glazing". And likewise for Material:WindowGas, Material:WindowShade, Material:WindowBlind, and Material:WindowScreen.
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.
Hope the latest commit fixed it.
### WindowConstruction EIO Fields Change | ||
Two fields are added to the Description of the Windows Report - "WindowConstruction", namely "Construction Report Conductance (Before Adjusted) {W/m2-K}" and "Convection Coefficient Adjustment Ratio". They represent the glazing-only nominal U (nominal conductance in winter) and the adjustment ratio assuming highly conductive frames are applied along with the glazing. | ||
|
||
See PR [#8653](https://github.com/NREL/EnergyPlus/pull/8653). |
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.
Also reference #9117 since that's when the actual eio output changes were made.
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.
Done.
|
||
\subsubsection{Field: Convection Coefficient Adjustment Ratio}\label{field-convection-coefficient-adjustment-ratio} | ||
|
||
For simple glazing (WindowMaterial:SimpleGlazingSystem), this is the adjustment ratio to both inside and outside window film coefficients assuming highly conducting frames are applied to achieve a high Nominal U-Factor. The adjustment ratio is calculated such that the glazing-only conductance evaluated at winter conditions equals the input Nominal U-Factor. |
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.
@xuanluo113 I did some more cleanup and updated the text for this field. Please confirm if this is accurate.
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.
@mjwitte Reads great.
EIO transition changes for simple glazing
Pull request overview
Adding Documentation and Transition Change descriptions for the previously merged branch #9117.
Pull Request Author
Add to this list or remove from it as applicable. This is a simple templated set of guidelines.
Reviewer
This will not be exhaustively relevant to every PR.