Environment_Engine: Added ElementsInSpace and dependant IsContaining methods / optimisations

[enarhi]

Issues addressed by this PR

Closes #3194

Added ElementsInSpace for Element0D (1D/2D to follow as a latter PR). Also added IsContaining method for Element0D as well as faster method for detecting list of points' containment in a single space.

Test files

Test Script

Test by checking that the method correctly identifies the elements in each space as per the different visualization output options in dark blue to the script's far right.

[enarhi]

@BHoMBot check compliance

[bhombot-ci]

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

[FraserGreenroyd]

Following discussion with @enarhi and @vietle-bh just now:

• This PR to be accepted as is with formatting changes suggested - and the whitespace at the bottom of IsContaining.cs to be removed too please @enarhi
• New issue to refactor IsContaining from a List<Panel> to a List<ICurve>
• New issue to then refactor IsContaining out from Environment_Engine and into Spatial_Engine
• New issue to then refactor Environment_Engine methods to utilise Spatial_Engine methods as appropriate

[enarhi]

@BHoMBot check compliance

[bhombot-ci]

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

[enarhi]

@BHoMBot check versioning

[bhombot-ci]

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

• check versioning

There are 6 requests in the queue ahead of you.

[FraserGreenroyd]

Looks good now following our chat earlier, thanks @enarhi

[FraserGreenroyd]

@enarhi can I leave raising the subsequent issues with you?

[enarhi]

@enarhi can I leave raising the subsequent issues with you?

Yes, I'll add those.

[FraserGreenroyd]

@enarhi can I leave raising the subsequent issues with you?

Yes, I'll add those.

Tack så mycket 👍

[FraserGreenroyd]

@BHoMBot check installer

[bhombot-ci]

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

• check installer

[enarhi]

@BHoMBot check required

[bhombot-ci]

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

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

[enarhi]

@BHoMBot check ready-to-merge

[bhombot-ci]

@enarhi 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: 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.