DM-56011: Update template pipelines for cell based coadds. - #293
Conversation
b198d39 to
9b04028
Compare
isullivan
left a comment
There was a problem hiding this comment.
We will need to make a few changes to the connections once lsst.images is in use, but none of the required tickets are likely to merge in the next few days so I wouldn't hold this up on those changes.
Two changes I would like to see on this ticket:
- this removes
clustering_PromptTemplate.yaml, butbps_PromptTemplate.yamlstill refers to it. That should be simple to point toclustering_QuickTemplate.yaml. - See my comment about changing to
full: FalseinconsolidateVisitSummary. It needs a test run to verify that everything still works without that, but I think it should.
There was a problem hiding this comment.
It's not part of your current changes, but please see if you can change this to full: False. I don't think anything uses the components loaded with full: True, which is intended to store things like the PSFs, transmission curves, and aperture correction. It uses ~10 more disk space, and doesn't work with future mode (because the transmission curve is not converted yet).
There was a problem hiding this comment.
We had to change this when we first made the pipeline because it threw an error otherwise, but it's been a while so I'll check and see if we can easily change it! Thanks!
9b04028 to
f85b67b
Compare
No description provided.