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

Update 2D error calculation #49

Open
signehl opened this issue Nov 4, 2024 · 1 comment
Open

Update 2D error calculation #49

signehl opened this issue Nov 4, 2024 · 1 comment
Labels

Comments

@signehl
Copy link
Contributor

signehl commented Nov 4, 2024

In line 956 the error due to projection is calculated using proj -VS:

paste -d" " <(cut -d" " -f1,2 ./tmp/llxy.txt) <(cut -d" " -f3,4 ./tmp/llxy.txt | proj -VS ${PROJSTR} | grep Areal | column -t | sed s/\ \ /,/g | cut -d, -f4) > ./tmp/xy_err.txt

After proj v6 this function is deprecated, and there is, to my knowledge no direct function that gives "Areal" so this should probably be updated to match the newest versions of proj.

@signehl signehl added the update label Nov 4, 2024
@mankoff
Copy link
Member

mankoff commented Nov 4, 2024

Suggested fixes:

  1. I'm not sure what changed (I'm still running a proj locally which puts out Areal), but my output is
Meridian scale (h) : 0.97460025 ( -2.54 % error )
Parallel scale (k) : 0.97460025 ( -2.54 % error ) 
Areal scale (s): 0.94984565 ( -5.015 % error )

So if only Areal is missing in the updated proj, we could take the Parallel and Meridian and multiply the scales.

  1. Install, use, and cite the Planimeter tool https://geographiclib.sourceforge.io/C++/doc/Planimeter.1.html

  2. If this is in docker, we should be able to use whatever version of proj we want. Although if proj is just a dependency of grass (and we want an updated grass), then it could get messy, with a grass container calling a separate proj-only container.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants