Skip to content

Update doc for default tone-mapper workflow #759

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Phemisters
Copy link
Contributor

darktable PR: darktable-org/darktable#18547

Update the default in preferences for the workflow - I moved sigmoid up as it is the new default and that also makes the description order correspond to the order of the combo-box.

While doing this I noticed the white balance now has "as shot to reference" and for me that seems to be what it defaults to. That is why I updated the sentence about white balance, as I think it applies in the same way as for "camera reference". You may prefer we wait to do that change when darktable PR 15602 is done.

@elstoc
Copy link
Contributor

elstoc commented Apr 18, 2025

Thanks. I will hold off merging this for now as I suspect we might need to make some other tweaks elsewhere in the doc to account for this. For example, while I'm not currently interested in changing the recommendations in the overview/workflow section (which suggests using filmic to process), it might still need some wording tweaks to account for the new default.

@Phemisters
Copy link
Contributor Author

we might need to make some other tweaks

I'm not sure if you have a plan for this. I have had a go at some small updates for this, but happy to revert those or indeed close this PR until someone has the time and knowledge for a more thorough update.

Copy link

This pull request has not had any activity in the past 60 days and will be closed in 365 days if not updated. Please verify it has no conflicts with the master branch and rebase if needed. Please add a comment if you need help or give permission to other people to finish your work.

@github-actions github-actions bot added the no-pr-activity No activity on this PR label Jun 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-pr-activity No activity on this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants