-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add bad pixel flags to extra.calibration #172
Conversation
docs/calibration.md
Outdated
The calibration pipeline produces masks along with corrected data, in keys | ||
called `image.mask` or `data.mask` depending on the detector. Zeros in the mask | ||
represent good pixels, while any other value indicates one or more reasons to | ||
consider the corresponding data dubious: |
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.
I would prefer as neutral wording as possible, given some bits simply mean to inform about a peculiarity like NON_STANDARD_SIZE
. Maybe conspicuous? (just grabbing that from a dictionary)
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.
Good point, I've had a go at rewording this.
Are you happy with the subheading remaining 'Bad pixel values' if we give more nuance in the description, or do you think that needs to change too?
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.
I suspect that term is just too established to change now 🤷
That should help, thanks. LGTM. |
The definitions are copied from pycalibration.