darktable-org / darktable

darktable is an open source photography workflow application and raw developer
https://www.darktable.org
GNU General Public License v3.0
9.71k stars 1.14k forks source link

color wheel in color balance rgb #12286

Closed ghost closed 2 years ago

ghost commented 2 years ago

The 4 way tab in the color balance rgb module doesn't use color wheel to adjust and it feels unintuitive. Lots of professional softwares use color wheels for that purpose like resolve, ACR, Lightroom and even premier pro. So I think it's better to have a color wheel interface. And by the way the Chinese translation of that tab looks kind of odd, so I wonder how can I help it improve . Thank you.

MStraeten commented 2 years ago

4 colorwheels will require more space without giving more control. Video software often uses color wheels because they match the used hardware decks also containing wheels to control color. Most photo editors are just using keys or mouse - so dragging a slider is far from being uncommon. So what’s the benefit? If you need wheels you can assign those controls to a midi controller ;)

jandren commented 2 years ago

There is one huge benefit, and that is that we get one 2D orthogonal controller instead of two 1D sliders which are dependent on each other (angle + magnitude). Shortcut and Midi users might also benefit from this by mapping two controllers to the x/y direction of the color wheel.

Graphically leaning users will probably also value the graphical control more than the cost of lost space (we can collapse the module after all). The tricky question to be answered if someone is to implement this is where would be but the toggle between the two input methods?

Jiyone commented 2 years ago

I suppose color wheels would means changing the way the module works. Now it's angle and radius, a color wheel is x and y position (with sin and cos I presume). So I believe converting x and y to angle and radius will be too slow or too much work so this have to be directly coded in the color core. :thinking:

lemirep commented 2 years ago

I've also been finding the slider unintuitive to work with when you want to visualize how highlights/midtones/shadows relate to one another. Doing things like split complimentary or adjacent colors requires you to compute angle values yourself with the sliders. Scratching my own itch, I started adding a color wheel control https://github.com/darktable-org/darktable/pull/12297 Anyone willing to give a try to and discuss improvements?

emko commented 2 years ago

4 colorwheels will require more space without giving more control. Video software often uses color wheels because they match the used hardware decks also containing wheels to control color. Most photo editors are just using keys or mouse - so dragging a slider is far from being uncommon. So what’s the benefit? If you need wheels you can assign those controls to a midi controller ;)

what? you think adding a midi controller equals the same as a color wheel you must be confused?, there is a big reason LR was begged to add color wheels for years just like every other editing software has of course you can still use sliders but visually color wheel is way easier to use.

i would have a header button bar where you can select different ways to interact with the colors kind of like how the masking module works or like how LR has a icon for 3 circles then one for each and have it default to sliders if that's what it takes. Ideally color wheels should be a option for any module that has a slider for colors

image

image

MStraeten commented 2 years ago

are you sure you get more control with those colorwheels? What’s the problem you want to solve? In the current solution you get exact numbers of your changes ( you need to add 3 number files to each wheel to get the same information (especially for subtile changes you don’t get much information in the LR solution) you can enter values manually (how to do this with a widget representing 2 values). Unfortunately all those suggestions just focus on the appearance but not on the way to use it… and how that affects workflows supported by the current slider solution

ghost commented 2 years ago

are you sure you get more control with those colorwheels? What’s the problem you want to solve? In the current solution you get exact numbers of your changes ( you need to add 3 number files to each wheel to get the same information (especially for subtile changes you don’t get much information in the LR solution) you can enter values manually (how to do this with a widget representing 2 values). Unfortunately all those suggestions just focus on the appearance but not on the way to use it… and how that affects workflows supported by the current slider solution

It’s not about the digits and accuracy the slider provides, it’s about the look the module eventually makes. Using numbers to color grade is literally a painful process. Besides color wheels are two dimensional they control hue and saturation at the same time while slider can only control one. It is a faster way to get the wanted result.

MStraeten commented 2 years ago

maybe for colorgrading but darktable is a raw processor, not intended for video production ;) for this changes the color correction module might be more appropriate… To get colors right the ui must be able to support even subtle changes of colors - mostly in the centre of the color wheel…

ghost commented 2 years ago

maybe for colorgrading but darktable is a raw processor, not intended for video production ;) for this changes the color correction module might be more appropriate… To get colors right the ui must be able to support even subtle changes of colors - mostly in the centre of the color wheel…

Well I think I get what you’re saying. You mean darktable is not for art creativity but for reconstruct the true scene that human can see. Is that what you mean? That’s totally acceptable. But I would still like to have more creativity options because the filmic rgb module produces better color than any other software I have ever used.

MStraeten commented 2 years ago

You mean darktable is not for art creativity

bullshit

But I would still like to have more creativity options

thats no contradiction - but don't mix up UI and UseCases: having colorwheel controls and the latest intention of lemirep are two different things.

if it's just about a color wheel control then there are ways to do this (see davinci - you don't lose detailed info in a different interface; all thats just UI not related to a Usecase):

image

image You might even switch between HSL or RGB like in the legacy colorbalance module

But if you want to visualize how shadows/midtones/highlights are related and be able to change this thats a different usecase requiring a different UI You'd have a look at color correction module - currently it just display the relative change, maybe it's worth to improve it to better support that usecase;) Do the tasks at the proper place - there's no need to do everything in one module in darktable ...

In general all those color balance/correction/grading usecases needs a consistent UX concept - not just arbitrary patches just because some users are used to different software solution for just their personal usecase ;)

ghost commented 2 years ago

You mean darktable is not for art creativity

bullshit

But I would still like to have more creativity options

thats no contradiction - but don't mix up UI and UseCases: having colorwheel controls and the latest intention of lemirep are two different things.

if it's just about a color wheel control then there are ways to do this (see davinci - you don't lose detailed info in a different interface; all thats just UI not related to a Usecase): image image You might even switch between HSL or RGB like in the legacy colorbalance module

But if you want to visualize how shadows/midtones/highlights are related and be able to change this thats a different usecase requiring a different UI You'd have a look at color correction module - currently it just display the relative change, maybe it's worth to improve it to better support that usecase;) Do the tasks at the proper place - there's no need to do everything in one module in darktable ...

In general all those color balance/correction/grading usecases needs a consistent UX concept - not just arbitrary patches just because some users are used to different software solution for just their personal usecase ;)

So the solution you prefer is to add another modules that serves the right purpose? That’s also acceptable. But no color wheel is really frustrating and disappointing. It’s not just a preference problem. And why do you always talk about resolve? You know it’s not for raw photo editing. Why don’t you talk about capture one or Lightroom or adobe camera raw? They are all raw editing tools and they all have color wheel. Currently only darktable does not have color wheel. So there’s no preference it’s a situation where darktable is the left over one.

MStraeten commented 2 years ago

i gave the sample of davinci because it’s not about copying adobe UI but dealing with color in a reasonable way. ‚I want to it like capture one or adobe lightroom‘ isn’t the best argument for changes in darktable. Darktable is not intended to be a free lightroom clone. And btw: I use davinci as a sample because I use it a lot but never c1 or adobe lightroom because I prefer the darktable way;)

ghost commented 2 years ago

i gave the sample of davinci because it’s not about copying adobe UI but dealing with color in a reasonable way. ‚I want to it like capture one or adobe lightroom‘ isn’t the best argument for changes in darktable. Darktable is not intended to be a free lightroom clone. And btw: I use davinci as a sample because I use it a lot but never c1 or adobe lightroom because I prefer the darktable way;)

So in the end it’s just your own preference isn’t it? I know it’s an open source software but go to the opposite way of the whole industry usually doesn’t turn out to be a good idea.

ghost commented 2 years ago

And btw I don’t think your attitude is truly want to help photographers but satisfy your own need. No wonder why so many people choose Lightroom or other software. Because they are much more modernized and up to date. At least they don’t lag on a thousand dollar Mac

MStraeten commented 2 years ago

being rude isn’t the best indication for a valuable request. In foss you need to provide arguments and not scream for candy like a spoiled little child

MStraeten commented 2 years ago

it seems you’re really a little spoiled child screaming for candy …

ghost commented 2 years ago

it seems you’re really a little spoiled child screaming for candy …

Find someone in the Chinese restaurant and let him translate for you. Chinese is not what you programmers and translating softwares can understand.

MStraeten commented 2 years ago

I do not want to shame them because of such compatriots

ghost commented 2 years ago

I do not want to shame them because of such compatriots

No they won’t. You just need to care about your own safety.

ghost commented 2 years ago

I do not want to shame them because of such compatriots

And I am so sorry about your existence in the whole programming community. You’re selfish and mean at the same time without a progressive eyesight, producing garbage sources codes that nobody would pay for them that’s why darktable is “open source” isn’t it? Keep rejecting users’ suggestions using unreasonable explanations and your own personal opinions is why you died.

MStraeten commented 2 years ago

is your playground closed?

ghost commented 2 years ago

is your playground closed?

Ancient man, what else can you say? Who plays in the playground these days. Do you know why everyone knows Tik tok? Because it does way better programming works than you did and using the most advanced AI. Billions of billions of people using it every single day while you have little poor users standing your attitude and laggy interface you made.