Structure_Engine: Add engine methods for retaining wall classes

[peterjamesnugent]

NOTE: Depends on

BHoM/BHoM#1621

Issues addressed by this PR

Closes BHoM_Structure_oM

Test files

See.
BHoM/BHoM#1621

Changelog

Added required extension methods for implemented interfaces for new retaining wall classes.

• Added Geometry() method for Stem and RetainingWall
• Added IsNull() method for Stem and RetainingWall
• Added SolidVolume() method for Stem and RetainingWall
• Added MaterialComposition() method for Stem and RetainingWall
• Added SetOutlineElements(List<IElement1D> outlineElements1D) method for Stem
• Added OutlineElemets1D() method for Stem
• Added Geometry3D() method for Stem and RetainingWall
• Migrated IsQuad, IsTriangular, IsRectangular, IsSquare to the Geometry namespace so that it can be used on PolyCurves - the methods in BH.Engine.Analytical are unchanged, and just reference the Geometry methods.

Additional comments

HasMargeablePropertiesWith(IElement2D) has not been added for any of the objects in reference to:
#3138 (comment)

[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

[peterjamesnugent]

@BHoMBot check core
@BHoMBot check versioning
@BHoMBot check unit-tests

[bhombot-ci]

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

• check core
• check versioning
• check unit-tests

[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 17 requests in the queue ahead of you.

[bhombot-ci]

The check core 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 versioning 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 check dataset-compliance

[bhombot-ci]

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

• check copyright-compliance
• check dataset-compliance

There are 2 requests in the queue ahead of you.

[peterjamesnugent]

@BHoMBot check ready-to-merge

[bhombot-ci]

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

• check ready-to-merge

[peterjamesnugent]

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

My commits relate to updating the methods because the changes to PadFoundation and Stem making use of IRegion and the versioning required.

[IsakNaslundBh]

Some comments below. Mostly the Volume calculation that is a bit iffy. The Create method comments are not as bad, even if the double check for planarity most likely needs to go.

[peterjamesnugent]

@BHoMBot check versioning
@BHoMBot check unit-tests

[pawelbaran]

@BHoMBot check versioning
@BHoMBot check unit-tests

[pawelbaran]

@BHoMBot check versioning
@BHoMBot check unit-tests

[bhombot-ci]

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

• check versioning
• check unit-tests

[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

[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

[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

[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

[bhombot-ci]

The check code-compliance 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 documentation-compliance has already been run previously and recorded as a successful check. This check has not been run again at this time.

[peterjamesnugent]

@BHoMBot check unit-tests

[bhombot-ci]

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

• check unit-tests

Changes implemented

[IsakNaslundBh]

Happy with the changes made and hence happy to approve this PR.

[bhombot-ci]

@peterjamesnugent 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

[bhombot-ci]

@peterjamesnugent 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

[bhombot-ci]

@peterjamesnugent just to let you know, I have provided a check-ready-to-merge 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 @IsakNaslundBh on BHoM