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-45192: Reconfigure RA for real ComCam #143

Merged
merged 7 commits into from
Aug 7, 2024
Merged

Conversation

mfisherlevine
Copy link
Contributor

No description provided.

nVisitsMin: 12
# For simulation 2 there are 60 visits per region
# between 60399.362 and 60402.419.
minMJD: 60399.362
Copy link
Contributor

Choose a reason for hiding this comment

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

Delete OR-specific minMJD and maxMJD overrides.

pipelines/_ingredients/LSSTComCam/DRP.yaml Outdated Show resolved Hide resolved
class: lsst.analysis.tools.tasks.photometricCatalogMatch.PhotometricCatalogMatchVisitTask
config:
extraColumns: ["x", "y", "ap09Flux", "ap09FluxErr"]
connections.refCatalog: uw_stars_20240524
Copy link
Contributor

Choose a reason for hiding this comment

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

All these refcats will need to change. You can delete any lines with "uw_stars" in them or leave them, but they'll need to be revisited on the next ticket.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah yes, good point, thanks. I've just done a find & replace on all those to change them to gaia_dr3_20230707, because that's at least a little less wrong - I think that's probably right for the astrometry bits, and means that we should then only need to change the photometry parts. Hopefully that's better than leaving them, anyway.

pipelines/_ingredients/LSSTComCam/DRP.yaml Outdated Show resolved Hide resolved
@mfisherlevine mfisherlevine merged commit c80fa98 into main Aug 7, 2024
3 checks passed
@mfisherlevine mfisherlevine deleted the tickets/DM-45192 branch August 7, 2024 12:05
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