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

Enable contrast control in all modes of tone equalizer #10987

Closed
kofa73 opened this issue Jan 25, 2022 · 2 comments
Closed

Enable contrast control in all modes of tone equalizer #10987

kofa73 opened this issue Jan 25, 2022 · 2 comments

Comments

@kofa73
Copy link
Contributor

kofa73 commented Jan 25, 2022

Is your feature request related to a problem? Please describe.
In tone equalizer's guided filter and eigf modes, a mask contrast compensation control is available. It can be used to fit the tonal range in the -8 EV .. 0 EV scale. The original intent was to counter the contrast reduction caused by blurring the mask; however, one can find examples of pixls.us where people actually use it to 'zoom into' a part of the tonal range they want to manipulate, to obtain finer control.

Since contrast reduction is not so severe when preserve details is set to one of the averaged modes and is not present at all with no, the contrast control is hidden, and the code uses 1.0 for the value (see https://discuss.pixls.us/t/processing-skies-with-clipped-channels-in-darktable-filmic-sigmoid/28995/105 by Aurélien). However, people would still like to be able to 'zoom in', see https://discuss.pixls.us/t/processing-skies-with-clipped-channels-in-darktable-filmic-sigmoid/28995/68.

Describe the solution you'd like
Enable the contrast control in all modes -- https://discuss.pixls.us/t/processing-skies-with-clipped-channels-in-darktable-filmic-sigmoid/28995/103

Alternatives

Additional note
It is my understanding that simply enabling the control could break existing edits: if someone with the current code tries several modes, first trying a guided filter, sets a contrast enhancement, then switches to one of the modes where the contrast adjustment is not used, a hard-coded value of 1.0 is applied by the formula. By naïvely modifying the code, when reprocessing the image, we’d suddenly apply the contrast adjustment read from config, originally set for the guided filter, and thus the mask and the effect would change.

@github-actions
Copy link

This issue did not get any activity in the past 60 days and will be closed in 365 days if no update occurs. Please check if the master branch has fixed it and report again or close the issue.

@github-actions
Copy link

This issue was closed because it has been inactive for 300 days since being marked as stale. Please check if the newest release or nightly build has it fixed. Please, create a new issue if the issue is not fixed.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Mar 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant