RFC: show vectorscope along with waveform (or histogram)#20337
Draft
dtorop wants to merge 1 commit intodarktable-org:masterfrom
Draft
RFC: show vectorscope along with waveform (or histogram)#20337dtorop wants to merge 1 commit intodarktable-org:masterfrom
dtorop wants to merge 1 commit intodarktable-org:masterfrom
Conversation
There are two sorts of scopes: - Histogram and waveform primarily show per-channel lightness information. - Vectorscope shows chromaticity information without regard to lightness. A long-time request has been to show both at once. That way it is possible quickly to get a sense of both the lightness and chromaticity of an image without flipping between scopes. This commit is one way to handle this: It makes a new "vectorscope" module which will only show vectorscope. The new module can be displayed along with the existing histogram/waveform module.
Collaborator
|
From the perspective of efficient use of GUI real estate, Plan B is the most appealing. |
Collaborator
|
Ignoring ease-of-implementation issues, I'd put B and C tied for first place. Taking implementation into account, I vote for C, then D. For useful combinations, I'd say only vectorscope+waveform and vectorscope+RGBparade, though I could conceive of someone wanting histogram plus one of the others. Waveform plus parade is just plain silly..... |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
There are two sorts of scopes:
A long-time request has been to show both at once. That allows for a sense of both the lightness and chromaticity of an image without flipping between scopes.
Here's a few ideas on how to implement this. I'd be most interested in thoughts.
Plan A: Separate out vectorscope into its own utility module
This draft PR is one way to handle this: It makes a new "vectorscope" module contains the vectorscope-related code which was previously in the histogram (aka "scopes") module. The new vectorscope module can be displayed along with the existing histogram/waveform module. For example:
Advantages:
Disadvantages:
Plan B: Make the scopes module be able to split into two panes
The scopes module currently shows one scope (e.g. histogram only), and has buttons to flip between scopes. Instead, allow splitting the scopes module via the GUI in order to show a second scope on its right half. For example, control-clicking the icon of a scope might open a second pane on the right to display that scope.
Here is a mockup:
Advantages:
Disadvantage:
This could be an opportunity for some refactoring:
Plan C: As with Plan B, but only waverform/vectorscope split possible
Add an additional mode for the scopes module which implements just a waveform/vectorscope split. This would be a fifth button in the upper left of the scopes module (in addition to vectorscope, waveform, RGB parade, and histogram).
Advantages:
Disadvantages:
Plan D: As with Plan A, but vectorscope and histogram/waveform can expand/collapse like other utility modules
As with this PR, but make the scopes modules be expandable (like history, duplicates, masks).
Advantages:
Disadvantages:
Plan E: Leave things as is
Users who are interested in seeing both the waveform and vectorscope can set up a shortcut to switch between these, or use the buttons in the GUI. It's awkward, but saves screen space.