ign-packo / PackO

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

Replace jimp by gdal #370

Closed amrosu closed 1 year ago

amrosu commented 1 year ago

Cette PR fait suite et remplace la PR #349

Remplacer jimp avec gdal semble bien améliorer les performances de l'API, notamment dans le cas d'une charge importante - plusieurs clients au même moment

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 3618834401


Changes Missing Coverage Covered Lines Changed/Added Lines %
middlewares/wmts.js 4 5 80.0%
gdal_processing.js 63 78 80.77%
<!-- Total: 67 83 80.72% -->
Files with Coverage Reduction New Missed Lines %
gdal_processing.js 2 88.62%
<!-- Total: 2 -->
Totals Coverage Status
Change from base Build 3618457860: 0.02%
Covered Lines: 5935
Relevant Lines: 6137

💛 - Coveralls
amrosu commented 1 year ago

Effet de bord détecté dans le client itowns : les vecteurs annexes se chargent bien, mais visibles seulement si ils sont des geojson EPSG:2154 ; la caméra semble bien se placer dans tous les cas. On peut peut-être accepter cette régression dans cette PR car c'est une fonctionnalité pas utilisée par l'utilisateur principal et faire un ticket pour la traiter ultérieurement.