New alerts - Githubissues

ign-packo / PackO

Outil pour le contrôle et la retouche du mosaïquage d'ortho

Other

6 stars 2 forks source link

New alerts #268

Closed ftoromanoff closed 2 years ago

ftoromanoff commented 2 years ago

Refonte des propriétés des couches d'alerte.

Modifications :

Ajout d'un statu 'checked' en addition de 'unchecked' et 'validated' avec la possibilité de passer de l'une aux autres
Ajout de la visualisation du nombre de 'checked' et de 'validated' ainsi que du nombre totale d'entités
Ajout de la propriété 'id' permettant la navigation direct vers une entité spécifiée
Affichage des couches 'Extra Layer' a tout les niveaux ~~Limitation pour la couche d'alerte pour des raisons d'affichage itowns ( à résoudre)~~
Lors de la sélection d'une couche d'alerte on centre sur la dernière entité visualisée

Autre modifications :

rafraichissement des couches d'intérêt uniquement (quand nécessaire)
refactorisation d'un bout de code en lien avec les contrôleurs

coveralls commented 2 years ago

Pull Request Test Coverage Report for Build 1819066132

1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.159%

Totals
Change from base Build 1751027278:	0.0%
Covered Lines:	5039
Relevant Lines:	5338

💛 - Coveralls

ftoromanoff commented 2 years ago

Il y a un problème sur les alertes linéaires, elles ne changent pas de couleurs lors des changements de statut (unchecked, checked, validated)

Corrigé. J'avais fais une issue sur itowns pour modifier la gestion des points ligne et polygone. Pour le moment il n'y a pas moyen de modifier le contour d'un polygone et l'affichage de linéaires de manière différentiels

amrosu commented 2 years ago

Il y a un problème sur les alertes linéaires, elles ne changent pas de couleurs lors des changements de statut (unchecked, checked, validated)

Corrigé. J'avais fais une issue sur itowns pour modifier la gestion des points ligne et polygone. Pour le moment il n'y a pas moyen de modifier le contour d'un polygone et l'affichage de linéaires de manière différentiels

Le problème sur les alertes linéaires semble résolu.

amrosu commented 2 years ago

La correction doit être faite dans cette branche

ftoromanoff commented 2 years ago

La correction doit être faite dans cette branche

Corrigé

amrosu commented 2 years ago

OK pour la correction de ce bug

ftoromanoff commented 2 years ago

Un autre problème : il y a une incohérence entre le nombre d'objets qui composent les vecteurs ZU et NON_ZU (un seul objet chacun) et le nombre d'objets affichés dans Progress : il trouve 20 objets pour ZU et 100, pour NON_ZU, mais on voit que les actions sur les statuts sont faites comme si on avait un seul objet

J'ai lancé une issue sur le github d'itowns à ce sujet. En attendant il faut limiter l'import de couche contenant des polygones multipart.

amrosu commented 2 years ago

Mettre à jour la doc en ajoutant l'information sur le fait que pour le moment les vecteurs multipolygones ne sont pas gérés par itowns

amrosu commented 2 years ago

La liste des vecteur d'alertes garde l'ordre de chargement des vecteurs et donc un vecteur d'alertes peut se trouver sous un ou plusieurs autres vecteurs et alors les statuts des entités (les couleurs correspondantes au changement de statut) sont difficilement ou pas visibles. Il serait donc bien de mettre toujours en avant (au-dessus de tous les autres vecteurs) le vecteur de la couche d'alertes active - on verra si ce besoin sera remonté par l'utilisateur

Mis sous forme de ticket (#274)

amrosu commented 2 years ago

Je viens de pousser un commit (a613060) qui gère le style des alertes dans le css, comme je le demandais plus haut

ftoromanoff commented 2 years ago

J'ai mergé les commits et ajouté une ligne dans le ReadMe