-
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
Combustion Turbine Generator Plant Comp Refactor #7559
Conversation
@Myoldmopar Ready for review. |
#include <EnergyPlus/PlantUtilities.hh> | ||
#include <EnergyPlus/UtilityRoutines.hh> | ||
|
||
namespace EnergyPlus { | ||
|
||
namespace CTElectricGenerator { | ||
|
||
//__________________________________________________________________________ | ||
//__________________________________________________________________________; |
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.
That's a funny change. 😆
NumCTGenerators = 0; | ||
getCTInputFlag = true; | ||
CTGenerator.deallocate(); | ||
} |
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.
All 3 global variables, check!
} // End Of InitLoopEquip | ||
// empty function to emulate current behavior as of conversion to using the PlantComponent calling structure. | ||
// calls from the plant side... do nothing. | ||
// calls from the ElectricPowerServiceManger call the init and calculation worker functions directly. |
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 does beg the question whether this component really ever should've been a part of the plant system if the sim call doesn't really do anything. But that's not a question for right now, this is looking good.
Real64 maxExhaustperCTPower = this->MaxExhaustperCTPower; | ||
|
||
// design turbine inlet temperature (C) | ||
Real64 designAirInletTemp = this->DesignAirInletTemp; |
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.
Kinda seems like these locals aren't even really needed, but maybe there's some logic down below where it necessitates putting the values on locals.
CI is happy, code changes look good, building and testing locally now. |
Built and ran locally. No issues. A handful of BND diffs and one EIO diff which was just a reorder on the six test files including this generator. Merging. |
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.