Skip to content

DM-56011: Update template pipelines for cell based coadds. - #293

Merged
erinleighh merged 2 commits into
mainfrom
tickets/DM-56011
Sep 29, 2026
Merged

erinleighh merged 2 commits into
mainfrom
tickets/DM-56011

Conversation

@erinleighh

Copy link
Copy Markdown
Contributor

No description provided.

@erinleighh
erinleighh marked this pull request as ready for review September 29, 2026 17:11

@isullivan isullivan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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, but bps_PromptTemplate.yaml still refers to it. That should be simple to point to clustering_QuickTemplate.yaml.
  • See my comment about changing to full: False in consolidateVisitSummary. It needs a test run to verify that everything still works without that, but I think it should.

Comment thread pipelines/LSSTCam/PromptTemplate.yaml Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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!

@erinleighh
erinleighh merged commit 67c9c00 into main Sep 29, 2026
2 checks passed
@erinleighh
erinleighh deleted the tickets/DM-56011 branch September 29, 2026 22:31
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.

2 participants