search

ign-packo / PackO

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

style cache IR/RGB et overview #308

Open ftoromanoff opened 2 years ago

ftoromanoff commented 2 years ago

index.js autour de la ligne 106 :

overviews.with_rgb = true; overviews.with_ir = true; const tabOpi = Object.keys(overviews.list_OPI); if (tabOpi.length > 0) { overviews.with_rgb = overviews.list_OPI[tabOpi[0]].with_rgb; overviews.with_ir = overviews.list_OPI[tabOpi[0]].with_ir; }

Je ne comprend pas pourquoi on ne teste que la première image. Soit il y a une amélioration de prévu pour avoir des caches IR/RGB incomplets, soit comme dans le cas présent, les caches doivent avoir autant d'images RGB que IR (dans le cas ou il y a les deux). Dans ce cas, le paramètre with_rgb et with_ir doivent être un paramètre de l'overview (ce que tu fais en initialisant overview.with_ir = true et overview.with_rgb = true) et non pas de chacune des OPI. et ainsi on ne teste que overview.dataset.with_rgb et non le paramètre lié a la première image... (Les paramètres with_rgb et with_ir pourraient être un paramètre de overview.dataSet, si besoin de plus de clareté)

ftoromanoff commented 2 years ago

gmaillet commented:

effectivement c'était dans la PR#250. A plusieurs endroits j'ai fait l'hypothèse que le cache était homogène. Par exemple, il n'est pas possible de créer un cache en RVB et d'utiliser l'update pour y ajouter de l'IR. Pour le moment, je pense que tous nos codes (scripts python, API et iTowns) sont cohérents et safe. Par contre, tu as raison, si ce choix est définitif il serait intéressant de le faire remonter au niveau du overview.

ftoromanoff commented 2 years ago

Je propose d'ajouter une propriété style dans le fichier overview qui serait un tableau contenant les valeurs de style disponible. (qui remplacerait les valeurs with_ir et with_rgb) exemple style = [IR] pour un cache IR ou style = [RVB, IRC, IR] pour un cache RVBIR