-
Notifications
You must be signed in to change notification settings - Fork 3
Add FrameEdgeProperties to Facade datasets #118
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
Conversation
Added FrameEdgeProperty dataset. Removed outdated datasets.
@BHoMBot check compliance |
@enarhi to confirm, the following checks are now queued:
|
Datasets updated to include author and confidence level
@BHoMBot check compliance |
@enarhi to confirm, the following checks are now queued:
|
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.
LGTM. A separate PR will pick up remaining items for updating Kawneer fragments and the material naming issue for the Transparent_Materials data used.
Closes #117 |
NOTE: Depends on Facade_oM #1363
Issues addressed by this PR
Closes #112
Adds generic datasets for FrameEdgeProperties and GlazingConstructions
Test Files
https://burohappold.sharepoint.com/sites/BHoM/02_Current/Forms/AllItems.aspx?csf=1&web=1&e=HlvYHp&cid=ab402372%2D7702%2D48ef%2Db6bc%2D9c3f66547c0a&RootFolder=%2Fsites%2FBHoM%2F02%5FCurrent%2F12%5FScripts%2F03%5FAlpha%2FBHoM%2FBHoM%5FDatasets%2FFacade%20Datasets&FolderCTID=0x0120008122C8891F89054B8ACED0196C70DFC4