Structure_Engine: Add engine methods for foundation classes

[peterjamesnugent]

NOTE: Depends on

BHoM/BHoM#1545

Issues addressed by this PR

Closes #3137

Test files

https://burohappold.sharepoint.com/:u:/r/sites/BHoM/02_Current/12_Scripts/01_Issue/BHoM/BHoM/Structure_oM/%231543-FoundationClasses/Foundation%20classes%20and%20associated%20methods.gh?csf=1&web=1&e=Q17tVv

Changelog

• Added necessary methods for PadFoundation and PileFoundation to work with IElement2D and IElementM respectively;
• Added Create methods for PadFoundation, PileFoundation and PileGroup
• Updated IsNull methods to include new foundation objects;
• Added GrahamScan method to determine the perimeter of a List<Point> which is used in the Create methods for a PileFoundation.

Additional comments

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

[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-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

[pawelbaran]

Good stuff 👍 but same as in the oM, I left quite a few comments - hopefully they will be more help than distraction.

[pawelbaran]

Btw, you may need a dedicated Geometry method for PileFoundation - easy to verify by checking if the objects have preview in GH

[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]

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

[peterjamesnugent]

@BHoMBot check unit-tests
@BHoMBot check copyright-compliance
@BHoMBot check dataset-compliance

[bhombot-ci]

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

• check unit-tests
• check copyright-compliance
• check dataset-compliance

There are 9 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

[peterjamesnugent]

@BHoMBot check copyright-compliance
@BHoMBot check unit-tests
@BHoMBot check dataset-compliance

[bhombot-ci]

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

• check copyright-compliance
• check unit-tests
• check dataset-compliance

There are 8 requests in the queue ahead of you.

[FraserGreenroyd]

@BHoMBot this is a DevOps instruction. I am requesting neutral checks on: unit-tests

[bhombot-ci]

@FraserGreenroyd I have provided neutral checks to the checks requested. These checks will need to be run properly to obtain full results.

[FraserGreenroyd]

Approving based on @Chrisshort92 review - following commits by @peterjamesnugent have been compliance updates.

[FraserGreenroyd]

@BHoMBot check ready-to-merge

[bhombot-ci]

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

• check ready-to-merge

[FraserGreenroyd]

@BHoMBot this is a DevOps instruction. I am requesting neutral checks on: ready-to-merge

[bhombot-ci]

@FraserGreenroyd I have provided neutral checks to the checks requested. These checks will need to be run properly to obtain full results.