Fix PlanarSurface conversion bug

[vietle-bh]

NOTE: Depends on

Issues addressed by this PR

Closes #1499

Instead of comparing objects, we can detect internal openings by their bounding boxes, which should be always smaller than the main external boundary's bounding box.

Test files

TBC

Changelog

Additional comments

[pawelbaran]

Instead of comparing objects, we can detect internal openings by their loop length, which is always shorter than the main external boundary.

Are you sure? 😉

[vietle-bh]

@pawelbaran : Not anymore! I've changed to comparing bounding boxes instead. Please check again 🥂

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

[vietle-bh]

@BHoMBot check copyright-compliance @BHoMBot check dataset-compliance @BHoMBot check ready-to-merge

[bhombot-ci[bot]]

@vietle-bh to confirm, the following actions are now queued: - check `copyright-compliance` - check `dataset-compliance` - check `ready-to-merge`

[pawelbaran]

@BHoMBot check ready-to-merge

[bhombot-ci[bot]]

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