Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DM-32430: Create DECam DRP Processing Pipeline #206

Merged
merged 1 commit into from Dec 9, 2021
Merged

Conversation

leeskelvin
Copy link
Contributor

During construction of the DECam DRP pipeline, color terms were added.
This modified some of the numbers being tested against in jointcal. This
ticket updates these empirical numbers to match the new outputs.

During construction of the DECam DRP pipeline, color terms were added.
This modified some of the numbers being tested against in jointcal. This
ticket updates these empirical numbers to match the new outputs.
Copy link

@mrawls mrawls left a comment

Choose a reason for hiding this comment

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

Since John P isn't concerned about these small numeric changes, I'm not either. Indeed, color terms probably are inching us closer to reality.

@parejkoj
Copy link
Collaborator

parejkoj commented Dec 8, 2021

Copying my comment from Slack: these new numbers are reasonable. The number of reference stars went down (some must have been removed when calculating the colorterms because they didn't have enough information?), the number of total matched stars went down a little because of that, and the final chi2 was improved and fewer stars were removed as outliers (because the fluxes were more correct).

@leeskelvin leeskelvin merged commit ca1ce8f into main Dec 9, 2021
@leeskelvin leeskelvin deleted the tickets/DM-32430 branch December 9, 2021 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants