Test (et remarques) du nouveau module balance de couleurs

[aurelienpierre]

Je viens de re-tester la branche master, la plupart des soucis ont disparu. Il en reste quelques uns mineurs :

* À la toute première ouverture du module, un clic sur la pipette ne marche pas : l'icône est bien sélectionnée, mais on ne peut pas sélectionner de rectangle dans l'image. Je dis bien à la toute première ouverture : même en fermant+relançant darktable, le bug ne se reproduit pas. Par contre c'est reproductible en supprimant le fichier .config/darktable/darktablerc ! La seule différence entre la config problématique et celle qui ne pose pas problème est :

Code :

--- /tmp/.config/darktable/darktablerc  2018-10-29 21:18:32.187617946 +0100
+++ /tmp/.config/darktable/darktablerc.avec-probleme      2018-10-29 21:18:15.595567629 +0100
@@ -543,7 +543,7 @@
ui_last/colorpicker_mode=
ui_last/colorpicker_model=
ui_last/colorpicker_restrict_histogram=
-ui_last/colorpicker_size=1
+ui_last/colorpicker_size=
ui_last/colorsamples_mode=
ui_last/colorsamples_model=
ui_last/display_profile_source=all

est-ce que tu as le message "wait for the preview to finish" à l'ouverture ? Normalement, la pipette ne s'active pas si c'est en train de recalculer la preview.

[mmoy]

Je viens de re-tester la branche master, la plupart des soucis ont disparu. Il en reste quelques uns mineurs :

* À la toute première ouverture du module, un clic sur la pipette ne marche pas : l'icône est bien sélectionnée, mais on ne peut pas sélectionner de rectangle dans l'image. Je dis bien à la toute première ouverture : même en fermant+relançant darktable, le bug ne se reproduit pas. Par contre c'est reproductible en supprimant le fichier .config/darktable/darktablerc ! La seule différence entre la config problématique et celle qui ne pose pas problème est :

Code :

--- /tmp/.config/darktable/darktablerc  2018-10-29 21:18:32.187617946 +0100
+++ /tmp/.config/darktable/darktablerc.avec-probleme      2018-10-29 21:18:15.595567629 +0100
@@ -543,7 +543,7 @@
ui_last/colorpicker_mode=
ui_last/colorpicker_model=
ui_last/colorpicker_restrict_histogram=
-ui_last/colorpicker_size=1
+ui_last/colorpicker_size=
ui_last/colorsamples_mode=
ui_last/colorsamples_model=
ui_last/display_profile_source=all

Ah, c'est un peu plus embêtant que je pensais : le bug arrive en fait quand on active la pipette de « color balance » et que la pipette (« color picker » dans la barre latérale de gauche) était en mode « point ». Donc ça n'arrive pas qu'au premier lancement de darktable.

---

est-ce que tu as le message "wait for the preview to finish" à l'ouverture ? Normalement, la pipette ne s'active pas si c'est en train de recalculer la preview.

Non (enfin j'ai vu aussi le message, mais pas pour ce problème).

[aurelienpierre]

"Ah, c'est un peu plus embêtant que je pensais : le bug arrive en fait quand on active la pipette de « color balance » et que la pipette (« color picker » dans la barre latérale de gauche) était en mode « point ». Donc ça n'arrive pas qu'au premier lancement de darktable."

Il me semble que ça fait ça avec tous les modules, non ?

[mmoy]

"Ah, c'est un peu plus embêtant que je pensais : le bug arrive en fait quand on active la pipette de « color balance » et que la pipette (« color picker » dans la barre latérale de gauche) était en mode « point ». Donc ça n'arrive pas qu'au premier lancement de darktable."

Il me semble que ça fait ça avec tous les modules, non ?

Effectivement ce n'est pas spécifique à « color balance » : ça se produit aussi dans « exposure » et « unbreak input profile ». Par contre dans « levels » et « tone curve », quand la pipette est en mode point, cliquer sur la pipette active la pipette, en la gardant en mode point.

[pascal]

Donc c'est le changement de type de pipette qui ne marcherait pas?

[mmoy]

Oui, a priori les pipettes concernés devraient toutes passer la pipette en mode area mais ne le font pas correctement.

---

... et le bug est déjà présent dans la 2.4 pour le module « exposition ».

[aurelienpierre]

"Ah, c'est un peu plus embêtant que je pensais : le bug arrive en fait quand on active la pipette de « color balance » et que la pipette (« color picker » dans la barre latérale de gauche) était en mode « point ». Donc ça n'arrive pas qu'au premier lancement de darktable."

Il me semble que ça fait ça avec tous les modules, non ?

Effectivement ce n'est pas spécifique à « color balance » : ça se produit aussi dans « exposure » et « unbreak input profile ». Par contre dans « levels » et « tone curve », quand la pipette est en mode point, cliquer sur la pipette active la pipette, en la gardant en mode point.

C'est parce que dans levels et tonecurve, c'est pas une vraie pipette (au sens de darktable), c'est juste un point dessiné via cairo pour marquer les coordonnées, ensuite il récupère la valeur du pixel à direct dans l'image (sans la pipette). C'est un mécanisme plus compliqué (du point de vue programmation) qui n'utilise pas la lib colorpicker.

Il y aurait peut-être du boulot à faire sur les libs, mais là il faudrait une équipe à temps plein.

[pascal]

Normalement j'ai corrigé le problème avec la pipette. À tester...

[mmoy]

J'ai galéré parce que je suis tombé sur ce bug https://redmine.darktable.org/issues/12383 mais une fois un revert du commit qui posait problème, je confirme que la pipette marche nickel ! Merci !

[rawfiner]

J'ai galéré parce que je suis tombé sur ce bug https://redmine.darktable.org/issues/12383 mais une fois un revert du commit qui posait problème, je confirme que la pipette marche nickel ! Merci !

Je vais jeter un oeil au bug ce soir, j'ai une piste sur l'endroit d'où ça peut venir.