-
Notifications
You must be signed in to change notification settings - Fork 14
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
DM-23983: Cannot apply crosstalk in Gen 3 DECam processing #140
Merged
Commits on Jun 1, 2020
-
Add CrosstalkCalib subclass of IsrCalib.
This new implementation supports both intra-chip crosstalk via the CrosstalkCalib.coeffs attribute and inter-chip crosstalk via the CrosstalkCalib.interChip dictionary. Generic methods have been moved into the calibration, and the CrosstalkTask.run updated to work with the new calibration and to apply the inter-chip crosstalk if it exists.
Configuration menu - View commit details
-
Copy full SHA for a54e212 - Browse repository at this point
Copy the full SHA a54e212View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5e7e9c1 - Browse repository at this point
Copy the full SHA 5e7e9c1View commit details -
Fix consequences of new CrosstalkCalib.
Simplify IsrMock as the calibration handles amplifier flips. Update measureCrosstalk to persist output as CrosstalkCalib, and fix broken unit tests.
Configuration menu - View commit details
-
Copy full SHA for 0b27906 - Browse repository at this point
Copy the full SHA 0b27906View commit details -
Add gen3 handler for crosstalkSources.
Update output type to a generic Exposure needed for running pipelines that split ISR processing into separate task calls (such as DECam ISR with Crosstalk).
Configuration menu - View commit details
-
Copy full SHA for a95332d - Browse repository at this point
Copy the full SHA a95332dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 31ec924 - Browse repository at this point
Copy the full SHA 31ec924View commit details -
Configuration menu - View commit details
-
Copy full SHA for 744d342 - Browse repository at this point
Copy the full SHA 744d342View commit details -
Configuration menu - View commit details
-
Copy full SHA for ced29bd - Browse repository at this point
Copy the full SHA ced29bdView commit details -
Configuration menu - View commit details
-
Copy full SHA for 3f2f06f - Browse repository at this point
Copy the full SHA 3f2f06fView commit details
Commits on Jun 4, 2020
-
Configuration menu - View commit details
-
Copy full SHA for 8deb4f6 - Browse repository at this point
Copy the full SHA 8deb4f6View commit details
Commits on Jun 9, 2020
-
Add comments on crosstalkSources issue.
DM-25348 will resolve this problem.
Configuration menu - View commit details
-
Copy full SHA for 5b9bb1a - Browse repository at this point
Copy the full SHA 5b9bb1aView commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.