OPI RVB / RVB + IR / IR part 2

[gmaillet]

Motivation

Pouvoir créer un cache avec soit:

• des OPI RVB
• des OPI RVB et IR
• des OPI IR

Principe

Comme GDAL ne gère pas correctement les COG Jpeg à 4 canaux (le 4ième canal est géré comme un masque binaire), on crée des COG RVB et des COG IR.

Cette PR est la suite de #245 avec les modifications dans l'API et dans iTowns.

[coveralls]

Pull Request Test Coverage Report for Build 2167684596

• 2022 of 2035 (99.36%) changed or added relevant lines in 12 files are covered.
• 1 unchanged line in 1 file lost coverage.
• Overall coverage increased (+0.3%) to 95.13%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
middlewares/wmts.js	33	34	97.06%
gdal_processing.js	138	150	92.0%
<!--	Total:	2022	2035	99.36%	-->

Files with Coverage Reduction	New Missed Lines	%
gdal_processing.js	1	91.79%
<!--	Total:	1		-->

Totals
Change from base Build 2165928414:	0.3%
Covered Lines:	5992
Relevant Lines:	6272

---

💛 - Coveralls

[amrosu]

A mettre à jour la doc

[gmaillet]

ok, je m'en charge

[ftoromanoff]

ReadMe Ligne 55 il manque un 'run' : npm run start-dev

[gmaillet]

Rebase à faire

rebase fait

[gmaillet]

La mise à jour d'un cache RVB avec les images IR et inversement n'est pas géré correctement - à faire

Actuellement :

• pour un "update" d'un cache IR en ajoutant les images RVB, calcul sans erreur, mais pas correct :

  • dans opi, il semble y avoir les dalles IR et RVB, mais dans ortho - uniquement des IR
  • dans itowns, rien dans la couche RVB ni pour l'ortho, ni pour l'opi dans la vue globale ; dans l'opi, on voit le RVB si on sélectionne une OPI
• pour un update d'un cache RVB avec des IR, le calcul échoue avec message d'erreur "GDALDataset::GetRasterBand(2) - Illegal band"

Pour le moment, il n'est pas autorisé de mélanger les types dans un cache. J'ai ajouté un test dans lors de mise à jour d'un cache.

[gmaillet]

ReadMe Ligne 55 il manque un 'run' : npm run start-dev

fait

[gmaillet]

A mettre à jour la doc

fait

[nlenglet-ign]

Deux petites remarques de mon côté :

• la nomenclature des dalles IR d'ortho : en prenant pour exemple une dalle dont le nom va être AA -> AAi.tif, si on applique un batch sur la branche n, cela devient n_AAi.tif, si on en a un deuxième, n_AA_1i.tif. Je pense qu'il serait peut être plus cohérent de réussir à garder le nom de la dalle tout du long.
• quand on a un cache IR seul, l'affichage sélectionné par défaut sous iTowns est le RGB, ce qui peut donner l'impression qu'il n'y a pas de données. Ces deux points n'ont rien de bloquant de mon point de vue.

[amrosu]

Dans le cas où on choisit un style qui n'est pas cohérent avec le cache (le type d'OPI n'existe pas dans le cache), alors il serait bien d'afficher un message en indiquant que le style choisi n'existe pas - actuellement rien n'est mis en place pour ce cas de figure, même pas dans la console du navigateur

[gmaillet]

Pour les caches IR, après avoir choisi style Ortho = IR et style OPI = IR, la sélection d'OPI ne fonctionne pas correctement : l'action est faite côté serveur (on voit le nom de l'OPI sélectionnée dans le menu, dans le champ "OPI selected"), mais pas sur la carte.

Effectivement, il y avait une faille dans le cas de cache IR au niveau de l'API, c'est corrigé dans gdal_processing.js

ans le cas où on choisit un style qui n'est pas cohérent avec le cache (le type d'OPI n'existe pas dans le cache), alors il serait bien d'afficher un message en indiquant que le style choisi n'existe pas - actuellement rien n'est mis en place pour ce cas de figure, même pas dans la console du navigateur

J'ai modifié le comportement: maintenant le choix du style ne se fait que sur des compositions colorées valides pour le cache en cours (suivant les cas: RVB ou RVB/IRC/IR ou IR) et par défaut, on affiche au départ un style valide pour le cache (RVB ou IR suivant les cas).