Skip to content

DM-55738: Update PromptTemplate and QuickTemplate for incremental Rubin and DECam template development - #287

Merged
erinleighh merged 2 commits into
mainfrom
tickets/DM-55738
Aug 15, 2026
Merged

DM-55738: Update PromptTemplate and QuickTemplate for incremental Rubin and DECam template development#287
erinleighh merged 2 commits into
mainfrom
tickets/DM-55738

Conversation

@erinleighh

Copy link
Copy Markdown
Contributor

No description provided.

@erinleighh
erinleighh requested a review from mrawls August 6, 2026 08:17

@mrawls mrawls left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Once these comments are addressed I think the pipelines will have the desired configurations - remains to be seen if they actually create templates we like 😄

Comment thread pipelines/_ingredients/QuickTemplate.yaml Outdated
Comment thread pipelines/_ingredients/QuickTemplate.yaml Outdated
Comment thread pipelines/_ingredients/QuickTemplate.yaml
consolidateVisitSummary:
class: lsst.pipe.tasks.postprocess.ConsolidateVisitSummaryTask
config:
full: True

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Consider adding a comment for why this override is needed (because we're starting with preliminary_visit_images?)

Comment thread pipelines/LSSTCam/PromptTemplate.yaml
@erinleighh erinleighh changed the title DM-55738: Update PromptTemplate to inherit from QuickTemplate DM-55738: Update PromptTemplate and QuickTemplate for incremental Rubin and DECam template development Aug 14, 2026
@erinleighh
erinleighh merged commit e5e450d into main Aug 15, 2026
2 checks passed
@erinleighh
erinleighh deleted the tickets/DM-55738 branch August 15, 2026 03:16
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