-
Notifications
You must be signed in to change notification settings - Fork 157
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: Context in bin adjustment and trapezoid surface bin adjustment #966
refactor: Context in bin adjustment and trapezoid surface bin adjustment #966
Conversation
Codecov Report
@@ Coverage Diff @@
## main #966 +/- ##
==========================================
+ Coverage 48.46% 48.48% +0.02%
==========================================
Files 334 334
Lines 17067 17091 +24
Branches 8074 8081 +7
==========================================
+ Hits 8271 8287 +16
- Misses 3094 3099 +5
- Partials 5702 5705 +3
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
The bin adjustment for surface used the default constructor of the geometry context to get the surface transform. This doesn't work for all context so it now uses the geometry context as an input.
This also add bin adjustment for trapezoid bound (and the relevant test).