Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update Slab/PerimeterInsulationDepth documentation. #341

Merged
merged 3 commits into from
Oct 11, 2022
Merged

Update Slab/PerimeterInsulationDepth documentation. #341

merged 3 commits into from
Oct 11, 2022

Conversation

shorowit
Copy link
Contributor

Closes #336.

@shorowit shorowit added the docs label Sep 13, 2022
@shorowit shorowit self-assigned this Sep 13, 2022
@shorowit shorowit added this to the v4.0 milestone Sep 13, 2022
@shorowit shorowit added the merge label Sep 13, 2022
@nmerket
Copy link
Contributor

nmerket commented Sep 20, 2022

🎥 Discussion in HPXML Working Group Meeting 2022-09-13

@nmerket
Copy link
Contributor

nmerket commented Oct 11, 2022

@shorowit This definition moved in another PR, so the merge went a little sideways. Can you verify that I got this right?

@nmerket
Copy link
Contributor

nmerket commented Oct 11, 2022

Upon further review, I'm pretty sure this is right. Merging...

@nmerket nmerket merged commit e1f4197 into master Oct 11, 2022
@nmerket nmerket deleted the perim_ins_depth_doc branch October 11, 2022 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@shorowit shorowit

Labels
docs merge
Projects
None yet
Milestone
v4.0
Development

Successfully merging this pull request may close these issues.

Update Slab/PerimeterInsulationDepth documentation
2 participants
@shorowit @nmerket