Skip to content

DM-53324: Update ConversionsForFakes cf. new make_injection_pipeline outputs#257

Merged
leeskelvin merged 1 commit into
mainfrom
tickets/DM-53324
May 12, 2026
Merged

DM-53324: Update ConversionsForFakes cf. new make_injection_pipeline outputs#257
leeskelvin merged 1 commit into
mainfrom
tickets/DM-53324

Conversation

@leeskelvin

@leeskelvin leeskelvin commented Apr 30, 2026

Copy link
Copy Markdown
Contributor

{Summary of changes. Prefix PR title with JIRA issue.}


  • Do unit tests pass (scons and/or stack-os-matrix)?
  • Did you run ap_verify.py on at least one of the standard datasets?
    For changes to metrics, the print_metricvalues script from lsst.verify will be useful.
  • Is the Sphinx documentation up-to-date?

@BrunoSanchez BrunoSanchez left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Lee, I see the changes, they look good to me. I agree, this is what RFC-1171 is about.

On AP we will need to start tagging collections with fakes and making sure this is not mixed. That's all. Approved.

@leeskelvin leeskelvin merged commit cd4e159 into main May 12, 2026
3 checks passed
@leeskelvin leeskelvin deleted the tickets/DM-53324 branch May 12, 2026 15:07
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