-
Notifications
You must be signed in to change notification settings - Fork 46
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
deltacchalf filtering: allow image_group mode on single data set #1334
Conversation
It works, and seems to be a valid thing to do, so don't prohibit it. Only catch whole data set deltacchalf mode, which won't work on a single data set (obviously).
Codecov Report
@@ Coverage Diff @@
## master #1334 +/- ##
==========================================
+ Coverage 64.19% 64.22% +0.02%
==========================================
Files 617 617
Lines 69784 69797 +13
Branches 9557 9557
==========================================
+ Hits 44801 44827 +26
+ Misses 23215 23207 -8
+ Partials 1768 1763 -5
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.
Thanks for sorting this out, looks good.
This wants linking to #1161 when merging in.
It works, and seems to be a valid thing to do, so don't prohibit it. Only catch whole data set deltacchalf mode, which won't work on a single data set (obviously). Fixes #1161
It works, and seems to be a valid thing to do, so don't prohibit it. Only catch whole data set deltacchalf mode, which won't work on a single data set (obviously). Fixes #1161
- `dials.scale`: Allow usage of `mode=image_group` with `filtering.method=deltacchalf` when only providing a single data set (#1334) - `dials.import`: When using a template and specifying an image_range, missing images outside of the range will not cause a failure (#1333) - `dials.stills_process`: Show better error message in specific spotfinding failure case (#1180)
It works, and seems to be a valid thing to do, so don't prohibit it. Only catch whole data set deltacchalf mode, which won't work on a single data set (obviously).
Fixes #1161