-
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
Documentation Fixes - 2018b #7737
Conversation
@mjwitte I'm going to consider this blocked until #7448 merges. There's a comment in #7026 (the relevant issue for this PR) to update documentation related to There are also a number of other issues referenced by @JasonGlazer which look to me like they may cause some reordering of the docs to match the ordering in the IDD. I'll need to discuss these with you and/or Jason or Edwin before working on those. |
I don't know if the docs or the IDD need to change order and names but they should be consistent. |
There is one minor code change here (a unit change at one location) and the rest are doc updates. Ready for review. |
@mitchute it has been 8 days since this pull request was last updated. |
@@ -139,7 +139,7 @@ \subsubsection{Inputs}\label{inputs-007} | |||
|
|||
\paragraph{Field: Basin Heater Setpoint Temperature}\label{field-basin-heater-setpoint-temperature} | |||
|
|||
This numeric field contains the set point temperature (˚C) for the basin heater described in the previous field. The basin heater is active when the outdoor air dry-bulb temperature falls below this setpoint temperature, as long as the tower fan is off and water is not flowing through the tower. This set point temperature must be greater than or equal to 2˚C, and the default value is 2˚C if this field is left blank. | |||
This numeric field contains the set point temperature (°C) for the basin heater described in the previous field. The basin heater is active when the outdoor air dry-bulb temperature falls below this setpoint temperature, as long as the tower fan is off and water is not flowing through the tower. This set point temperature must be greater than or equal to 2˚C, and the default value is 2˚C if this field is left blank. |
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.
It's kinda hard to see here, but did this also need the other degree symbols in this line to be replaced?
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 if so, there is a conflict in the doc cmake rules. I'm happy to help resolve those and address this at the same time, if it needs to be done. Otherwise I can just resolve the conflicts.
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'll fix it up now.
I've built locally, everything is fine there. So there are a number of references to the blast translator and DOE translator in the auxiliary programs pdf. The removal of the tex files seems fine, but maybe we need to remove all the references? I see that you asked that on Feb 3 @mitchute. In my opinion, yeah I think so, but I don't mind if it is right now or later. I pulled in develop and resolved the conflict. I'll push those changes up. I'm inclined to just drop this in for now and add the final blast/doe2 cleanups to a future issue. @mjwitte if you have a strong feeling about cleaning all this up right now, we can do it, but otherwise this should just drop in. |
Bombs away . . . drop it in. |
@@ -3369,7 +3369,7 @@ namespace PlantCondLoopOperation { | |||
SetupEMSActuator(ActuatorName, | |||
UniqueIDName, | |||
ActuatorType, | |||
"[W]", | |||
"[fraction]", |
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.
@Myoldmopar @mitchute Does this really work as a fraction? The other supervisory on/off actuators here use Units
which = "[on/off]".
EnergyManagementSystem:Actuator Available,CHILLED WATER LOOP,Supply Side Half Loop,On/Off Supervisory,[on/off]
EnergyManagementSystem:Actuator Available,CW DEMAND SIDE INLET PIPE,Plant Component Pipe:Adiabatic,On/Off Supervisory,[fraction]
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 See this comment: #7026 (comment)
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.
Well, fancy that. Who knew. Carry on.
This PR addresses documentation issues referenced in #7026.
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.