Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor projection #403

Merged
merged 9 commits into from
Oct 7, 2024
Merged

Refactor projection #403

merged 9 commits into from
Oct 7, 2024

Conversation

nlenglet-ign
Copy link
Collaborator

@nlenglet-ign nlenglet-ign commented Jul 20, 2023

Ajout des projections nécessaires à l'utilisation en production

Mise à jour des paramètres par défaut dans overviews comme demandé :

  • taille tuile : 512x512 px au lieu de 256x256
  • taille dalle : 8x8 tuiles au lieu de 16x16

Mise à jour swagger en correspondance

Mise à jour de la doc et des tests

@nlenglet-ign nlenglet-ign added the wip Travail en cours (ne pas merger) label Jul 20, 2023
@coveralls
Copy link

coveralls commented Jul 20, 2023

Pull Request Test Coverage Report for Build 5715493716

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 22 of 22 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 95.964%

Totals Coverage Status
Change from base Build 5398217406: 0.0%
Covered Lines: 5939
Relevant Lines: 6141

💛 - Coveralls

@nlenglet-ign nlenglet-ign removed the wip Travail en cours (ne pas merger) label Aug 3, 2023
Copy link
Collaborator

@amrosu amrosu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Il manque la mise mise à jour de l'aide pour create_cache, notamment la valeur par défaut pour le param "overviews" qui fait référence à LAMB93_5cm.json qui n'existe plus.
Il faut aussi aussi à jour la doc README

@nlenglet-ign nlenglet-ign added wip Travail en cours (ne pas merger) and removed wip Travail en cours (ne pas merger) labels Aug 3, 2023
Copy link
Collaborator

@amrosu amrosu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Il faudrait s'assurer de la rétrocompatibilité LAMB93 <-> RGF93_LA93, notamment pour permettre de continuer à travailler sur des caches créés avant le changement et qui contiennent donc la "vielle" nomenclature LAMB93.

Dans la base de données, comme on enregistre le crs avec code EPSG, ça devrait être bon, pas d'impact suite au changement de nomenclature pour LA93

@nlenglet-ign nlenglet-ign added the wip Travail en cours (ne pas merger) label Oct 12, 2023
@nlenglet-ign
Copy link
Collaborator Author

nlenglet-ign commented Nov 14, 2023

Tests réalisées pour la rétrocompatibilité LAMB93 <-> RGF93_LA93 :

  • itowns OK (vérifier avec plus de logs)
  • export vue QGis OK (test saisie à faire)
  • soucis côté doc API lors de requêtes WMTS de type GetTile / GetFeatureInfo

@amrosu
Copy link
Collaborator

amrosu commented Nov 17, 2023

Le problème ave GetTile/GetFeatureInfo dans la doc vient de TILEMATRIXSET qui est sur la forme d'une liste contenant les nouveaux identifiants de la pyramide (et donc pas compatible avec les vieux cache contenant les vieux identifiants). Pour corriger cela, je propose de passer à un champ libre - commit 4d92df7.
J'ai également corrigé une petite coquille dans l'exemple POST/ cache - commit 9c93ff4

@nlenglet-ign
Copy link
Collaborator Author

Les saisies sont OK sous QGis.
Le comportement attendu dans swagger est le bon également.

@nlenglet-ign nlenglet-ign removed the wip Travail en cours (ne pas merger) label Nov 21, 2023
@amrosu amrosu added the wip Travail en cours (ne pas merger) label Jun 26, 2024
@amrosu
Copy link
Collaborator

amrosu commented Jun 26, 2024

J'ai remplacé les LAMB93->RGF93_LA93 restants dans la doc et les fichiers des tests regress, plus rebase de la branche

@amrosu
Copy link
Collaborator

amrosu commented Jul 2, 2024

Avec le changement de la taille de la tuile (512x512 px au lieu de 256x256 px), on retrouve le problème d'incohérence dans l'affichage des niveaux de zoom avec le client itowns décrit dans le ticket #358 :

[...]
Problème du client itowns : il semble y avoir un décalage entre le 'level' (et donc le 'levelCOG') demandé et celui affiché si la tuile est différente de 256. Le décalage est de 1 niveau (level) si taille tuile = 512 et de 2 niveaux si tuile=1024.

Actuellement, dans le client itowns, le niveau maximal d'affichage du graphe/contour et de saisie est lié à la taille de la dalle (ex: Dezoom 1/16 pour dalles de 16x16, 1/8 pour 8x8 et 1/4 pour 4x4). L'affichage actuel dans itowns du graphe/contour est 'décalé' pour des caches avec des tuiles différentes de 256x256, voir plus haut, avec des incohérences avec le message 'Graph not displayed', message qui n'est pas dépendant de 'level'.
[...]

Dans ce même ticket on avait conclu de ne pas corriger ce problème dans le client itowns, car :

Actuellement, dans client itowns, pas de correction sur 'level', ni d'évolution sur le niveau max d'affichage de graphe/contour et de saisie ne sont demandées par l'utilisateur principal (voir PR #351) :

[...] ces corrections liées à iTowns peuvent être mises en attente parce que:
* il n'est pas acté que l'on utilisera effectivement des caches avec des tuiles d'une autre dimension que 256
* on a décidé de ne plus faire d'évolutions lourdes sur la partie iTowns pour l'instant

@amrosu
Copy link
Collaborator

amrosu commented Jul 2, 2024

Et pour info, le repère géodésique pour Mayotte changera bientôt selon le service de géodésie de l'IGN : RGM23 remplacera RGM04. Pour le moment, RGM23 n'est pas officielle, donc pas de code EPSG, ni de définition d'emprise ... D'ici-là, on gardera RGM04 pour Mayotte

@amrosu amrosu removed the wip Travail en cours (ne pas merger) label Jul 2, 2024
@amrosu amrosu added the wip Travail en cours (ne pas merger) label Sep 16, 2024
@amrosu amrosu removed the wip Travail en cours (ne pas merger) label Sep 16, 2024
Copy link
Collaborator

@gmaillet gmaillet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Validé, à merger

@amrosu amrosu merged commit 97b0163 into master Oct 7, 2024
4 checks passed
@amrosu amrosu deleted the refactor_projection branch October 7, 2024 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Modification taille des tuiles et dalles par défaut Ajout des projections pour utilisation dans PackO
4 participants