Structure_oM: Add classes for retaining wall objects

[KalleEdstroem]

NOTE: Depends on

Issues addressed by this PR

Closes #1619
Closes #1634

Added definition for retaining wall objects

• RetainingWall
• Stem
• (making use of already defined PadFoundation as a footing)

Test files

Grasshopper test file.
Flowchart of classes and comments on assumptions.

https://burohappold.sharepoint.com/:f:/r/sites/BHoM/02_Current/12_Scripts/01_Issue/BHoM/BHoM/Structure_oM/%231621-retainingWallObjects?csf=1&web=1&e=who8Qd

Changelog

• RetainingWall class added in Structure_oM
• Stem class added in Structure_oM

Additional comments

[KalleEdstroem]

Objects definition and required methods implemented.

Two methods are still WIP and need updates from my side:

• Geometry3D() for Stem
• MaterialComposition() for RetainingWall

Regarding the Stem and Geometry3d(). Need to agree if the thicknessBottom and thicknessTop should taper in a pyramid shape or if one side always should be in a plane perpendicular to the XY-plane and one side should be angled. Then it needs to be achieved in the Geometry3D() which is a bit tricky.

The MaterialComposition() needs some work figuring out how to handle the if a ReinforcementDenisty fragment is attached to both either the Stem or Footing and the Higher level object RetainingWall.

[KalleEdstroem]

Discussions offline agreed on making the stem into a pyramid shape to start with for Geometry3D() method.

MaterialComposition() Updated and gives correct output. Have one persistent issue with an error message getting dispalyed for when ReinforcementDenisty is used from lower level objects (Stem and Footing).

[peterjamesnugent]

Some description changes - and I wonder if we sub out Footing for PadFoundation - lets discuss when you're back.

[peterjamesnugent]

@BHoMBot check compliance

[bhombot-ci]

@peterjamesnugent 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 1 requests in the queue ahead of you.

[peterjamesnugent]

@BHoMBot check required

[bhombot-ci]

@peterjamesnugent to confirm, the following actions are now queued:

• check code-compliance
• check documentation-compliance
• check project-compliance
• check core
• check null-handling
• check serialisation
• check versioning
• check installer

There are 56 requests in the queue ahead of you.

[bhombot-ci]

@KalleEdstroem 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 @peterjamesnugent on BHoM_Engine

[bhombot-ci]

@KalleEdstroem just to let you know, I have provided a check-installer 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 @peterjamesnugent on BHoM_Engine

[bhombot-ci]

The check versioning has already been run previously and recorded as a successful check. This check has not been run again at this time.

[bhombot-ci]

The check installer has already been run previously and recorded as a successful check. This check has not been run again at this time.

[peterjamesnugent]

@BHoMBot check copyright-compliance

[bhombot-ci]

@peterjamesnugent to confirm, the following actions are now queued:

• check copyright-compliance

There are 28 requests in the queue ahead of you.

[bhombot-ci]

@KalleEdstroem just to let you know, I have provided a check-installer 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 @peterjamesnugent on BHoM_Engine

[peterjamesnugent]

Discussed with @IsakNaslundBh. I have reviewed the PR raised by @KalleEdstroem including testing using the test-scripts.

My commits relate to adding inheritance of IRegion for the Stem and PadFoundation objects and the versioning that results from that.

[bhombot-ci]

@KalleEdstroem 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 @peterjamesnugent on BHoM_Engine

[bhombot-ci]

@KalleEdstroem just to let you know, I have provided a check-installer 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 @peterjamesnugent on BHoM_Engine

[IsakNaslundBh]

Slight comment on the versioning. Especially the ToOld becomes a bit nonsensical without specifying the type. (ToOld is not used anywhere, but if we ever do start using it, We should make sure it is correct)

[bhombot-ci]

@KalleEdstroem 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 @pawelbaran on BHoM_Engine

[bhombot-ci]

@KalleEdstroem 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 @peterjamesnugent on BHoM_Engine

[bhombot-ci]

@KalleEdstroem just to let you know, I have provided a check-installer 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 @peterjamesnugent on BHoM_Engine

[peterjamesnugent]

@BHoMBot check required

[bhombot-ci]

@peterjamesnugent to confirm, the following actions are now queued:

• check code-compliance
• check documentation-compliance
• check project-compliance
• check core
• check null-handling
• check serialisation
• check versioning
• check installer

[IsakNaslundBh]

Approving after updates to the versioning

[peterjamesnugent]

@BHoMBot check copyright-compliance

[bhombot-ci]

@peterjamesnugent to confirm, the following actions are now queued:

• check copyright-compliance

There are 9 requests in the queue ahead of you.

[IsakNaslundBh]

@BHoMBot check ready-to-merge

[bhombot-ci]

@IsakNaslundBh to confirm, the following actions are now queued:

• check ready-to-merge