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

Lower the default number of bins #35

Open
thewtex opened this issue Sep 7, 2017 · 1 comment
Open

Lower the default number of bins #35

thewtex opened this issue Sep 7, 2017 · 1 comment

Comments

@thewtex
Copy link
Member

thewtex commented Sep 7, 2017

256 makes computation very slow, and this is larger than the typical number of effective intensity bins encountered in many images. A better default is 8, 10, 16?

@blowekamp
Copy link
Member

Related is the default range of the histogram. This defaults to the range of input pixel type. I think that reducing the size of the histogram may result in the all the intensities being in one bin for some common medical image modalities. Without adding a feature of automatically detecting the min/max of the input, it is risking producing constant images by default.

So I think this is a good idea. Along with adding a feature request for auto-range detection by default.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants