ign-packo / PackO

Outil pour le contrôle et la retouche du mosaïquage d'ortho
Other
6 stars 2 forks source link

Createcache gpao v2 #327

Closed nlenglet-ign closed 1 year ago

nlenglet-ign commented 2 years ago

Version gpao de la création de cache proposée par Greg Contient également les modifications nécessaires pour prendre en compte les évolutions qui avaient eu lieu :

coveralls commented 2 years ago

Pull Request Test Coverage Report for Build 2933581693


Totals Coverage Status
Change from base Build 2711340139: 0.0%
Covered Lines: 6010
Relevant Lines: 6295

💛 - Coveralls
ftoromanoff commented 2 years ago

ReadMe :

create_cache.py : -r 1 : plein de verbose avec des json en format sting => illisible. utile ? Si oui les rendre plus lisible update_cache.py : même remarque

Les noms des images dans le graph doivent être exactement la même ou peut on ajouter/enlever un préfix ? => je n'ai rien trouvé dans la doc

Gestion d'images à 4 canaux en entrée ? python scripts/create_cache.py -I "D:\PackO_CreateCache\11_OPI/.tif" -R "D:\PackO_CreateCache\11_OPI/.tif" -o ressources/LAMB93_20cm.json -c cache_11OPI_RGBIR -g "D:\PackO_CreateCache\69_2020_mosaiquage_mtd_region.gpkg" -t '69_2020_mosaiquage_mtd_region' -r 1 > D:\PackO_CreateCache\tmp.txt => ERREURS ERROR 4: D:\PackO_CreateCache\11_OPI/20FD6925_ix00009_01064.tif: No such file or directory (11fois) ERROR 1: Destination dataset has 3 bands, but at least 4 are needed (bcp de fois)

nlenglet-ign commented 1 year ago

ReadMe :

* l223 : ne fonctionne pas

* l259: idem

Le readme a été mis à jour.

create_cache.py : -r 1 : plein de verbose avec des json en format sting => illisible. utile ? Si oui les rendre plus lisible update_cache.py : même remarque

Les sorties console ont été refondues.

Les noms des images dans le graph doivent être exactement la même ou peut on ajouter/enlever un préfix ? => je n'ai rien trouvé dans la doc

Pas de modification nécessaire de la part de l'utilisateur au niveau du nom des images.

Gestion d'images à 4 canaux en entrée ? python scripts/create_cache.py -I "D:\PackO_CreateCache\11OPI/.tif" -R "D:\PackO_CreateCache\11OPI/.tif" -o ressources/LAMB93_20cm.json -c cache_11OPI_RGBIR -g "D:\PackO_CreateCache\69_2020_mosaiquage_mtd_region.gpkg" -t '69_2020_mosaiquage_mtd_region' -r 1 > D:\PackO_CreateCache\tmp.txt => ERREURS ERROR 4: D:\PackO_CreateCache\11_OPI/20FD6925_ix00009_01064.tif: No such file or directory (11fois) ERROR 1: Destination dataset has 3 bands, but at least 4 are needed (bcp de fois)

Pas de gestion d'images 4 canaux.

ftoromanoff commented 1 year ago

Test GPAO:

Erreur job decoupage :

Commande : python D:\PackO\scripts\cut_opi.py -R D:\PackO_CreateCache\11_OPI_rvb\OPI_20FD6925x00007_00928.tif -c cache_gpao_RGB_gpao

Traceback (most recent call last): File "D:\PackO\scripts\cut_opi.py", line 6, in cache.cut_opi() File "D:\PackO\scripts\cache.py", line 156, in cut_opi with open(args.cache + '/overviews.json') as json_overviews: FileNotFoundError: [Errno 2] No such file or directory: 'cache_gpao_RGB_gpao/overviews.json' FIN

J'ai l'impression qu'il y a un problème de chemin relatif pour le dossier de cache. En faisant une bidouille, j'ai réussi à relancer les jobs en échecs.

ligne 156 et 199 de cache.py, on utilise args.cache + '/overviews.json', il faudrait donc que dans le create.json le args.cache soit le chemin absolu.

ftoromanoff commented 1 year ago

Test GPAO:

Erreur job decoupage :

Commande : python D:\PackO\scripts\cut_opi.py -R D:\PackO_CreateCache\11_OPI_rvb\OPI_20FD6925x00007_00928.tif -c cache_gpao_RGB_gpao

Traceback (most recent call last): File "D:\PackO\scripts\cut_opi.py", line 6, in cache.cut_opi() File "D:\PackO\scripts\cache.py", line 156, in cut_opi with open(args.cache + '/overviews.json') as json_overviews: FileNotFoundError: [Errno 2] No such file or directory: 'cache_gpao_RGB_gpao/overviews.json' FIN

J'ai l'impression qu'il y a un problème de chemin relatif pour le dossier de cache. En faisant une bidouille, j'ai réussi à relancer les jobs en échecs.

ligne 156 et 199 de cache.py, on utilise args.cache + '/overviews.json', il faudrait donc que dans le create.json le args.cache soit le chemin absolu.

semble corrigé

amrosu commented 1 year ago

Ne pas oublier de nettoyer les commits de la branche avant sa fusion