Redefine FileWidget using Tray Publisher

[ccaillot]

Linked to #27

Include

• One Widget which manage Representations and Reviews

Setting

You can enable/disable to authorize reviews in settings : project_settings/traypublisher/simple_creators

[ccaillot]

@hfarre, je te laisse tester

[hfarre]

Small visual retake if possible:

Avoid to have a scroll bar if there are to many autorized extension.

Maybe put a return to the line in case there are a certain number of extension ?

[hfarre]

I created an image like this:

But I got this once it is created, is this normal ?:

[hfarre]

Error found as I did as follow:

I drag and drop 2 times the same file. And drag it one under the other

As I published, I got:

[ccaillot]

@hfarre tout semble etre fixé, je te laisse pull les changements et re-tester

[hfarre]

Parfait, je m'en vais tester ça !

[hfarre]

Lors d'un publish classique d'une image en Repr/Review, il y a une erreur dans l'integrate asset:

[hfarre]

J'ai une question de fonctionnement
J'ai glissé l'élément 1, puis desactivé l'oeil, il est donc passé en violet.
Je glisse l'élément 2 pour en faire l'element de review:

Mais l'élément 1 passe en orange, ce qui signifie que 1 sera review+repre et 2 sera uniquement review ?
Mais si je veux 1 en repre, et 2 en review, 1 ne devrait il pas être en violet du coup ?

[ccaillot]

J'ai une question de fonctionnement J'ai glissé l'élément 1, puis desactivé l'oeil, il est donc passé en violet. Je glisse l'élément 2 pour en faire l'element de review:

Mais l'élément 1 passe en orange, ce qui signifie que 1 sera review+repre et 2 sera uniquement review ? Mais si je veux 1 en repre, et 2 en review, 1 ne devrait il pas être en violet du coup ?

Effectivement, il y a un petit soucis de comprehension de mon cote, je fix ça

[BenSouchet]

J'ai une question de fonctionnement J'ai glissé l'élément 1, puis desactivé l'oeil, il est donc passé en violet. Je glisse l'élément 2 pour en faire l'element de review:

Mais l'élément 1 passe en orange, ce qui signifie que 1 sera review+repre et 2 sera uniquement review ? Mais si je veux 1 en repre, et 2 en review, 1 ne devrait il pas être en violet du coup ?

Selon moi il y a un changement à faire au niveau des couleurs, cela devrait être
Violet = representation
Orange = Represention & review (same element) donc quand l'oeil est coché
Jaune = Review

Ce qui fait une representation est toujours violet, même avec un enfant review jaune, et ne passe en orange que lorsqu'elle n'a pas d'enfant et que c'est elle même ça propre review.

A voir si ça semble plus cohérant avec cette idée

[BenSouchet]

Ou sinon la legende de Orange doit être: Representation with attached review(s)

[hfarre]

Dans ce cas, il faudrait peut être une 4eme couleur ?

• Repre file (no reviewable) with one or more reviewable children

[BenSouchet]

Dans ce cas, il faudrait peut être une 4eme couleur ?

• Repre file (no reviewable) with one or more reviewable children

A voir mais ça va commencer à faire arbre de noel, je pense que max 3 couleurs c'est bien

[ccaillot]

All good ?

[ccaillot]

Lors d'un publish classique d'une image en Repr/Review, il y a une erreur dans l'integrate asset:

J'ai testé de publié avec jpeg aucun soucis, par contre avec un png. Les Anatomy template de l'integrateAsset nécessite d'etre mis a jour afin de pouvoir gérer cette host : ''traypublisher" ou sinon mettre a jour les settings afin qu'ils pointent sur un template qui existe.

DEBUG: Source: C:\Users\ccaillot\Pictures\lookmanager.png
DEBUG: Creating new version ...
DEBUG: Prepared version: v005
DEBUG: Anatomy template name: comp_publish
CRITICAL: Error when registering
Traceback (most recent call last):
  File "C:\Users\ccaillot\quad\quadpype\src\quadpype\plugins\publish\integrate.py", line 185, in process
    self.register(instance, file_transactions, filtered_repres)
  File "C:\Users\ccaillot\quad\quadpype\src\quadpype\plugins\publish\integrate.py", line 273, in register
    prepared = self.prepare_representation(
  File "C:\Users\ccaillot\quad\quadpype\src\quadpype\plugins\publish\integrate.py", line 643, in prepare_representation
    publish_template_category = anatomy.templates[template_name]
KeyError: 'comp_publish'

project_settings/global/tools/publish/template_name_profiles

Aucun des profiles présent dans project_anatomy/templates/publish ne match avec comp_publish. Problème logique.

Ce problème n'est donc pas en lien avec la feature présente dans cette PR.

[hfarre]

Lors d'un publish classique d'une image en Repr/Review, il y a une erreur dans l'integrate asset:

J'ai testé de publié avec jpeg aucun soucis, par contre avec un png. Les Anatomy template de l'integrateAsset nécessite d'etre mis a jour afin de pouvoir gérer cette host : ''traypublisher" ou sinon mettre a jour les settings afin qu'ils pointent sur un template qui existe.

DEBUG: Source: C:\Users\ccaillot\Pictures\lookmanager.png
DEBUG: Creating new version ...
DEBUG: Prepared version: v005
DEBUG: Anatomy template name: comp_publish
CRITICAL: Error when registering
Traceback (most recent call last):
  File "C:\Users\ccaillot\quad\quadpype\src\quadpype\plugins\publish\integrate.py", line 185, in process
    self.register(instance, file_transactions, filtered_repres)
  File "C:\Users\ccaillot\quad\quadpype\src\quadpype\plugins\publish\integrate.py", line 273, in register
    prepared = self.prepare_representation(
  File "C:\Users\ccaillot\quad\quadpype\src\quadpype\plugins\publish\integrate.py", line 643, in prepare_representation
    publish_template_category = anatomy.templates[template_name]
KeyError: 'comp_publish'

project_settings/global/tools/publish/template_name_profiles

Aucun des profiles présent dans project_anatomy/templates/publish ne match avec comp_publish. Problème logique.

Ce problème n'est donc pas en lien avec la feature présente dans cette PR.

Hum, je pense comprendre en effet

Mais c'est curieux, pour mon test, j'ai fait une famille "image" en task "DA" qui est pourtant bien setté ici:

[hfarre]

All good ?

Parfait, ça me parait beaucoup plus clair comme ça !
je vais quand même voir en situation de test

[BenSouchet]

Deux dernière choses:

1. Pour la légende Orange je mettrais: Representation & review (same file) elements

2. Et j'ajouterais un QLabel en dessous du Treewidget et avant les labels Allowed File types avec comme texte: <i>Drag & drop to add or re-organize elements</i>

(la balise <i></i> c'est pour l'italique)

[BenSouchet]

Et sinon super le taff !

[hfarre]

petit bug:

J'ai glissé le fichier abc en premier, il est donc en orange.
Je lui glisse une image dedans pour en faire son review element:

Et quand je clique sur create, il me donne ça:

Y a plus l'image

[ccaillot]

petit bug:

J'ai glissé le fichier abc en premier, il est donc en orange. Je lui glisse une image dedans pour en faire son review element:

Et quand je clique sur create, il me donne ça:

Y a plus l'image

ma faute, c'est réparé

[hfarre]

Yep good for me

[ccaillot]

Merge in release/4.0.20