-
Notifications
You must be signed in to change notification settings - Fork 99
2127 flat, dark, background in pyFAI-calib2 [PROPOSAL] #2143
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
Conversation
for more information, see https://pre-commit.ci
… into 2127_flat_and_dark
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
Now there is an instance of a PreProcessedImageModel, which takes into account the mask, the flat and the dark Most likely, there are still things to wire if the image is changed during the tasks |
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.
Can't this be replaced by preprocessedImageModel ?
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.
correct, DarkedMaskedImageModel.py is no longer needed
Is this still WIP ? |
Yes, I will test the change of the image and I will label Ready to merge |
now, flat and dark images can be uploaded through CLI, dialog file and drag&drop, and they will be properly computed along all the calib2 tasks |
@jonwright are you willing to test it ? |
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.
LGTM
flat.changed.connect(self.__invalidateValue) | ||
self.__value = None | ||
|
||
def __computeImageData(self): |
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.
Is this cached anywhere ?
Sure - we will try this later today or during the weekend - sometime soon after we figure out how to install pyFAI from a pull request. |
you can use `./bootstrap pyFAI-calib2` to test locally without installing.
You may install all dependencies before-hand with a a "pip install -r requirements.txt --upgrade"
|
Proposal for including dark current and flat field images in pyFAI-calib2
Idea:
darked_masked_data = Masking(data_array) - dark_current_array
corrected_image = (darked_masked_image_data - darked_masked_background_data) / flat_field
inspired by: https://lookingatnothing.com/index.php/archives/2251