-
Notifications
You must be signed in to change notification settings - Fork 10
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
AutoShotWrap grid override #338
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #338 +/- ##
==========================================
+ Coverage 85.30% 86.59% +1.28%
==========================================
Files 46 48 +2
Lines 2219 2440 +221
Branches 306 353 +47
==========================================
+ Hits 1893 2113 +220
+ Misses 281 274 -7
- Partials 45 53 +8 ☔ View full report in Codecov by Sentry. |
Hi @markspec, can we add at least 1 test for each configuration? |
Closes #336 |
@tasansal , please can you look at this?
|
@markspec approved but there's a test conflict; please fix so we can merge. thanks! |
…xed by unwrapping shot_point for shot_lines.
71cb19d
to
5c14ffb
Compare
This allows ingestion with gun indexed by unwrapping shot_point for shot_lines.