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

Use geo_types more #46

Open
nms-scribe opened this issue Dec 27, 2023 · 0 comments
Open

Use geo_types more #46

nms-scribe opened this issue Dec 27, 2023 · 0 comments

Comments

@nms-scribe
Copy link
Owner

nms-scribe commented Dec 27, 2023

I originally chose not to work with geo_types due to lack of conversion with gdal. Gdal (in rust) provides mechanisms to convert into geo_types, but doesn't provide methods to convert the other way.

Then I ended up writing my own geometry classes that wrap gdal that look a lot more like geo_types, so I could have more type-safety.

And then I ended up importing geo and geo_types for their spherical mapping algorithms.

In any case, to simplify my code, I shgould just write a conversion going back to gdal, and then replace my geometry code with geo_types, where the conversion code actually converts instead of wraps. This will reduce potential error spots, since geo_types doesn't return a lot of errors from their functions.

This could also replace the coordinates module.

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

1 participant