-
Notifications
You must be signed in to change notification settings - Fork 172
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
CNX-619 Revit Analytical Panel #3645
base: dev
Are you sure you want to change the base?
CNX-619 Revit Analytical Panel #3645
Conversation
Separated methods pertaining to StructuralMaterial outside of the ConvertAnalyticalStick.cs file. Didn't make sense that these functions were in the ConvertAnalyticalStick.cs when the ConvertAnalyticalSurface.cs referenced them
Material properties were sent as revit internal units. Inconsistent with the Revit model / project units. These can't be used for connection applications (e.g. receiving analytical elements in ETABS)
ETABS currently only created properties for Element2Ds with a CSIProperty2D, but what about Property2D? These should also be received without us defaulting to the "Slab1" ETABS section.
Walls were previously assigned with slab sections which is incorrect. The WallPropertyToNative() was implemented (previously raised a ConversionNotSupportedException for some reason)
Testing on Revit 2022 - ETABS connection
var propName = CreateOrGetProp(area.property, out bool isExactMatch); | ||
if (!isExactMatch) | ||
var propName = CreateOrGetProp(area.property, out bool isPropertyHandled); | ||
if (!isPropertyHandled) |
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.
isPropertyHandled is a more appropriate name than isExactMatch
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.
ConvertAnalyticalSurface was using a method of GetStructuralMaterial which is inside ConvertAnalyticalStick (?). ConvertStructuralMaterial includes material related functions and these were moved out of ConvertAnalyticalStick.cs
Description & Motivation
Element2D
(fromObjects.Structural.Geometry.Element2D
) was not being received in a meaningful way in ETABSSlab1
(the first property in the list of predefined properties in a new model)Element2D
is part of the Structural kit, therefore, in a structural analysis software, at least some of the attributes (e.g. thickness and material) should be recievableElement2D
, an issue with the property and material attributes of anElement2D
coming from Revit was also noted. All of these values remained in the Revit internal units presenting a dissociation from theModelUnits
.Property2D
component can serve as input to theCSIElement2D
component. However, upon receiving in ETABS this will be handled as a genericSlab1
element.Changes:
Element2D
,Property2D
andStructuralMaterial
implementedScaleToSpeckle()
when sending from Revit. Previously Revit internal units were provided. Scalings for stress and mass density parameters was also included for consistency.displayValue
) for Revit version < 2023 was fixed, previously the 2D planar surface was displayed and not the 3D mesh of the associated physical element (done for Revit versions >= 2023 and for theElement1D
of versions < 2023)CSIConverter
was updated to handleAreaToNative()
not just for properties of typeCSIProperty2D
but also forProperty2D
Validation of changes:
Element2D
appropriately converted (floors and walls) and can be seen in the following streams:Checklist: