-
Notifications
You must be signed in to change notification settings - Fork 10
8.0 Deployment #484
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
8.0 Deployment #484
Conversation
Since Area objects have the diffload defined in the 3-3 Axis, the corresponding Type Number is 3...not 2.
In line with what already done with PierForce
Co-authored-by: Peter Nugent <[email protected]>
Co-authored-by: Peter Nugent <[email protected]>
Co-authored-by: Peter Nugent <[email protected]>
…ETABS_Toolkit-#430-AddSpandrelForceResult
No Von Mises Flexural Moment gets extracted since it's not a direct output given by ETABS. It could be possibly computed within the BHoM in case it's required.
The Von Mises Moment is not returned by ETABS natively and it must be computed directly in the BHoM from the Von Mises stresses diagram.
Duplication removed.
Thickness of panels is extracted differently depending on the type of deck/slab/wall section property assigned to the panel. The ETABS API provides different functions for each of their types and they must be all called somehow until the correct one gets hit. The control parameter is the integer ouput ret of the ETABS API functions (0=Worked Fine, 1=Failed)
Rather than moving each node one by one, all nodes are grouped into different buckets based on same movement in x/y/z direction. In this way, it is possible to call the .move() method on multiple nodes rather than on a single node at the time. This makes the algorithm much faster taking advantage of the pros of the Hash Tables data structure.
@IsakNaslundBh I am now running force checks on the checks requested. These will be queued and actioned ASAP. |
@IsakNaslundBh I have provided force checks to the checks requested. |
@BHoMBot just to let you know, I have provided a |
@BHoMBot just to let you know, I have provided a |
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.
Review by Gabriel Yakoub and @GCRA101, Tested against installer on 241216. Functionality working as expected. Some minor bugs to be addressed in coming milestone that are non-critical to functionality. Passed for merge.
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.
Agree with review comments from @Chrisshort92 .
Approved.
@BHoMBot just to let you know, I have provided a |
ETABS_Toolkit
Feature
Bug
Compliance