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-15514: Write insertFakeObjectsTask to add fake sources for QA purposes #134

Merged
merged 1 commit into from Jun 6, 2019

Conversation

sr525
Copy link
Contributor

@sr525 sr525 commented Feb 21, 2019

No description provided.

@@ -564,6 +564,25 @@ src:
python: lsst.afw.table.SourceCatalog
tables: raw
template: ''
fakeSourceCat:
description: "A calexp sized catalog of fake sources"
Copy link
Contributor

Choose a reason for hiding this comment

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

What does "calexp sized catalog" mean, is is somehow the size of a calexp? is it a calexp, sized catalog? Put in a more detailed description

tables: raw
template: ''
deepCoadd_fakeSourceCat:
description: "A deepCoadd_calexp sized catalog of fake sources"
Copy link
Contributor

Choose a reason for hiding this comment

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

same as above

@@ -5,6 +5,13 @@ deepCoadd:
python: lsst.afw.image.ExposureF
template: deepCoadd/%(filter)s/%(tract)d/%(patch)s.fits
level: Skytile
fakes_deepCoadd:
description: "A coadd assembled from direct warps, produced by AssembleCoaddTask, with fakes added."
Copy link
Contributor

Choose a reason for hiding this comment

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

fakes => fake objects, breaks degeneracy between fake objects inserted in the warps and having fake warps included in the coadd

@@ -28,6 +35,13 @@ deepCoadd_calexp:
python: lsst.afw.image.ExposureF
template: deepCoadd-results/%(filter)s/%(tract)d/%(patch)s/calexp-%(filter)s-%(tract)d-%(patch)s.fits
level: None
fakes_deepCoadd_calexp:
description: "Same as deepCoadd_calexp but with fakes"
Copy link
Contributor

Choose a reason for hiding this comment

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

see above: fakes => fake objects

@sr525 sr525 merged commit 119cc93 into master Jun 6, 2019
@timj timj deleted the tickets/DM-15514 branch June 25, 2020 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants