Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR reorganizes the codebase by moving functionality from the utils module into two new modules: math and geometry. This improves code organization by better separating mathematical utilities from geometric operations.
Key changes:
- Created new
geometrymodule containing area, angle, intersection, and normal functions - Created new
mathmodule containing interval arithmetic and mathematical helper functions - Moved files from
ipc/utils/to appropriate new modules with corresponding Python bindings - Updated all import paths throughout the codebase to reflect the new module structure
Reviewed Changes
Copilot reviewed 37 out of 44 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| src/ipc/math/* | New math module with interval arithmetic and mathematical utilities (formerly in utils) |
| src/ipc/geometry/* | New geometry module with area, angle, intersection, and normal computations |
| tests/src/tests/math/, tests/src/tests/geometry/ | New test directories matching the new module structure |
| src/ipc/utils/CMakeLists.txt | Removed moved files from utils module build |
| python/src/geometry/, python/src/math/ | Python bindings for new modules |
| Various cpp/hpp files | Updated include paths from ipc/utils/ to ipc/math/ or ipc/geometry/ |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #198 +/- ##
=======================================
Coverage 97.39% 97.40%
=======================================
Files 147 149 +2
Lines 23647 23696 +49
Branches 795 796 +1
=======================================
+ Hits 23031 23081 +50
+ Misses 616 615 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Description
Organize geometry and math utilities.
Type of change