-
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
Chiller Electric Reformulated EIR Plant Component Refactor #7683
Conversation
Yep, back to purely text-diffs. Feel free to commit the CondOutletTemp change again now so CI will demonstrate the diffs and we can clearly point the finger at that one variable change. Then we'll all feel better understanding that it's just a bug-fix. |
@@ -229,7 +225,7 @@ namespace ChillerReformulatedEIR { | |||
this->CondOutletNodeNum, | |||
this->QCondenser, | |||
this->CondInletTemp, | |||
RefEIRReport(this->myIdx).CondOutletTemp, | |||
this->CondOutletTemp, |
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 looks like there are a couple things going on that could cause the diff. Is it due to the removal of the RefEIRReport instance? Or the module level variable? Are you able to split out which is the root cause? It could make it even easier to digest the diff and verify we understand the implications. I feel like in the end it's going to be found that the root problem was that a temp was not properly updated on every return
path, but I'd like to see if we can nail down the exact line or code path that is causing it. It may even help resolve #7680
Deleted the results to re-run Windows on that last commit. |
The most recent commits are comment- or style-only changes. The results from the most recent CI runs should be valid. |
Thanks @mitchute. I am only like 99.8% confident in the automagic of clang-format, so I'll let CI have a pass over it to be sure. Then I'll merge this in. |
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.