-
Notifications
You must be signed in to change notification settings - Fork 394
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
Correction of Various Errors in the Tips and Tricks Documentation #7857
Conversation
A variety of problems were noted in Tips and Tricks. Some of these were web sites that were no longer valid. Others were glaring formatting issues. In some cases, minor language edits were made (like where a chapter was totally blank). While this was not a full clean-up of the document, it looks much better and all of the hyperlinks should now work and point to the correct address.
This seems like a good set of straightforward changes. I will build locally but I don't have any problems. And CI is happy of course, with one ERR file that is quite unrelated 😆 . |
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.
Good to go. Built locally, all changes look fine.
San Francisco & 0:50 & 1:34 & 9:59 \tabularnewline | ||
Colorado & 0:51 & 1:40 & 10:31 \tabularnewline | ||
Tampa & 0:51 & 1:36 & 10:05 \tabularnewline | ||
Washington DC & 0:51 & 1:36 & 9:48 \tabularnewline |
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.
Tables look much nicer here.
@@ -4,4 +4,4 @@ \section{Weather Data for Simulations}\label{weather-data-for-simulations} | |||
|
|||
Drury B. Crawley. 1998. ``Which Weather Data Should You Use for Energy Simulations of Commercial Buildings?'' in ASHRAE Transactions, pp.~498-515, Vol. 104, Pt. 2. Atlanta: ASHRAE. (PDF 197 KB) | |||
|
|||
PDF: \url{http://energyplus.gov/pdfs/bibliography/whichweatherdatashouldyouuseforenergysimulations.pdf} | |||
Click \href{https://energyplus.net/sites/all/modules/custom/weather/weather_files/whichweatherdatashouldyouuseforenergysimulations.pdf}{here} for the PDF of the above paper. |
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.
Works for me.
@@ -1,6 +1,6 @@ | |||
\section{Variable Refrigerant Flow Air Conditioner}\label{variable-refrigerant-flow-air-conditioner} | |||
|
|||
\textbf{\emph{The EnergyPlus v7.0 release (October 2011) includes an initial model for VRF systems.~ See AirConditioner:VariableRefrigerantFlow and related objects.}} | |||
\textbf{\emph{Since its V7.0 release (October 2011), EnergyPlus has included a model for VRF systems.~ See AirConditioner:VariableRefrigerantFlow and related objects.}} |
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.
Better wording for sure.
@@ -20,7 +20,9 @@ \subsection{WINDOW Data File}\label{window-data-file} | |||
|
|||
Please note that there is a bug in WINDOW 5 that causes two of the lines in the EnergyPlus data file to be joined. This bug is fixed in versions of Window 5.02 (and above). To be sure, you can check the data file for a line that looks like: | |||
|
|||
GLAZING SYSTEM OPTICAL DATAAngle~~~~ 0~~~ 10~~~ 20~~~ 30~~~ 40~~~ 50~~~ 60~~~ 70~~~ 80~~~ 90 Hemis | |||
GLAZING SYSTEM OPTICAL DATA |
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.
👍
@@ -1 +1,3 @@ | |||
\chapter{Utilities}\label{utilities} | |||
|
|||
EnergyPlus comes with a wide variety of pre- and post-processing that are found in the PreProcess and PostProcess folders in the main EnergyPlus install folder. These utilities have been developed since the initial release of EnergyPlus and can be helpful during the creation of input data for EnergyPlus or interpreting its 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.
Yeah that'll be better than nothing...literally!
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 know! 😂😂😂 How long has this been like this?🤦♂️ Thanks for reviewing this!
Pull request overview
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.