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

DOCS: Description of example in skimage.feature.graycomatrix documentation does not seem to match the code #7293

Open
FedericoWZhaw opened this issue Jan 12, 2024 · 2 comments
Labels
📄 type: Documentation Updates, fixes and additions to documentation

Comments

@FedericoWZhaw
Copy link

Description:

In the scikit image documentation on the gray-level co-occurrence matrix, the Example states: "Compute 2 GLCMs: One for a 1-pixel offset to the right, and one for a 1-pixel offset upwards."

The code below however computes 4 GLCMs
result = graycomatrix(image, [1], [0, np.pi/4, np.pi/2, 3*np.pi/4], levels=4).

Should it not just be
result = graycomatrix(image, [1], [0, np.pi/2], levels=4)?

@lagru
Copy link
Member

lagru commented Jan 15, 2024

Thanks for the hint @FedericoWZhaw! #955 makes me think that that when the other angles were added the comment was just forgotten. Would you like to make a PR that fixes the comment? 😊 🙏

@FedericoWZhaw
Copy link
Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📄 type: Documentation Updates, fixes and additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants