-
Notifications
You must be signed in to change notification settings - Fork 30
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
A framework for computing similarity between 2D gray-tone images #231
Conversation
Framework for calculating similarities between 2D gray-tone images of equal size.
This will be great additions, thank you @onatlandsmyr! Do you wan't a review right away? If yes, I'll get to it today. Also, do you want help with tests? And documentation? For this PR, I envision a module entry in the API reference documentation with all public classes/methods/functions fully documented from nice docstrings. |
A review would be good, sooner the better. Two more PRs are coming soon and they depend on this PR. I'll try to contribute more to the tests and docs, but any help is much appreciated. |
Okay, great, will review soon and mention wherever I plan to contribute. |
length of shape -> ndim, and removed unnecessary squeeze call.
Produce value of 1 with equal pattern and template
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.
This looks great, the general approach looks easily extendable and customizable, which is very nice to have.
There are many comments, some questions, some proposals for changes. Many of the comments are related to docstring formatting... Do you wan't to tackle these, or should I? I will add the indexing module to the API reference and see that all docstrings behave nicely, if that's okay with you.
Signed-off-by: Håkon Wiik Ånes <hwaanes@gmail.com>
Signed-off-by: Håkon Wiik Ånes <hwaanes@gmail.com>
…y into similarity-metrics
Signed-off-by: Håkon Wiik Ånes <hwaanes@gmail.com>
Signed-off-by: Håkon Wiik Ånes <hwaanes@gmail.com>
Yeah, the tests will fail since I didn't change the |
Change data shape from (N,nm) to (nm,N) to correspond better with cdist in scipy and general logic.
Signed-off-by: Håkon Wiik Ånes <hwaanes@gmail.com>
…y into similarity-metrics
Signed-off-by: Håkon Wiik Ånes <hwaanes@gmail.com>
Signed-off-by: Håkon Wiik Ånes <hwaanes@gmail.com>
Signed-off-by: Håkon Wiik Ånes <hwaanes@gmail.com>
Will merge after tests pass. Great work, @onatlandsmyr! |
Signed-off-by: Håkon Wiik Ånes <hwaanes@gmail.com>
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.
Let's get this in and move on to the next PR! Let's change the SimilarityMetric
class and related functions if necessary in that PR.
Similarity Metrics for 2D gray-tone images of equal size
A framework for computing similarity metrics between 2D gray-tone images of equal size. Provides a wrapper for user-defined metrics for use in, soon to be pull requested, template matching and dictionary indexing.
Zero-mean Normalized Cross-Correlation and Normalized Dot Product are implemented for inputs being dask arrays.
Progress of the PR
Minimal example of the bug fix or new feature
For reviewers
later.
__init__.py
.the unreleased section in
doc/changelog.rst
.