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.

Sign up for GitHub

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

Jump to bottom

UTM to LL subtract overflow #7

Closed
ktmattson-alion opened this issue Apr 20, 2021 · 1 comment
Closed

UTM to LL subtract overflow #7

ktmattson-alion opened this issue Apr 20, 2021 · 1 comment

Comments

@ktmattson-alion
Copy link

this line in geo.rs let lambda0 = (((z - 1) * 6 - 180 + 3) as f64).to_radians(); fails for all UTM zones < 31

it should be let lambda0 = (((z as i32 - 1) * 6 - 180 + 3) as f64).to_radians(); or zone could be i32
@DaveKram
Copy link
Owner

DaveKram commented Apr 21, 2021 •
edited
Loading

Good catch. Fixed in 15e04c0

I'm going to bump the version to 1.3.1 and bump on crates.io

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@DaveKram @ktmattson-alion