-
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
DM-45856: Add new verification pipelines for IsrTaskLSST to _ingredients. #56
Conversation
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.
Same kind of questions as on the cp_pipe PR.
connections.outputExposure: "verifyBiasIsrExp" | ||
connections.outputBin1Exposure: "verifyBiasIsrExpBin8" | ||
connections.outputBin2Exposure: "verifyBiasIsrExpBin64" | ||
doBias: true |
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.
Same ordering comment as on cp_pipe
. As-applied would be my preference, but alphabetical at least makes it faster to find a particular option.
doFlat: true | ||
doDiffNonLinearCorrection: false | ||
doBootstrap: false | ||
# TODO: Add deferred charge and brighter-fatter pipelines |
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.
Ticket number?
doVariance: true | ||
doSaturation: false | ||
doSuspect: false | ||
doCrosstalk: true |
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.
Does this need the quadratic crosstalk terms enabled? I think those are disabled by default.
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.
Ooh, good point.
823711e
to
17110a0
Compare
Note that this pipeline needs work/testing in DM-46445.
a9224b2
to
ca46a41
Compare
No description provided.