Ground_Engine: Consolidate method for a Borehole

[peterjamesnugent]

Issues addressed by this PR

Closes #3271

Test files

https://burohappold.sharepoint.com/:u:/s/BHoM/Ed88wg7PXYJEmTkOJcxKgkgBDI2rWoJoVNS3sJPTk4iwJw?e=24SqQ2

Changelog

• Added ConsolidateStrata method that uses a property to combine sequential Stratum. For example, combining Stratum with the same ObservedGeology property.

Additional comments

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

[jtrainor1]

Component manages to combine the AGS data correctly.

[peterjamesnugent]

@BHoMBot check unit-tests

[bhombot-ci]

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

• check unit-tests

[pawelbaran]

@BHoMBot check unit-tests

[bhombot-ci]

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

• check unit-tests

[pawelbaran]

@BHoMBot check copyright-compliance

[bhombot-ci]

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

• check copyright-compliance

[pawelbaran]

@BHoMBot check unit-tests

[bhombot-ci]

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

• check unit-tests

[pawelbaran]

@BHoMBot check unit-tests

[bhombot-ci]

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

• check unit-tests

[pawelbaran]

@BHoMBot check unit-tests

[bhombot-ci]

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

• check unit-tests

[pawelbaran]

@BHoMBot check unit-tests

[bhombot-ci]

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

• check unit-tests

[pawelbaran]

@BHoMBot check compliance

[bhombot-ci]

@pawelbaran 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.

[pawelbaran]

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

[bhombot-ci]

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

• check core
• check null-handling
• check serialisation

There are 3 requests in the queue ahead of you.

[pawelbaran]

@BHoMBot check installer
@BHoMBot check versioning

[bhombot-ci]

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

• check installer
• check versioning

There are 2 requests in the queue ahead of you.

[bhombot-ci]

FAO: @FraserGreenroyd
@pawelbaran is seeking dispensation on this Pull Request to skip a required check. Please can you provide authorisation for the check to be skipped, or provide assistance as appropriate.

The check they wish to have dispensation on is unit-tests.

If you are providing dispensation on this occasion, please reply with:

@BHoMBot this is a DevOps instruction. I am authorising dispensation to be granted on check ref. 27566220787

[pawelbaran]

Dispensating the unit test check as this PR does not introduce any new failures compared to develop (e.g. here).

[pawelbaran]

Re-approving on the basis of previous approval by @jtrainor1 - the code has only been rebased since then.

[pawelbaran]

@BHoMBot this is a DevOps instruction. I am authorising dispensation to be granted on check ref. 27566220787

[bhombot-ci]

@pawelbaran I have now provided a passing check on reference 27566220787 as requested.

[pawelbaran]

@BHoMBot check ready-to-merge

[bhombot-ci]

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

• check ready-to-merge