DM-41308: Create task to combine injection catalogs for matching #30
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sr525
reviewed
Jan 11, 2024
sr525
reviewed
Jan 11, 2024
sr525
reviewed
Jan 15, 2024
sr525
reviewed
Jan 15, 2024
sr525
reviewed
Jan 15, 2024
sr525
reviewed
Jan 15, 2024
sr525
reviewed
Jan 15, 2024
sr525
reviewed
Jan 15, 2024
jtmccann
force-pushed
the
tickets/DM-41308
branch
7 times, most recently
from
1e948bf
to
b690759
Compare
jtmccann
force-pushed
the
tickets/DM-41308
branch
3 times, most recently
from
0b59bbb
to
6d361f5
Compare
sr525
reviewed
Jan 29, 2024
sr525
reviewed
Jan 29, 2024
sr525
reviewed
Jan 29, 2024
sr525
reviewed
Jan 29, 2024
sr525
reviewed
Jan 29, 2024
sr525
reviewed
Jan 29, 2024
jtmccann
force-pushed
the
tickets/DM-41308
branch
from
6d361f5
to
9d9e655
Compare
jtmccann
force-pushed
the
tickets/DM-41308
branch
from
9d9e655
to
b11cf98
Compare
jtmccann
force-pushed
the
tickets/DM-41308
branch
3 times, most recently
from
3873bbc
to
4db55e8
Compare
jtmccann
force-pushed
the
tickets/DM-41308
branch
from
4db55e8
to
e21fae0
Compare
leeskelvin
reviewed
Feb 21, 2024
python/lsst/source/injection/utils/consolidate_injected_deepCoadd_catalogs.py
Outdated
Show resolved
Hide resolved
leeskelvin
reviewed
Feb 21, 2024
leeskelvin
reviewed
Feb 21, 2024
jtmccann
force-pushed
the
tickets/DM-41308
branch
from
e21fae0
to
dd35b23
Compare
leeskelvin
reviewed
Feb 22, 2024
python/lsst/source/injection/utils/consolidate_injected_deepCoadd_catalogs.py
Outdated
Show resolved
Hide resolved
leeskelvin
reviewed
Feb 22, 2024
leeskelvin
reviewed
Feb 22, 2024
jtmccann
force-pushed
the
tickets/DM-41308
branch
from
dd35b23
to
29747e3
Compare
sr525
reviewed
Feb 22, 2024
| ---------- | ||
| catalog: `astropy.table.Table` | ||
| A catalog of sources. | ||
| Writes is-patch-inner, is-tract-inner, and is-primary flags. |
There was a problem hiding this comment.
I got these doc strings straight from the code in meas_algorithms, so I'd assume it probably doesn't matter
There was a problem hiding this comment.
As I read it, "is-patch-inner" is meant to be descriptive, rather than referencing anything specific in the code. I.e., "This thing writes an 'is-patch-inner' flag". Is that accurate?
jtmccann
added a commit
that referenced
this pull request
Mar 21, 2024
DM-41308: Create task to combine injection catalogs for matching
jtmccann
added a commit
that referenced
this pull request
Mar 21, 2024
DM-41308: Create task to combine injection catalogs for matching
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.