-
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
How often to run auto mask #168
Comments
I don't think that this is too magical: We are consistent when we produce a new mask (every run of xrun, or potentially every non-setup run of xrun if this issue is accepted). I understand the time based concerns with setup scans, they exist to check if the data is reasonable before taking more "production" quality shots. Making the mask generation dynamic could be rather difficult to implement, since once the data processing pipeline is running there is no current way to change it's parameters (although we may be able to put something into the experimental metadata?) |
In discussion with @sbillinge: |
I'm moving this discussion here as the prior issue was really about implementing a feature not a more in depth discussion of masking.
@sbillinge :I am still torn on this masking issue. I would actually like dynamic vs. static masking to be a user-selectable preference even for non-setup runs. Speed is one issue, but also users getting consistent results and there not being too much "magic" in the data analysis pipeline that they can't explain in their papers. From this perspective, I am much more comfortable with a workflow where a mask is created and then re-used for a "set" of measurements and somehow stored. I can then reproduce results exactly by using the same mask, and then deliberately change the mask if I am not happy with the results.
I am not against offering dynamic masking to users as an option, but I am still uncomfortable with having it as the default.
We could give it similar behavior to dark correction, where if it can't find a "fresh" mask it will generate a dynamic one.
The text was updated successfully, but these errors were encountered: