Feature request #3437
Add natgrid to IGNF's RGF93 projections
Status: | Closed | ||
---|---|---|---|
Priority: | Low | ||
Assignee: | Jürgen Fischer | ||
Category: | Projection Support | ||
Pull Request or Patch supplied: | Resolution: | invalid | |
Easy fix?: | No | Copied to github as #: | 13497 |
Description
Hi,
This ticket follows #3394. The official IGNF register does not specify the use of the gsb natgrid in the proj's description of the RGF93's systems because the gris was intented to convert from NTF to RGF93.
However this grid is also useful when reprojecting or using OTF's projection of RGF93 layers on a NTF basemap, reducing the diff to cm instead of ~50m.
History
#1 Updated by jc - over 13 years ago
- Status changed from Open to Closed
- Resolution set to invalid
WTF ?
ndgrids are elaborate towgs84 clause. And RGF93 +towgs84 clauses are right 0,0,0, because RGF93 is WGS84 compliant.
While errors could be present on IGNF registry published for many years now, it's quite doubtful.
And also, why change the IGNF registry here, on QGIS and not on Proj.4 project ?
#2 Updated by Jean-Roc Morreale over 13 years ago
sorry if it wasn't clear, I'm not saying there is errors in the IGNF as the proj definitions work as attented when working between RGF93 based layers or when projecting NTF layers to RGF93. However, some users needs to do the reverse job (RGF93 to NTF, not talking WG84) and with the official defs the natgrid isn't used and so cause a 50m error.
Please have a look to this thread (french), there is a use case and files to test :
http://www.forumsig.org/showpost.php?p=257466&postcount=24
About proj4, a ticket and a patch for the updated and official register have been submitted on trac (#88) 3 months ago but is still pending and its avaibality will depend on a proj4's release (this the reason behind #3394, pending too). About #3437, as it is an unofficial modification I didn't wanted to push it first into proj4.
#3 Updated by jc - over 13 years ago
This french thread show there is a projection problem, but the solution can't be this patch, else WGS84 data won't mismatch. I talk WGS84 because it's the proj4 reference.
I read your file a_lire.txt in your package test_qgis_l93.zip and I see only a +wktext addition (also in your forum comment), not any +nadgrids add. Is there a mistake in your a_lire.txt file ?
#4 Updated by Jean-Roc Morreale over 13 years ago
in my test, +wktext was enought to call the use of the grid. If you've a valid solution, it would be welcome.
#5 Updated by jc - over 13 years ago
To "call the use of the grid" or to "have good results" ?
If +wktext is enough on your tests, why add here +nadgrids ? Have you got tests with the addition you propose here or only with the +wktext clause addition alone ?
I just wonder why you test a patch and propose here an other patch apparently not tested. With your tests published on forumsig.org, I agree with a patch with +wktext alone.
#6 Updated by Jean-Roc Morreale over 13 years ago
Bon, en français ça évitera de continuer plus qu'il n'est nécessaire, n'étant pas dev ou quoi que ce soit j'ai cru bon de préciser la grille appelée par wktext, le test a été fait avec le fichier SQL joint ici et comme j'obtenais le résultat attendu (à savoir une couche en lambert zone projetée à la volée dans un projet utilisant une zone ntf et sans décalage de 50m) j'ai ouvert ce ticket. Si tu estimes que cette option n'est pas nécessaire, soit, je te laisse le soin de reproposer un patch ici ou sur le trac de proj4.
#7 Updated by jc - over 13 years ago
L'anglais avait l'avantage que d'autres intervenants simple lecteur pouvait aussi nous suivre, mais perso le français ça ne me dérange pas, c'est plus simple :)
Je vais reprendre les tests de mon côté. Ce n'est peut-être qu'une maladresse dans la façon de présenter le patch et qu'effectivement la modif est bonne. Je n'avais fait que regarder le test publié sur forumSIG. Je regarde de plus près et donne des nouvelles.
Le problème est surtout que je trouve la modif a priori plus néfaste que non nécessaire. Je reviens ici dès que j'ai fait mes tests de mon côté, il y a surement une modif à faire vu les tests qui ont déjà été fait.