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
Fix bitrot for using new style brighter-fatter kernels #120
Conversation
0ed9f9e
to
a0b22c0
Compare
1de0e16
to
2d36927
Compare
2d36927
to
fa3be3f
Compare
elif brighterFatterKernel.detectorKernelFromAmpKernels: | ||
brighterFatterKernel = brighterFatterKernel.detectorKernelFromAmpKernels[ccd.getId()] | ||
else: | ||
raise RuntimeError("Failed to extract kernel from new-style BF kernel.") | ||
else: | ||
# TODO DM-15631 for implementing this | ||
raise NotImplementedError("Per-amplifier brighter-fatter correction not implemented") |
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 you duplicate this logic into the gen3 runQuantum() method (line isrTask.py:827)? The kernel object should be supplied as inputs['bfKernel'], and can be overwritten in place by the appropriate detectorKernel (the linearizer gets and uses a detector as well). Adding the inputs['bfGains'] entry should ensure this works cleanly in that case.
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 was about to try, but I have no way of testing what I've done as I don't have any Gen3 repos (or even knowledge yet), and combined with no experience in such things, I think it's best left. I'll ticket, leave a TODO:
and assign it to you. Sorry and thanks!
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.
DM-22776
fa3be3f
to
3822d18
Compare
Remove unused config option for specifying files