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

Wrong SRID for Irish grid #510

Open
JimBacon opened this issue Mar 28, 2024 · 3 comments
Open

Wrong SRID for Irish grid #510

JimBacon opened this issue Mar 28, 2024 · 3 comments

Comments

@JimBacon
Copy link
Member

In modules/sref_osie/plugins/sref_osie.php the SRID is set as 29901 but it should be 29903, assuming that 'Irish Grid (TM75)' is intended.

Not really sure of the implications of this but to correct it might require reprojecting records currently in the database.

@johnvanbreda
Copy link
Contributor

Remarks on this page imply you are correct: https://epsg.io/29901. Also they imply that the difference may be very small (within the bounds of the accuracy of the calculation). Picking a random point for comparison implies the difference may be less than a metre, so maybe a transform is not necessary:

select st_astext(st_transform(st_geomfromtext('POINT(7.74561 52.64007)', 4326), 29901)),
st_astext(st_transform(st_geomfromtext('POINT(7.74561 52.64007)', 4326), 29903))

Using 29901 may have come from conversion from old Recorder 6 documentation about the Irish Grids.

@JimBacon
Copy link
Member Author

JimBacon commented Apr 3, 2024

Agreed. And the difference is smaller still if the example point is in Ireland ;-)
Very convenient.

johnvanbreda added a commit to Indicia-Team/media that referenced this issue May 22, 2024
johnvanbreda added a commit to Indicia-Team/client_helpers that referenced this issue May 22, 2024
johnvanbreda added a commit that referenced this issue May 22, 2024
@johnvanbreda
Copy link
Contributor

Switched SRID in the develop branch. Once deployed we should monitor in case the slight shift in grid squares causes problems in mapping (e.g. 2 nearly identical grid squares overlapping, rather than aggregating to one).

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

No branches or pull requests

2 participants