add toggle to show keypoint coverage #21
Merged
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.
Add a toggle to show the keypoints that the detector has identified for more easily debugging whether the an image contains copied sections.
I was looking into an image that has some copied subsections that were not detected by sherloq and noticed that at low contrast the image has very few keypoints detected. Bumping up the contrast helped the detectors find more points but it wasn't immediately obvious what was wrong because the detectors do still detect many keypoints.
My hard-coded color was not chosen thoughtfully, but could be set to the inverse of the center point.