Skip to content
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

Gabor aspect ratio #2049

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

wwjd228
Copy link

@wwjd228 wwjd228 commented Apr 19, 2016

Description

[Tell us about your new feature, improvement, or fix! If relevant, please supplement the PR with images.]
I add a new parameter about aspect ratio in function gabor_kernel.

Checklist

[It's fine to submit PRs which are a work in progress! But before they are merged, all PRs should provide:]

[For detailed information on these and other aspects see scikit-image contribution guidelines]

References

[If this is a bug-fix or enhancement, it closes issue # ]
[If this is a new feature, it implements the following paper: ]

This parameter is the spatial aspect ratio, and specifies the ellipticity of the support of the Gabor function.
This parameter is the spatial aspect ratio, and specifies the ellipticity of the support of the Gabor function.
This parameter is the spatial aspect ratio, and specifies the ellipticity of the
support of the Gabor function.

And add a example at ./doc/examples/filters
This parameter is the spatial aspect ratio, and specifies the ellipticity of the support of the Gabor function.

And add an example at ./doc/examples/filters
Aspect ratio is the spatial aspect ratio, and specifies the ellipticity of the support of the Gabor function.
@@ -14,7 +14,7 @@ def _sigma_prefactor(bandwidth):


def gabor_kernel(frequency, theta=0, bandwidth=1, sigma_x=None, sigma_y=None,
n_stds=3, offset=0):
n_stds=3, offset=0, aspectratio=1):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please rename to aspect_ratio

@ahojnnes
Copy link
Member

ahojnnes commented May 7, 2016

Thanks for your contribution. This needs a unit test with an aspect ratio of != 1

@sciunto
Copy link
Member

sciunto commented Dec 26, 2016

@wwjd228 Did you have a chance to look at it? The modifications required are minor :)

Rename the parameter "aspectratio" to "aspect_ratio".
@wwjd228
Copy link
Author

wwjd228 commented Jan 22, 2017

Sorry, I don't know how to create a unit test with an aspect ratio of != 1.
Could you teach me how to do this??

@codecov-io
Copy link

codecov-io commented Jan 22, 2017

Current coverage is 90.71% (diff: 100%)

No coverage report found for master at 8400804.

Powered by Codecov. Last update 8400804...2a8670f

Base automatically changed from master to main February 18, 2021 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants