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-39466: Added flagging of sources near the INJECTED and INJECTED_TEMPLATE mask plane #293
Conversation
e0888b4
to
b28c2fc
Compare
3724b0a
to
10d485b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code changes look fine, and my local tests with ap_verify
pass. Please add a new unit test to check that detectAndMeasure
correctly flags sources with footprints including pixels with the INJECTED
or INJECTED_TEMPLATE
mask set. A simple Jenkins re-run (without ci_hsc
or ci_imsim
) should be sufficient if you just add a test.
43b4832
to
fcb4529
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've added my review of the new unit test
fcb4529
to
a770d1e
Compare
I have applied the changes requested for the test. This improves significantly the quality of this unit test. Thanks very much |
Added requested test. Modified a test from subtract and migrated it to detectandmeasure
a770d1e
to
0f3fc58
Compare
No description provided.