Geometry_Engine: numerical solution in IsCoplanar replaced with geometrical one

[pawelbaran]

Issues addressed by this PR

Closes #2423 Closes #3294

Test files

General test files for the methods changed in this PR uploaded to SharePoint Test file for https://github.com/BHoM/BHoM_Engine/issues/2423 linked in the issue

Changelog

Additional comments

This is the last PR in the current push in Geometry_Engine 🙈

[pawelbaran]

@BHoMBot check compliance

[bhombot-ci[bot]]

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

[pawelbaran]

@BHoMBot check required

[bhombot-ci[bot]]

@pawelbaran 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[bot]]

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

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.

[pawelbaran]

Thanks @peterjamesnugent for the review. Updating the UTs is on my radar, to be done in sprint 6 based on https://github.com/BHoM/BHoM_Engine/issues/3277 👍

[FraserGreenroyd]

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

[bhombot-ci[bot]]

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

@FraserGreenroyd to confirm, the following actions are now queued: - check `ready-to-merge`