Note de ce sujet :
  • Moyenne : 0 (0 vote(s))
  • 1
  • 2
  • 3
  • 4
  • 5
Nouveau module : filmic
#31
Le comportement est un peu imprévisible, et pas toujours celui attendu.
Aurélien, photographe portraitiste, spécialiste calcul.
Développeur de filmique, égaliseur de tons, balance couleur, etc.
darktable est mon métier, pensez à m'aider :
[Image: 2FAd4rc]
Répondre
#32
J'ai fait mes premiers tests avec l'outil, ça marche plutôt bien pour l'instant et ça a l'air vraiment intéressant.

Je suis sur la branche master de dt : 6135fe8 (Release the cache before updating the database., 2018-11-09).

Il y a une mauvaise interaction entre le bouton « auto tune source » et les pipettes. Si on utilise les pipettes avant le color picker, tout est cassé : la zone utilisée en dernier par le colorpicker est utilisée par le bouton « auto tune source », et du coup ça fait n'importe quoi. Et ça continue à faire n'importe quoi longtemps après :-(.

Le problème est dans le code au niveau de optimize_button_pressed_callback : le module demande à utiliser le colorpicker sur l'ensemble de l'image, mais il enchaîne directement sur l'utilisation du résultat, alors que le pipeline n'a pas été relancé (dt_dev_reprocess_all(self->dev) n'a pas l'air de relancer le pipe immédiatement, si j'ai bien compris il demande à le relancer mais le calcul sera fait après). Il faudrait faire comme pour les callbacks des pipettes : quand on clique, positionner which_colorpicker sur une valeur spécifique à l'optimiseur, et dans un autre callback récupérer la valeur. Je n'ai pas trouvé de moyen simple d'avoir le fameux « autre callback ».

J'ai tenté un correctif sans succès, mais j'ai avancé un peu en corrigeant le truc pas grave (icone qui n'était pas désactivée) : https://github.com/darktable-org/darktable/pull/1814
Répondre
#33
Bon, les maths devraient être réglées dans ma dernière PR.

(09-11-18, 23:46)mmoy a écrit : J'ai fait mes premiers tests avec l'outil, ça marche plutôt bien pour l'instant et ça a l'air vraiment intéressant.

Je suis sur la branche master de dt : 6135fe8 (Release the cache before updating the database., 2018-11-09).

Il y a une mauvaise interaction entre le bouton « auto tune source » et les pipettes. Si on utilise les pipettes avant le color picker, tout est cassé : la zone utilisée en dernier par le colorpicker est utilisée par le bouton « auto tune source », et du coup ça fait n'importe quoi. Et ça continue à faire n'importe quoi longtemps après :-(.

Le problème est dans le code au niveau de optimize_button_pressed_callback : le module demande à utiliser le colorpicker sur l'ensemble de l'image, mais il enchaîne directement sur l'utilisation du résultat, alors que le pipeline n'a pas été relancé (dt_dev_reprocess_all(self->dev) n'a pas l'air de relancer le pipe immédiatement, si j'ai bien compris il demande à le relancer mais le calcul sera fait après). Il faudrait faire comme pour les callbacks des pipettes : quand on clique, positionner which_colorpicker sur une valeur spécifique à l'optimiseur, et dans un autre callback récupérer la valeur. Je n'ai pas trouvé de moyen simple d'avoir le fameux « autre callback ».

J'ai tenté un correctif sans succès, mais j'ai avancé un peu en corrigeant le truc pas grave (icone qui n'était pas désactivée) : https://github.com/darktable-org/darktable/pull/1814

Merci pour ton aide. GTK et moi…
Aurélien, photographe portraitiste, spécialiste calcul.
Développeur de filmique, égaliseur de tons, balance couleur, etc.
darktable est mon métier, pensez à m'aider :
[Image: 2FAd4rc]
Répondre
#34
(09-11-18, 23:46)mmoy a écrit : Il y a une mauvaise interaction entre le bouton « auto tune source » et les pipettes. Si on utilise les pipettes avant le color picker, tout est cassé : la zone utilisée en dernier par le colorpicker est utilisée par le bouton « auto tune source », et du coup ça fait n'importe quoi. Et ça continue à faire n'importe quoi longtemps après :-(.

Le problème est dans le code au niveau de optimize_button_pressed_callback : le module demande à utiliser le colorpicker sur l'ensemble de l'image, mais il enchaîne directement sur l'utilisation du résultat, alors que le pipeline n'a pas été relancé (dt_dev_reprocess_all(self->dev) n'a pas l'air de relancer le pipe immédiatement, si j'ai bien compris il demande à le relancer mais le calcul sera fait après). Il faudrait faire comme pour les callbacks des pipettes : quand on clique, positionner which_colorpicker sur une valeur spécifique à l'optimiseur, et dans un autre callback récupérer la valeur. Je n'ai pas trouvé de moyen simple d'avoir le fameux « autre callback ».

Ce bout là est toujours problématique dans master. C'est plus grave que ça en a l'air : le bouton reste cassé quand on réinitialise le module et quand on change d'image :-(. Donc faut vraiment corriger, c'est plus qu'un petit glitch de l'interface.

En fait, je me dis que le bouton « auto tune source » aurait tout intérêt à être une vraie pipette dans l'interface :

* Pour l'utilisateur, ça permet de spécifier une portion de l'image sur laquelle faire les mesures min/moyenne/max, par exemple pour exclure un point trop brillant (exemple : photo en contre-jour, on peut sélectionner un rectangle avec de l'ombre, un bout du sujet et un bout très éclairé, mais exclure le soleil du rectangle de mesure).

* En terme de cohérence d'interface graphique, ce bouton est très similaire à ce que fait la pipette de « exposition » (sauf que cette dernière ne regarde que le min et le max, pas la moyenne), et dans « exposition » c'est une pipette. Donc tant qu'à faire un truc similaire à une fonctionnalité existante, autant que la GUI soit similaire aussi.

* En terme de code, ça ramène aux cas déjà traités des autres pipettes, c'est plus facile d'être cohérent.

Bref, à mon avis c'est plus facile à coder et mieux pour l'utilisateur.
Répondre
#35
L'idée du bouton auto-tune, c'est justement d'avoir un réglage taille unique rapide et pas trop regardant, mais débrayable au cas où ça foire. Pour des photos d'extérieur sans contre-jour majeur, ça marche très bien et ça économise 6 clics. C'est bien aussi pour avoir un aperçu rapide d'une photo à la place de la courbe de base, que j'essaie désespérément de faire déprécier.

Je vais regarder.
Aurélien, photographe portraitiste, spécialiste calcul.
Développeur de filmique, égaliseur de tons, balance couleur, etc.
darktable est mon métier, pensez à m'aider :
[Image: 2FAd4rc]
Répondre
#36
(10-11-18, 08:58)aurelienpierre a écrit : L'idée du bouton auto-tune, c'est justement d'avoir un réglage taille unique rapide et pas trop regardant, mais débrayable au cas où ça foire. Pour des photos d'extérieur sans contre-jour majeur, ça marche très bien et ça économise 6 clics. C'est bien aussi pour avoir un aperçu rapide d'une photo à la place de la courbe de base, que j'essaie désespérément de faire déprécier.

Je vais regarder.

Si tu transformes le bouton en pipette, mais que la pipette sélectionne 99% de l'image par défaut (comme c'est le cas en interne aujourd'hui), tu peux faire la même chose en 1 clics et demi :

* 1 clic sur la pipette => la zone s'affiche, l'image est recalculée, hop c'est joli et ça a choisi tout les bons réglages.

* 1 clic sur la pipette => on désactive la pipette.

Je dis 1,5 clic et pas 2 parce que le deuxième est optionnel : tu peux garder la pipette activée et passer à autre chose, la pipette sera désactivée toute seule au prochain changement de module par exemple.

Regarde ce que fait la pipette de « exposition », c'est vraiment ça que j'ai en tête.
Répondre
#37
Merci Mathieu, j'avais remarqué ce problème, effectivement quand on utilise une pipette et ensuite réglage auto : il y a un rendu bizarre. Je n'étais dit qu'il faut utilisé réglage auto doit être utilisé en premier et que si l'on a utilisé les pipettes, il ne faut pas l'utiliser après. Je remarque aussi un comportement bizarre quand on a bougé les curseurs. J'avais remarqué au dans l'algorithme de "correction de profil d'entrée".
J'ai aussi remarqué que l'utilisation de réglage auto 2 fois de suite crée une photo difficilement exploitable et l'impossibilité de réinitialiser le module et il faut sortir de darktable. J'ai aussi remarqué qu'il est impossible d'utiliser le réglage auto dans "correction de profil d'entrée".
Je dis peut-être une bêtise : ne serai t’il pas possible de dévaliser le bouton quand on a utilisé les pipettes, les curseurs et quand on a utilisé une fois réglage auto et aussi dans "correction du profil d'entrée" ?

Je viens de recompiler et je n'arrive plus à reproduire les problèmes. Ca semble résolu !
Répondre
#38
Je suis arrivé au meme conclusion que jpg54 et Mathieu. La première a été nickel en suivant le workflow d'Aurélien
La seconde me pose énormément de soucis et jouer avec les pipettes n'arrange rien (version 2.5.0+844-g47c68b30b)
je vais tenter en redémarrant dt et voir s'il réinitialise le module
Répondre
#39
En principe, oui ça remet en place mais il faut que tu remontes dans l'historique et que tu le comprimes.
Répondre
#40
La solution pour le moment sélectionner et désélectionner une pipette, ensuite le mode auto refonctionne.
--
Pascal - GNU/Debian (sid) - version darktable git/master
http://photos.obry.net
Répondre


Atteindre :


Utilisateur(s) parcourant ce sujet : 2 visiteur(s)