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

Adding point data to test Krovak projection #116

Closed
wants to merge 4 commits into from

Conversation

jachym
Copy link
Contributor

@jachym jachym commented Jul 24, 2014

Just adding data, so that Krovak projection can be tested as well. Transformation is values is based on cs2cs (from original Proj4) and they should work. Command line test:

echo 12.806988 49.452262|cs2cs +init=epsg:4326 +to +proj=krovak +lat_0=49.5 +lon_0=24.83333333333333 +alpha=30.28813972222222 +k=0.9999 +x_0=0 +y_0=0 +ellps=bessel +pm=greenwich +units=m +no_defs +towgs84=570.8,85.7,462.8,4.998,1.587,5.261,3.56

The definition corresponds with czech projection aka EPSG:5514

@calvinmetcalf
Copy link
Member

I literally just added an auto test runner, so if you could add an empty commit it'll show us if it passes the tests.

@jachym
Copy link
Contributor Author

jachym commented Jul 25, 2014

Hi,

the issue was, that if I defined new projection,

      proj4.defs("EPSG:5514", "+proj=krovak +lat_0=49.5 +lon_0=24.83333333333333 +alpha=30.28813972222222 +k=0.9999 +x_0=0 +y_0=0 +ellps=bessel +pm=greenwich +units=m +no_defs +towgs84=570.8,85.7,462.8,4.998,1.587,5.261,3.56");

And than made instance of this projection, using

    var proj = proj4.Proj("EPSG:5514");

Then, if I wanted to create new instance with the same definition:

    var proj2 = proj4.Proj("EPSG:5514");

it gave different transformation results. Reason for it was, that the proj.datum was redefined with different values. And this set of commits creates tests for it and finally solves it.

@calvinmetcalf
Copy link
Member

sweet

@calvinmetcalf
Copy link
Member

932ca62

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.

None yet

2 participants