DM-45301: Write subsection on Difference Imaging about streaks, glints, and dropped diaSources #35
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
timj
reviewed
Apr 14, 2025
diffim.tex
Outdated
| After the \texttt{detectAndMeasure} step, some bogus DiaSources are expected due to reflective artificial objects orbiting the Earth. | ||
| These may originate from, e.g., satellites or space debris, which are most numerous in low-Earth orbit. | ||
| Objects in this region, with orbital altitude below 2000 km, move fast across the sky and can leave long streaks that cross one or more LSSTCam detectors. | ||
| For more details on how LSSTCam will be affected by bright streaks, see \citet{Tyson20,Hasan22,Polin24,Snyder24,Polin25}. |
There was a problem hiding this comment.
What is the Tyson 20 reference? It's not in the local.bib changes below.
Also, please use ADS bibcodes as bib keys so that we can keep track of the citations without risk of duplicate keys (Since we do move all the references into lsst-texmf to allow us to remove duplication and we can't do that if there are lots of Smith19 references to different things).
There was a problem hiding this comment.
cmsaunders
reviewed
Apr 15, 2025
diffim.tex
Outdated
|
|
||
| \item Streak detection and masking. | ||
| This is part of \texttt{meas\_algorithms} and runs during \texttt{detectAndMeasure}. | ||
| It adds a STREAK mask plane to the difference images by searching for linear features with a Canny filter and the Kernel-Based Hough Transform \citep{FernandesOliveira08}. |
There was a problem hiding this comment.
Maybe this line or a variation on it could be moved to 6.3.2 MaskStreaksTask, and then just put a reference to that subsection here.
|
@mrawls I've rebased again and switched the references to use ADS bibcodes. I won't merge because there is one unresolved comment. |
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.
This branch is now extremely out of date, but the text and new references should be good.