Raccourci RVB/IR/IRC

[amrosu]

Mise en place du raccourci pour basculer l'ortho et l'opi entre RVB/IR/IRC

[coveralls]

Pull Request Test Coverage Report for Build 2195395221

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 95.13%

---

Totals
Change from base Build 2194542368:	0.0%
Covered Lines:	5992
Relevant Lines:	6272

---

💛 - Coveralls

[gmaillet]

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é)

(Je sais que c'es plus en lien avec la PR précédente mais je n'avais pas encore appréhendé correctement cette démarche)

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.