search

BHoM / Robot_Toolkit

Tools for Robot Structural Analysis
GNU Lesser General Public License v3.0
11 stars 3 forks source link

Add support for pulling new Modal results from robot #518

Closed IsakNaslundBh closed 1 year ago

IsakNaslundBh commented 1 year ago

NOTE: Depends on

https://github.com/BHoM/BHoM/pull/1521

Issues addressed by this PR

Add support for pulling of new modal results classes from Robot.

Test files

https://burohappold.sharepoint.com/:f:/r/sites/BHoM/02_Current/12_Scripts/02_Pull%20Request/BHoM/BHoM/Structure_oM/%231521-AddUpdatedStructuralDynamicsResults?csf=1&web=1&e=OeKgWS

Changelog

Additional comments

IsakNaslundBh commented 1 year ago

@BHoMBot check compliance

bhombot-ci[bot] commented 1 year ago
@IsakNaslundBh to confirm, the following actions are now queued: - check `code-compliance` - check `documentation-compliance` - check `project-compliance` - check `branch-compliance` - check `dataset-compliance` - check `copyright-compliance` There are 29 requests in the queue ahead of you.
IsakNaslundBh commented 1 year ago

@BHoMBot check compliance

bhombot-ci[bot] commented 1 year ago
@IsakNaslundBh to confirm, the following actions are now queued: - check `code-compliance` - check `documentation-compliance` - check `project-compliance` - check `branch-compliance` - check `dataset-compliance` - check `copyright-compliance` There are 6 requests in the queue ahead of you.
zm247 commented 1 year ago

Isak, PR's approved for the dynamic model extraction. Thanks for sorting this out. Hopefully we will get it on the next Beta version.

bhombot-ci[bot] commented 1 year ago
@IsakNaslundBh just to let you know, I have provided a `check-versioning` result to this Pull Request as it was detected to be linked to other Pull Requests in a series. The comment which triggered this check came from @FraserGreenroyd on BHoM
FraserGreenroyd commented 1 year ago

@BHoMBot check serialisation @BHoMBot check core @BHoMBot check null-handling @BHoMBot check installer

bhombot-ci[bot] commented 1 year ago
@FraserGreenroyd to confirm, the following actions are now queued: - check `serialisation` - check `core` - check `null-handling` - check `installer` There are 27 requests in the queue ahead of you.
FraserGreenroyd commented 1 year ago

@BHoMBot check ready-to-merge

bhombot-ci[bot] commented 1 year ago
@FraserGreenroyd to confirm, the following actions are now queued: - check `ready-to-merge`