Physical_Engine: Add support for GeneralMaterialTakeoff

[IsakNaslundBh]

NOTE: Depends on

BHoM/BHoM#1516

Issues addressed by this PR

• Adds first support methods and collection system for GeneralMaterialTakeoff class
• Add Aggregation and mapping support for GeneralMaterialTakeoff
• Add GeneralTakeoff capability for physical surface obejcts
• Add method to generate a general takeoff given a construction and an area.

Test files

https://burohappold.sharepoint.com/:f:/r/sites/BHoM/02_Current/12_Scripts/02_Pull%20Request/BHoM/BHoM/Physical_oM/%231516-AddGeneralTakeoffObject?csf=1&web=1&e=iAsS7e

Changelog

Additional comments

[IsakNaslundBh]

@BHoMBot check compliance

[bhombot-ci]

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

[michaelhoehn]

I've tested this with the provided sample script and everything works as expected. Nicely done @IsakNaslundBh 👍

[IsakNaslundBh]

@BHoMBot check compliance
@BHoMBot check unit-tests
@BHoMBot check required

[bhombot-ci]

@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
• check unit-tests
• check code-compliance
• check documentation-compliance
• check project-compliance
• check core
• check null-handling
• check serialisation
• check versioning
• check installer

There are 2 requests in the queue ahead of you.

[IsakNaslundBh]

@BHoMBot check unit-tests
@BHoMBot check code-compliance
@BHoMBot check copyright-compliance
@BHoMBot check documentation-compliance
@BHoMBot check project-compliance
@BHoMBot check core
@BHoMBot check null-handling
@BHoMBot check serialisation

[bhombot-ci]

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

• check unit-tests
• check code-compliance
• check copyright-compliance
• check documentation-compliance
• check project-compliance
• check core
• check null-handling
• check serialisation

There are 29 requests in the queue ahead of you.

[alelom]

Approving 615ffc4 and 675fe72 based on code changes and the previous approving review.

[bhombot-ci]

The check unit-tests 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 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 copyright-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.

[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 null-handling 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 serialisation has already been run previously and recorded as a successful check. This check has not been run again at this time.

[FraserGreenroyd]

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

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

@BHoMBot check ready-to-merge

[bhombot-ci]

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

• check ready-to-merge