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

Feature request: Intensity values & auto brightness #123

Open
StephanPreibisch opened this issue May 6, 2021 · 3 comments
Open

Feature request: Intensity values & auto brightness #123

StephanPreibisch opened this issue May 6, 2021 · 3 comments

Comments

@StephanPreibisch
Copy link
Member

Hi, we should think about adding the very basic features (that are more complex in this context) of displaying a pixels intensity and auto brightness adjustment to BDV. Happy to discuss and contribute!

@kgabor
Copy link

kgabor commented Sep 30, 2022

I also miss the values displayed when pointing to (x,y,z) pixels... Also a colorbar scale(s) would be nice.

@tischi
Copy link
Contributor

tischi commented Jan 17, 2023

@tpietzsch

This reminded me of this issue , because the user will have to navigate larger value ranges when supporting more data types.

Auto-contrast UI

What about adding an option for auto-contrast somewhere here?
Maybe add set range auto?

image

Pixel values UI

Two suggestions:

  1. Add value next to name, current, active, color
  2. Make a new card, called Sources pixel values, where you repeat all the source names

Maybe the second approach could have the advantage that if one hides this card, the code would know that it does not have to fetch the pixel values (in case that is expensive).

image

@tischi
Copy link
Contributor

tischi commented Jan 18, 2023

Maybe to clarify my terminology:

range: the current min max values of the Converter
bounds: the min and max limits of the UI sliders

For datatypes like uint64 it is, in my view, not trivial to set the bounds such that the user can still sensibly use them.
In MoBIE we have thus implemented some logic that computes the bounds given the current range, happy to discuss this.

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

3 participants