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

Update white balance observer tooltip #6970

Closed
Lawrence37 opened this issue Mar 4, 2024 · 14 comments · Fixed by #6975
Closed

Update white balance observer tooltip #6970

Lawrence37 opened this issue Mar 4, 2024 · 14 comments · Fixed by #6975
Labels
scope: GUI Changes to GUI, not core functionality
Milestone

Comments

@Lawrence37
Copy link
Collaborator

The tool-tip says the default observer is 10 degrees, but it is actually 2. The rest of the tool-tip should be reviewed for accuracy (is 10 degrees still the better option in most cases?). Here's the current tooltip:

The color management in Rawtherapee (White balance, channel multipliers, highlight recovery,...) uses the spectral data of the illuminants and colors. Observer is an important parameter of this management which takes into account the angle of perception of the eye. In 1931 it was fixed at 2° (privileges the use of the cones). In 1964 it was fixed at 10° (privileges the use of the cones, but partially takes into account the rods).
To avoid a (rare) drift of the colors due to the choice Observer 10° - probably due to the conversion matrix - Observer 2° must be selected.
In a majority of cases Observer 10° (default) will be a more relevant choice.

@Lawrence37 Lawrence37 added the scope: GUI Changes to GUI, not core functionality label Mar 4, 2024
@Lawrence37 Lawrence37 added this to the v5.11 milestone Mar 4, 2024
@Desmis
Copy link
Collaborator

Desmis commented Mar 4, 2024

@Lawrence37
As you say elsewhere, developing documentation (website, rawpedia, video, etc.) is a long process, which requires many conditions.

The corollary problem, also complex, is the evolution of this documentation through that of the software functionalities.

This “Observer” case is just one example among others.
Here is what I propose. Of course, you can change, my english is so bad.

No changes for the first part:
"The color management in Rawtherapee (White balance, channel multipliers, highlight recovery,...) uses the spectral data of the illuminants and colors. Observer is an important parameter of this management which takes into account the angle of perception of the eye. In 1931 it was fixed at 2° (privileges the use of the cones). In 1964 it was fixed at 10° (privileges the use of the cones, but partially takes into account the rods)."

And for the second:
In a majority of cases Observer 2° (default) will be the best choice.
To avoid a (rare) drift of the colors due to the choice Observer 2° - probably due to the conversion matrix - Observer 10° must be selected.

Jacques

@Desmis
Copy link
Collaborator

Desmis commented Mar 5, 2024

If you are OK, I will change the tooltip tomorrow, directly in dev (without PR, if you agree).

Jacques

@alice-mm
Copy link

alice-mm commented Mar 5, 2024

Hi and thanks for taking care of this!
I suggested an alternate version (for the last part) here:
https://discuss.pixls.us/t/questions-how-to-choose-a-dcp-profile-if-at-all/42404/10?u=alicem

@Desmis
Copy link
Collaborator

Desmis commented Mar 5, 2024

@alice-mm

For me it's OK

In the rare occurrence where a drift of the colors is noticed with “Observer 2°” – generally due to the conversion matrix –, “Observer 10°” must be selected instead.

But I am not at all english...
Thank you.

Jacques

@alice-mm
Copy link

alice-mm commented Mar 5, 2024

I’m French so I’ll let @Lawrence37 be the judge. xD

@Desmis
Copy link
Collaborator

Desmis commented Mar 5, 2024

@alice-mm
In french :)

Sympathique qu'il y ait des français(e) qui participent.
N'hésites pas à me contacter pour n'importe quel sujet (si je le maîtrise bien sûr)

Jacques

@Lawrence37
Copy link
Collaborator Author

Well, I'm embarrassed. I know the distinction between hyphens and dashes, but I had to look up the difference between an en dash (short dash) and an em dash (long dash). The em dash (—) is correct here. To make it more concise, I suggest

In the rare case of a color drift with "Observer 2°" — generally due to the conversion matrix — “Observer 10°” must be selected.

I have one question for Jacques. The original tool-tip reads "probably due to the conversion matrix" while Alice's is "generally due to the conversion matrix". "probably" and "generally" have different meanings. "probably" implies we don't know the cause, but we suspect it has something to do with the conversion matrix. "generally" means most color drifts that have been observed are caused by the conversion matrix. Essentially, the difference between the words is whether we know the conversion matrix causes color drifts. Which one is right?

@Desmis
Copy link
Collaborator

Desmis commented Mar 6, 2024

@Lawrence37

For me it's "probably", because we don't know what "Adobe" does ( or else we would have to send the CIA or the FSB...)

I will take your last text, with :
In the rare case of a color drift with "Observer 2°" — probably due to the conversion matrix — “Observer 10°” must be selected.

And to be sure, I will create a PR.

Jacques

