-
Notifications
You must be signed in to change notification settings - Fork 393
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
Convert window shading flag to enum class #8435
Merged
Merged
Changes from 36 commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
bb83a27
first commit
xuanluo113 d33eb73
fix a bug
xuanluo113 4815c24
Merge branch 'develop' of https://github.com/NREL/EnergyPlus into sha…
xuanluo113 79aa3d8
fix another
xuanluo113 8664ca3
fix more
xuanluo113 b013ad5
Merge branch 'develop' of https://github.com/NREL/EnergyPlus into sha…
xuanluo113 f455c3e
fix more
xuanluo113 1959173
ems fix
xuanluo113 f9748ed
merge
xuanluo113 4dcba6e
fix ems type in unit test
xuanluo113 0ea1c27
merge
xuanluo113 f645e08
Merge branch 'develop' of https://github.com/NREL/EnergyPlus into sha…
xuanluo113 91e0628
bug fixing
xuanluo113 bc631d8
fix
xuanluo113 89feb91
Merge branch 'develop' of https://github.com/NREL/EnergyPlus into sha…
xuanluo113 88347f6
remove shaded
xuanluo113 6aabfc7
fix index
xuanluo113 cdf9ff7
macros
xuanluo113 daf5a74
more macros
xuanluo113 6bcbe09
window control type
xuanluo113 ad7df70
Merge branch 'develop' of https://github.com/NREL/EnergyPlus into sha…
xuanluo113 6cc1bf4
naming
xuanluo113 e7af4ec
replace first part
xuanluo113 1cff5a5
window control type enum
xuanluo113 d58aed0
remove attributes
xuanluo113 af671a5
address comments
xuanluo113 5bfefa3
temporary fix
xuanluo113 45edddf
EMS value
xuanluo113 726654f
Merge branch 'develop' of https://github.com/NREL/EnergyPlus into sha…
xuanluo113 e99c7fe
EMS value
xuanluo113 6bb6ae4
fix glare control logic
xuanluo113 f9582dc
bug fix
xuanluo113 3fe9ff2
Merge branch 'develop' of https://github.com/NREL/EnergyPlus into sha…
xuanluo113 9e876f9
merge
xuanluo113 6679ba5
Merge branch 'develop' of https://github.com/NREL/EnergyPlus into sha…
xuanluo113 0107de2
Merge remote-tracking branch 'upstream/develop' into shading-flag
Myoldmopar e652b59
Merge remote-tracking branch 'upstream/develop' into shading-flag
Myoldmopar 5491529
Merge branch 'shading-flag' of https://github.com/energy-plus/EnergyP…
xuanluo113 d389e71
address comments
xuanluo113 b777b7d
unit test
xuanluo113 f3bce29
Merge branch 'develop' of https://github.com/NREL/EnergyPlus into sha…
xuanluo113 6be6636
Merge branch 'develop' of https://github.com/NREL/EnergyPlus into sha…
xuanluo113 b723cb7
Merge branch 'develop' of https://github.com/NREL/EnergyPlus into sha…
xuanluo113 0f5b941
Merge branch 'develop' of https://github.com/NREL/EnergyPlus into sha…
xuanluo113 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
So, it appears that Shading Types have been mixed in/consolidated with the Shade Status types. I'm not sure that's a good plan. While similar,
WSC_ST_InteriorShade
had a different meaning fromIntShadeOn
. I'll see how this plays out in the sections of code that use(d) these two sets of values.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 Please see this thread for the explanation. You are right. Mixing the types and status was not an optimal design decision in the first place (I only reassured this when I was 80% done with this branch). But changing this may require some redesign of the logic and a lot more refactoring. It's more like a refactoring rather than a 10x effort. I think this could happen later.