Correction bug sur les caches IR où juste l'ortho n'était pas mise à jour dans la vue#450
Conversation
| gdal.setPROJSearchPath(projPath); | ||
| } | ||
|
|
||
| function findIRFromRVB(url) { |
There was a problem hiding this comment.
Je trouve que le nom n'est pas bien choisi. On ne cherche rien. La fonction permet de récupère le chemin d'accès de l'image IR quand on connait le nom du chemin d'accès de l'image RVB.
getIrUrl(RvbUrl) ?
There was a problem hiding this comment.
En plus on ne se base pas forcement sur l'url RVB, ou alors
if (fname.includes('_ix') === false) n'a pas de sens
Ce n'est pas plutot le passage de Opi name a band name , ou un truc dans le genre ?
There was a problem hiding this comment.
Modification du nom en gitIrUrl(rgbUrl)
There was a problem hiding this comment.
La condition permet juste de vérifier que le chemin mis en entrée n'est pas une OPI IR, pour ne pas rechanger le nom ex: RVB '000x000' -> IR '000_ix000' si IR en entrée sans condition '000_i_ix000' n'a plus de sens.
Cependant la condition peut être amélioré pour aussi vérifier que le chemin mis en entrée n'est pas une ortho IR, ex: RVB 'blabla.tif' -> IR 'blablai.tif' -> IR de IR 'blablaii.tif' pour évité ce cas on peut rajouter une condition sur le 'i.' qui ne doit pas être présent. D'après la nomenclature des noms des ortho ou opi on ne peut pas trouver un 'i.' dans le nom d'un image sauf dans le cas d'une ortho IR fait par Packo
There was a problem hiding this comment.
Non sera fait dans une autre PR.
ACornuIGN
added
wip
|
Suppression du commit de re-factorisation qui sera fait dans une autre PR. A cause de la complexité et de la profondeur de la re-factorisation dans le fichier gdal_processing.js vu avec François |
ACornuIGN
force-pushed
the
fix_update_ortho_IR
branch
from
72c3e8f to
1b192b9
Compare
ACornuIGN
added
wip
|
Nettoyage des commits + rebase sur 2Opi |
|
Erreur de manipulation PR merge avant nettoyage. |
2 participants
Bug présent sur master v2.4.2
Correction d'un bug sur les caches IR où juste l'ortho n'était pas mise à jour dans la vue après une saisie (le graph et le contour étaient bien mise à jour dans le vue), alors que l'ortho était bien calculée et présente dans le cache. C'était tout le temps l'ortho d'origine qui était afficher.
Le bug était dû lors de la récupération des tuiles par
GetTilede wmts qui vérifiait que la présence de nouvelles tuiles RVB et non IR.Correction en ajoutant à la vérification déjà existante, une vérification de la présence de nouvelles tuiles IR.