EnergyPlus™ is a whole building energy simulation program that engineers, architects, and researchers use to model both energy consumption and water use in buildings.
While working on changes to daylighting for Spaces, there were several comments suggesting further refactoring. These comment links aren't the greatest (because they don't show the line of code being commented on), but following into the specific commits will show the line of code at the time. These comments were posted during 9.6.0 development (roughly August-Sept 2020), so it's possible some of this code has already changed due to other refactoring sweeps.
[ ] See if call to DaylightingManager::initDaylighting in HeatBalanceSurfaceManager::InitSurfaceHeatBalance can be skipped if there's no daylighting in the model - or maybe skip some sections in that function.
Details
Some additional details for this issue (if relevant):
Platform (Operating system, version)
Version of EnergyPlus (if using an intermediate build, include SHA)
Unmethours link or helpdesk ticket number
Checklist
Add to this list or remove from it as applicable. This is a simple templated set of guidelines.
While refactoring DaylightingManager.hh please also think about eliminating use of so many old short Fortran variable names such as DILLSW (and dozens of other 2 to 6 character variable names in all caps).
Issue overview
While working on changes to daylighting for Spaces, there were several comments suggesting further refactoring. These comment links aren't the greatest (because they don't show the line of code being commented on), but following into the specific commits will show the line of code at the time. These comments were posted during 9.6.0 development (roughly August-Sept 2020), so it's possible some of this code has already changed due to other refactoring sweeps.
DaylightingManager::initDaylighting
inHeatBalanceSurfaceManager::InitSurfaceHeatBalance
can be skipped if there's no daylighting in the model - or maybe skip some sections in that function.Details
Some additional details for this issue (if relevant):
Checklist
Add to this list or remove from it as applicable. This is a simple templated set of guidelines.