Skip to content

DM-54482: Integrate PSF-shapelet subtask into calibrateImage as a donut-detector#481

Merged
laurenam merged 2 commits into
mainfrom
tickets/DM-54482
May 12, 2026
Merged

DM-54482: Integrate PSF-shapelet subtask into calibrateImage as a donut-detector#481
laurenam merged 2 commits into
mainfrom
tickets/DM-54482

Conversation

@laurenam
Copy link
Copy Markdown
Contributor

@laurenam laurenam commented May 5, 2026

Checklist

When making changes to YAML files in the schemas directory:

@laurenam laurenam requested a review from TallJimbo May 5, 2026 20:41
@laurenam laurenam force-pushed the tickets/DM-54482 branch 2 times, most recently from 87657a0 to c5ea5d2 Compare May 10, 2026 19:02
@JeremyMcCormick JeremyMcCormick self-requested a review May 11, 2026 16:43
Copy link
Copy Markdown
Collaborator

@JeremyMcCormick JeremyMcCormick left a comment

Choose a reason for hiding this comment

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

I'm putting a hold on this temporarily so these changes can be coordinated with #446.

@JeremyMcCormick JeremyMcCormick self-requested a review May 11, 2026 16:56
@JeremyMcCormick
Copy link
Copy Markdown
Collaborator

JeremyMcCormick commented May 11, 2026

Okay, I looked over these changes, and this PR can go in before #446.

Please verify with a Jenkins run before merging as this changes lsstcam.yaml and apdb.yaml which could potentially affect the pipelines build and tests.

laurenam added 2 commits May 11, 2026 12:34
Including lsstcam.yaml, hsc.yaml, imsim.yaml, and apdb.yaml.

These parameters provide image quality assessment, the main one being
the shapeletsScore, which spans values [0.0, 1.0], with lower values
indicating better image quality.
@laurenam laurenam force-pushed the tickets/DM-54482 branch from c5ea5d2 to 58ed15e Compare May 11, 2026 19:34
@laurenam laurenam merged commit 67af733 into main May 12, 2026
18 checks passed
@laurenam laurenam deleted the tickets/DM-54482 branch May 12, 2026 18:44
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.

3 participants