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
Improved Controller:WaterCoil FindRoot error message with TemperatureAndHumidityRatio #10435
Conversation
I was thinking if we knew if CW or HW coil the message could be made even clearer. |
@rraustad We do figure out the coil type earlier, but haven't been saving that. How's this?
Anything else you would like to add? |
This looks good, and I could live with it, but... You already know the controller is set to reverse action:
So these new lines don't need the "Chilled water coils should be reverse action and" in this case but do if controller is set to Normal action (I suspect this is warned elsewhere for CW coils?).
And now I know what's causing the issue with the UnmetHours post (but not sure how to fix it):
|
How's 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.
This warning is much clearer now and targets specific coil types (e.g., CW vs HW).
This can merge once CI is clean. Now I want to know why the SPM or controller code was setting the CW SP temp below the CW temp (where CW SP temp = humrat SP???), and if that's a problem, and if that can be corrected. |
Well, that's a good point, if the setpoint was limited to the ChW temp, then the findroot error shouldn't occur. The humidity control won't be achieved, but it's not achieved anyway in this case. I can try adding that here. |
Should the setpoint always be adjusted to >= ChW temp (and similar to HW temp)? Even without humidity control? |
From what I recall of the controller code, I think the initial conditions need to be static, min/max flow, min/max SP, etc. There is a place where you updated the controller to switch between temp and humidity control where I think this is happening but not sure why it's showing up now. I think if on one iteration the CW SP is changed and then on a subsequent interation that CW SP is changed to something outside the initial limits then there is a problem. It's like a SolveRoot failure with -2.
|
I think what's happening is the controller has switched to humrat control, set the controller vars based on humrat, and then reports that data as if it's temperature controlled. Here the SP is lower than the sensed value, which is fine, but the water temp is still the water temp at 8 C. Why haven't we seen this before?
|
Correct, when humidity control is active, the sensed value is changed to humidity ratio, not temperature. Will re-work this section, yet again. I think I'll report the dewpoint temp of the humrat setpoint and say the chilled water temp needs to be below that. Also, this file has other issues that are causing this problem in the first place - the chilled water coil is coming on at 8pm on January 1, probably because the OA dampers have closed and the fixed supply air temp in this VAV system wants cold air . . . |
Yes, but the controller converged on temperature control? before it switched over to humrat control. Is it just the warning that is incorrectly being reported? i.e., the humrat is below the water temp so the code threw this warning. |
This feels like it might be ready...but the discussion earlier doesn't feel entirely wrapped. I'm not sure if I should dig in here or not. @rraustad approved the changes, so I'm leaning toward reviewing/merging. I'll pivot to other PRs for now but come back to this later. |
I have no other suggestions on the warning message. The issue of why this message is being reported can be addressed here or elsewhere. @mjwitte ? |
Assuming CI is all green, this is finished.
-------- Original message --------From: Richard Raustad ***@***.***> Date: 3/15/24 8:10 AM (GMT-06:00) To: NREL/EnergyPlus ***@***.***> Cc: "Michael J. Witte" ***@***.***>, Mention ***@***.***> Subject: Re: [NREL/EnergyPlus] Improved Controller:WaterCoil FindRoot error
message with TemperatureAndHumidityRatio plus Limit setpoints by water temp
(PR #10435)
I have no other suggestions on the warning message. The issue of why this message is being reported can be addressed here or elsewhere. @mjwitte ?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Pull request overview
This pull request modifies the error message to:
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.