-
Notifications
You must be signed in to change notification settings - Fork 392
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
Update to shading airflow coefficients feb2022 #9314
Conversation
- Fixed bug in WCE when reported U-value for single layer with exterior shade was incorrect.
…fficients_Feb2022
…fficients_Feb2022
@vidanovic Is this intended for v22.1 release? |
@mjwitte Yes, for the April release. |
I'm building and testing this locally now. Most of the changes here are buried down in the third-party WCE code, so I won't comment on any design or functionality there. The rest of the changes look minimal, but they are expected to produce diffs due to updated coefficients. I'll merge develop in here one more time, but I suspect this can go after that. |
@mitchute Thanks! |
For some reason the Windows CI bot specifically had gone down, but the Linux ones were all running well. Probably more forced updates. Anyway, this seems like reasonable CI results. No reason to really hold it up I don't think. |
Pull request overview
NOTE: ENHANCEMENTS MUST FOLLOW A SUBMISSION PROCESS INCLUDING A FEATURE PROPOSAL AND DESIGN DOCUMENT PRIOR TO SUBMITTING CODE
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.