Add Tesselation sample_boundary Warp implementation

[Alexey-Kamenev]

PhysicsNeMo Pull Request

Description

Added Tesselation sample_boundary Warp implementation.

Checklist

• I am familiar with the Contributing Guidelines.
• New or existing tests cover these changes.
• The documentation is up to date with these changes.
• The CHANGELOG.md is up to date with these changes.
• An issue is linked to this pull request.

Dependencies

[Alexey-Kamenev]

/blossom-ci

[Alexey-Kamenev]

/blossom-ci

[ktangsali]

This is fantastic! Thanks!

A few questions. Have you verified that the sample_interior and sample_boundary work together? Additionally, it would be great if we documented these additions better.

API Docs: https://github.com/NVIDIA/physicsnemo-sym/blob/main/docs/api/physicsnemo.sym.geometry.rst#geometrytessellation

Features Docs: https://github.com/NVIDIA/physicsnemo-sym/blob/main/docs/user_guide/features/csg_and_tessellated_module.rst#tesselated-geometry

Additionally, would be great to add some speed comparisons. If I understand correctly, the default is not being changed in this PR right?

[ktangsali]

This is fantastic! Thanks!

A few questions. Have you verified that the sample_interior and sample_boundary work together? Additionally, it would be great if we documented these additions better.

API Docs: https://github.com/NVIDIA/physicsnemo-sym/blob/main/docs/api/physicsnemo.sym.geometry.rst#geometrytessellation

Features Docs: https://github.com/NVIDIA/physicsnemo-sym/blob/main/docs/user_guide/features/csg_and_tessellated_module.rst#tesselated-geometry

Additionally, would be great to add some speed comparisons. If I understand correctly, the default is not being changed in this PR right?

A few speed comparisons would be the cherry on the cake.

[Alexey-Kamenev]

All great comments, thank you!

A few questions. Have you verified that the sample_interior and sample_boundary work together?

Yes! I've confirmed that sample_interior still works with the Warp Tesselator.
I verified it both visually and by comparing the results with the default Tesselator.

Additionally, it would be great if we documented these additions better.
API Docs: https://github.com/NVIDIA/physicsnemo-sym/blob/main/docs/api/physicsnemo.sym.geometry.rst#geometrytessellation

Done.

Features Docs: https://github.com/NVIDIA/physicsnemo-sym/blob/main/docs/user_guide/features/csg_and_tessellated_module.rst#tesselated-geometry
Additionally, would be great to add some speed comparisons. If I understand correctly, the default is not being changed in this PR right?
A few speed comparisons would be the cherry on the cake.

That is correct, the default is not changed in this PR.
I've added documentation and performance comparison table.

[Alexey-Kamenev]

/blossom-ci