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

Update defect finding task to add metadata #25

Merged
merged 3 commits into from Feb 14, 2020
Merged

Conversation

mfisherlevine
Copy link
Collaborator

Update the defect finding task to add all metadata necessary for ingest task to use the newly generated defects.

python/lsst/cp/pipe/defects.py Outdated Show resolved Hide resolved
python/lsst/cp/pipe/defects.py Show resolved Hide resolved
python/lsst/cp/pipe/defects.py Show resolved Hide resolved
python/lsst/cp/pipe/defects.py Outdated Show resolved Hide resolved
Copy link
Member

@timj timj left a comment

Choose a reason for hiding this comment

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

I have questions but I've also got a couple of things that need to be cleaned up.

python/lsst/cp/pipe/defects.py Outdated Show resolved Hide resolved
python/lsst/cp/pipe/defects.py Outdated Show resolved Hide resolved
python/lsst/cp/pipe/defects.py Outdated Show resolved Hide resolved
python/lsst/cp/pipe/defects.py Outdated Show resolved Hide resolved
python/lsst/cp/pipe/defects.py Show resolved Hide resolved
python/lsst/cp/pipe/defects.py Show resolved Hide resolved
python/lsst/cp/pipe/defects.py Show resolved Hide resolved
@mfisherlevine mfisherlevine force-pushed the tickets/DM-23448 branch 5 times, most recently from ee394ad to c1d509b Compare February 14, 2020 00:04
Also modify path so it is compatible with the ingest code
Add option to ignore filters in flats
@mfisherlevine mfisherlevine merged commit da2ac89 into master Feb 14, 2020
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.

None yet

3 participants