Skip to content
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

Use new cp_pipe pipelines and new IsrTaskLSST #96

Open
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

jchiang87
Copy link
Contributor

@jchiang87 jchiang87 commented Oct 9, 2024

Summary of changes:

  • Use the new sequence of cp_pipe pipelines according to DM-45856
    • Add [cpBias,cpDark,cpFlat]Bootstrap versions
    • Add cpLinearizer pipeline as a precursor to cpPtc
    • Switch to standard cpDefects pipeline (which now builds combined darks and flats)
    • Use default cp_pipe ISR configs, including doCrosstalk=True, doLinearize=True where relevant.
    • Omit cpBfk since the BF kernels aren't used by eo_pipe
    • Omit mosaicking steps in cpBias, cpDark, cpFlat
  • Integrate IsrTaskLSST into bias stability code and replace IsrTask steps in various eo_pipe pipelines
  • Use linearizerPtc as input to eoLinearityPlots pipeline
  • Use cp_pipe combined darks and flats as inputs to eo[Bright,Dark]Defects pipelines instead of building combined frames in the eo_pipe pipelines.

Because of the cp_pipe re-ordering, in the runtime environment set up, the top-level input collection to the eo_pipe pipelines will need to be

u/${USER}/flat${PAYLOAD_MODIFIER}_${PTC_RUN}_${WEEKLY}

for PTC runs and

u/${USER}/flat${PAYLOAD_MODIFIER}_${B_PROTOCOL_RUN}_${WEEKLY}

for B-protocol runs, instead of

u/${USER}/ptc${PAYLOAD_MODIFIER}_${PTC_RUN}_${WEEKLY}

which could be used for all runs.

@jchiang87 jchiang87 force-pushed the u/jchiang/isrTaskLSST branch 2 times, most recently from e899cef to 58c06f0 Compare October 24, 2024 05:14
@jchiang87 jchiang87 force-pushed the u/jchiang/isrTaskLSST branch 2 times, most recently from 97d702c to e45d6d3 Compare November 12, 2024 20:52
@jchiang87 jchiang87 force-pushed the u/jchiang/isrTaskLSST branch from e45d6d3 to af6eb96 Compare November 16, 2024 04:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant