-
Notifications
You must be signed in to change notification settings - Fork 48
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
Respect the bounding_box in inverse transforms #498
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #498 +/- ##
==========================================
- Coverage 87.28% 87.04% -0.24%
==========================================
Files 22 21 -1
Lines 3821 3891 +70
==========================================
+ Hits 3335 3387 +52
- Misses 486 504 +18 ☔ View full report in Codecov by Sentry. |
@bmorris3 Do you mind testing this PR with your code? |
@Cadair Can you take a look at this PR and see if it's OK for sun related code? |
Fixes #496
This is a fix similar in spirit to #497 though it uses methods of
modeling.ModelBoundingBox
.