@Lawrence37
Copy link
Collaborator Author

Sounds good.

@Desmis
Copy link
Collaborator

Desmis commented Mar 6, 2024

@Lawrence37
@alice-mm

I just create a Pull-Request with this (small, but important) change

Jacques

@alice-mm
Copy link

alice-mm commented Mar 6, 2024

Well, I'm embarrassed. I know the distinction between hyphens and dashes, but I had to look up the difference between an en dash (short dash) and an em dash (long dash). The em dash (—) is correct here.

Haha sorry for launching you on that topic. Hum yeah they’re sometimes used interchangeably for this, but ultimately the em may be better yeah. Note, however, that it has no spaces on the side, generally, unlike the en. (While in French we use spaces regardless of en or em. 🙃) Wikipedia gives examples such as: The food—which was delicious—reminded me of home.

"probably" implies we don't know the cause

Sorry about this! I indeed wrongly assumed that we kinda knew what was happening.

@Desmis
Copy link
Collaborator

Desmis commented Mar 6, 2024

in French

D'où ma remarque sur la CIA ou le FSB...
La colorimétrie est quelque chose d'incroyablement complexe, par elle même. Mais en plus il y a des inconnues.
Que font réellement dans leur traitement du capteur, chaque constructeur (Canon, Nikon, Sony,...), ils ne livrent que ce qu'ils veulent. Comment Adobe "décrypte" les Raw des constructeurs , ont-ils les "secrets" ou non...?

Bref, pas simple.
Si tu es intéressé, c'est un peu complexe... je peux expliquer - ce que j'ai fait (le plus complexe et de loin - l'algorithme WBauto Temperature correlation" ) - ou du moins ce que je crois comprendre.

Jacques

@Lawrence37 Lawrence37 linked a pull request Mar 7, 2024 that will close this issue
@alice-mm
Copy link

alice-mm commented Mar 9, 2024

Bref, pas simple. Si tu es intéressé, c'est un peu complexe... je peux expliquer - ce que j'ai fait (le plus complexe et de loin - l'algorithme WBauto Temperature correlation" ) - ou du moins ce que je crois comprendre.

Je crois que c’est bien au-dessus de mon niveau. xD

By the way, do such changes automatically get reflected on files such as rtdata/languages/Francais? There seems to be a diff-script thingy that gets run before releases.
Maybe one day I’ll try to help a little on French translation. I never realized there was one, as I use the English version. 😆
Oh yeah OK there are !-s indicating whether an entry has been translated or not. I suppose that’s how the diff script can do its magic.

@Desmis
Copy link
Collaborator

Desmis commented Mar 9, 2024

In french

En fait, rien n'est simple, y compris pour la documentation.
Je suis français, vieux...(très), parlant mal anglais (mes petits fils parlent beaucoup mieux que moi)...et aucune formation d'informatique (je suis autodidacte...), donc, code brouillon, etc.

Depuis que je coopère avec Wayne @Wayne_Sutton50 ou @Wayne_Sutton qui est anglophone de naissance et habite en France depuis longtemps, pour moi, tout va bien...Notre coopération se passe au mieux... Seul problème en termes de convivialité, nous sommes à 1000km de distance.

Non seulement il traduit, mais il essaye, critique, souhaite des modifications - souvent de GUI - le plus souvent sur la compréhension de ce que cela fait...

Ce qui amène que la version 'anglaise' est mieux (c'est presque un comble) que la française, que ce soit pour la documentation, mais aussi les Labels et Tooltips. C'est un travail considérable, on y passe beaucoup de temps. L'inconvénient c'est que Rawpedia (le support) est antique... Il y a un projet d'amélioration (je ne comprends pas tout ce qui est prévu).
Mais en attendant, si je fais, ou un autre développeur (je n'aime pas ce terme), une modification logicielle (algorithmes, GUI,...) il faudrait en théorie revoir la documentation. Jette un oeil à "Local Adjustments" dans Rawpedia... tu verras..Le travail est colossal.

En tous cas, je retiens ta proposition, si tu souhaites coopérer comme Wayne (traduire, tester, critiquer...), cela présente tout un tas d'avantages (à part le temps à y passer), tout le monde est gagnant.
Contactes moi, le cas échéant.

Mais cette proposition tient aussi pour les autres participants, traduire de anglais à Français ou l'inverse... Et toutes les langues...Japonais, allemand, néerlandais..
Détail qui n'en est pas un: il faut avoir accès à Rawpedia... donc être autorisé..Moi je l'ai cet accès , mais qui autorise ??
L'idéal dans un monde parfait, c'est que la doc, les tooltips, soient accesibles via une Pull-request...

Merci.

Bon week-end.

Jacques

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: GUI Changes to GUI, not core functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants