IGNF / trainminator

Outil de labellisation développé en interne à l'IGN/DSI/(SDM et SIMV)
GNU General Public License v3.0
4 stars 0 forks source link

Tuilage pour l'avancement #51

Closed CelestinHuet closed 5 months ago

CelestinHuet commented 5 months ago

Pour que cela fonctionne, il faut dans Segmented_data la couche patches. Un projet correctement configuré se trouve dans store-echange/CelestinHuet/chantier_zone_1.

Lorsque l'on saisie un patch avec le menu Labeling, cela fait passer en vert le patch. Lorsque l'on saisie avec le menu Delete All ou Delete Current, cela le fait passer en rouge.

Faut-il modifier un peu la symbiologie pour la rendre plus claire ? Ce qui m'embête, c'est lorsqu'un patch vert est entouré de patches rouges (et inversement). Dans ce cas, ce n'est pas évident de le voir

ngonthier commented 5 months ago

Super pour cette première version. J'aurai deux questions et une remarque.

  1. Serait-il possible de vérifier si tout les segments de la dalle sont bien annoté avant de pouvoir valider la zone comme effectuée ? (Je me dis que la réponse doit être non car la dalle ne fait pas partie du dictionnaire de enfants que vous avez créer pour gérer cela).
  2. Serait-il possible de désactiver cette fonctionnalité si elle gène l'utilisateur ?

Par rapport à la symbologie pour que ce soit plus visible, quelque chose à tester pourrait être un masque gris complètement opaque pour ne pas confondre avec la labellisation qui a une certaine transparence.

LMartelet commented 5 months ago

Super pour cette première version. J'aurai deux questions et une remarque.

  1. Serait-il possible de vérifier si tout les segments de la dalle sont bien annoté avant de pouvoir valider la zone comme effectuée ? (Je me dis que la réponse doit être non car la dalle ne fait pas partie du dictionnaire de enfants que vous avez créer pour gérer cela).
  2. Serait-il possible de désactiver cette fonctionnalité si elle gène l'utilisateur ?

Par rapport à la symbologie pour que ce soit plus visible, quelque chose à tester pourrait être un masque gris complètement opaque pour ne pas confondre avec la labellisation qui a une certaine transparence.

J'ai pas encore regardé ce que ça donne mais ta première remarque ne me parait pas trop possible à faire comme on ne veut pas forcément annoter tous les segments d'un patch. A moins que tu pensais plutôt à vérifier que tous les segments qui ont été annotés aient une annotation conforme à l'attente (annoté sur les deux dates).

ngonthier commented 5 months ago

Oui effectivement je voulais dire qu'un segment devait être annoté aux deux dates (ou pas du tout).

LMartelet commented 5 months ago

Je trouve ça pour le moment très peu visible (surtout comparé à la saisie à côté qui est super flashy en terme de couleur). Je suis d'accord pour finalement avoir une symbo qui masque en gris opaque. Mais dans ce cas je pense que pouvoir afficher ou non la couche facilement doit être pris en compte. Techniquement on peut le faire en allant dans le groupe de couche mais ça pourrait induire des erreurs car il ne faut pas aller toucher les autres couches présentes au même endroit.

CelestinHuet commented 5 months ago

Concernant la symbologie, il faudrait l'afficher en gris opaque en permanence, ou bien seulement lorsque l'on est sur la couche dans le slider ?

Pour la vérification de la complétion, il faudrait vérifier non seulement lorsque l'on modifie le quadrillage, mais aussi à chaque fois que l'on modifie un segment (si on modifie un segment qui est dans une zone notée comme complète). C'est faisable, mais cela risque de ralentir un peu à chaque saisie

ngonthier commented 5 months ago

Oui je pense que l'idéal serait d'avoir un affichage en gris en permanence.

Que si le dallage est affiché en permanence, il n'y a besoin de valider qu'au moment ou l'utilisateur annoter un patch comme fini.

CelestinHuet commented 5 months ago

Les améliorations sont faites !

ngonthier commented 5 months ago

@LMartelet, lorsque tu auras reviewer cette pull request tu pourras la merger dans proto_diff et ensuite indiquer à Maxime que la branche peut être utiliser.

ngonthier commented 5 months ago

Juste pour clarifier si jamais quelqu'un vient lire cette conversation un jour. Le fichier de patch doit être à la racine et non dans Segmented_data.