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

Info label #25

Closed a-dewit closed 6 months ago

a-dewit commented 6 months ago
ngonthier commented 6 months ago

J'ai voulu tester de mon côté et j'ai l'erreur suivante.

AttributeError: type object 'QgsGeometry' has no attribute 'fromPoint' 
Traceback (most recent call last):
  File "C:\PROGRA~1/QGIS33~1.3/apps/qgis/./python/plugins\trainminator-infoLabel\TnT_MapCanvas.py", line 224, in mouseMoveEvent
    geo_pt =  QgsGeometry.fromPoint(QgsPoint(qgspointXY.x(), qgspointXY.y()))
AttributeError: type object 'QgsGeometry' has no attribute 'fromPoint'

Version de Python : 3.9.5 (tags/v3.9.5:0a7dcbd, May 3 2021, 17:27:52) [MSC v.1928 64 bit (AMD64)] Version de QGIS : 3.32.3-Lima Lima, 67d46100b5

Je suis la bête noir de TrainMinator je pense.

LMartelet commented 6 months ago

J'ai voulu tester de mon côté et j'ai l'erreur suivante.

AttributeError: type object 'QgsGeometry' has no attribute 'fromPoint' 
Traceback (most recent call last):
  File "C:\PROGRA~1/QGIS33~1.3/apps/qgis/./python/plugins\trainminator-infoLabel\TnT_MapCanvas.py", line 224, in mouseMoveEvent
    geo_pt =  QgsGeometry.fromPoint(QgsPoint(qgspointXY.x(), qgspointXY.y()))
AttributeError: type object 'QgsGeometry' has no attribute 'fromPoint'

Version de Python : 3.9.5 (tags/v3.9.5:0a7dcbd, May 3 2021, 17:27:52) [MSC v.1928 64 bit (AMD64)] Version de QGIS : 3.32.3-Lima Lima, 67d46100b5

Je suis la bête noir de TrainMinator je pense.

C'est un soucis de version de QGIS. La méthode fromPoint de la classe QgsGeometry n'ayant été ajouté qu'à la version 3.34. Avant il faut plutôt utiliser la méthode fromPointXY. Dans notre cas ça ne doit pas changer car ce qui est passé c'est un point sur la carte (X,Y) issus d'un event et pas un point ayant potentiellement une hauteur (Z) et la 4ème dimension (M). Je pense qu'il vaudrait mieux faire la modif dans le code car la version 3.34 de QGIS n'est pas encore la dernière version considérée comme stable.

LMartelet commented 6 months ago

Ce n'est peut-être que chez moi mais j'ai encore le problème avec les noms de classe très long (que ce soit sur cette branche ou sur la branche proto-diff).

CelestinHuet commented 6 months ago

Je m'aperçois aussi que là où sont affichés les labels, il ne faut pas mettre "Classe 2016" et "Classe 2019" dans le code, mais plutôt récupérer le vintage de la couche. Sinon, si on change de dates, on voit toujours 2016 et 2019

a-dewit commented 6 months ago

Les vintage sont récupérés et la taille max du QLabel est fixée -> à tester

CelestinHuet commented 6 months ago

Je vais être encore embêtant... Il faudrait aussi fixer la taille dans la fenêtre differential

CelestinHuet commented 6 months ago

Ou plus exactement mettre une taille minimale de 200 ? Ma fenêtre différential bouge encore un peu car sa largeur est plus faible que la fenêtre principale à cause du texte AttributSelectingToolsGroup

a-dewit commented 6 months ago

ha mais tu fais bien, j'ai été un peu vite ^^ Je fais les modifications

CelestinHuet commented 6 months ago

C'est bon, j'ai mis sur proto_diff !