-
Notifications
You must be signed in to change notification settings - Fork 396
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
Sizing speed up related to reducing the number of warmup days for design days #7567
Changes from 3 commits
996c27e
6f92fc0
2c8de15
5a78a2e
84ae8b6
aa62db0
b3231ef
c355018
30f2ff1
534225e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -482,11 +482,12 @@ Building, | |
\key Urban | ||
\default Suburbs | ||
N2 , \field Loads Convergence Tolerance Value | ||
\note Loads Convergence Tolerance Value is a fraction of load | ||
\note Loads Convergence Tolerance Value is a change in load from one warmup day to the next | ||
\type real | ||
\minimum> 0.0 | ||
\maximum .5 | ||
\default .04 | ||
\units W | ||
N3 , \field Temperature Convergence Tolerance Value | ||
\units deltaC | ||
\type real | ||
|
@@ -511,12 +512,14 @@ Building, | |
N5 ; \field Minimum Number of Warmup Days | ||
\note The minimum number of warmup days that produce enough temperature and flux history | ||
\note to start EnergyPlus simulation for all reference buildings was suggested to be 6. | ||
\note However this can lead to excessive run times as warmup days can be repeated needlessly. | ||
\note For faster execution rely on the convergence criteria to detect when warmup is complete. | ||
\note When this field is greater than the maximum warmup days defined previous field | ||
\note the maximum number of warmup days will be reset to the minimum value entered here. | ||
\note Warmup days will be set to be the value you entered when it is less than the default 6. | ||
\note Warmup days will be set to be the value you entered. The default is 1. | ||
\type integer | ||
\minimum> 0 | ||
\default 6 | ||
\default 1 | ||
|
||
ShadowCalculation, | ||
\unique-object | ||
|
@@ -1155,13 +1158,26 @@ SizingPeriod:DesignDay, | |
\minimum 0 | ||
\maximum 3 | ||
\default 0 | ||
N14; \field Sky Clearness | ||
N14, \field Sky Clearness | ||
\note Used if Sky Model Indicator = ASHRAEClearSky or ZhangHuang | ||
\minimum 0.0 | ||
\maximum 1.2 | ||
\default 0.0 | ||
\note 0.0 is totally unclear, 1.0 is totally clear | ||
\type real | ||
N15, \field Maximum Number Warmup Days | ||
\note If used this design day will be run with a custom limit on the maximum number of days that are repeated for warmup. | ||
\note Limiting the number of warmup days can improve run time. | ||
\note TRACE(R) 3D Plus custom feature. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @Myoldmopar I think we want to delete this note? And same for 1176. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Right, this should go away |
||
\type integer | ||
A13; \field Begin Environment Reset Mode | ||
\note If used this can control if you want the thermal history to be reset at the beginning of the design day. | ||
\note When using a series of similiar design days, this field can be used to retain warmup state from the previous design day. | ||
\note TRACE(R) 3D Plus custom feature. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. missed it here |
||
\type choice | ||
\key FullResetAtBeginEnvironment | ||
\key SuppressAllBeginEnvironmentResets | ||
\default FullResetAtBeginEnvironment | ||
|
||
SizingPeriod:WeatherFileDays, | ||
\memo Use a weather file period for design sizing calculations. | ||
|
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.
So the code was already interpreting this input as W, but it was described as a fraction? If so, it appears that the E+ documentation needs to be updated to reflect this.
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.
Yeah if this was described as a fraction in the docs then they need to be changed.