Skip to content

Add to_geo_feature to get properties HashMap with geo::Geometry #246

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

Merged
merged 5 commits into from
May 3, 2025

Conversation

michaelkirk
Copy link
Member

Related to #245 and another comment by @frewsxcv on discord, it seems there's some interest in parsing various data sources to geo::Geometry + properties. Here's my attempt!

This works for any FeatureProcessor — like GeoJson, Flatgeobuf, Shapefile, etc.

This works for any FeatureProcessor — like GeoJson, Flatgeobuf,
Shapefile, etc.
Copy link
Member

@frewsxcv frewsxcv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for adding this!

@frewsxcv frewsxcv added this pull request to the merge queue May 3, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 3, 2025
@frewsxcv frewsxcv added this pull request to the merge queue May 3, 2025
@frewsxcv frewsxcv removed this pull request from the merge queue due to a manual request May 3, 2025
@frewsxcv frewsxcv enabled auto-merge May 3, 2025 13:29
@frewsxcv
Copy link
Member

frewsxcv commented May 3, 2025

Regarding that last commit, it's because the debug representation changed in georust/geo#1332

@frewsxcv frewsxcv added this pull request to the merge queue May 3, 2025
Merged via the queue into main with commit 79fbc26 May 3, 2025
2 checks passed
@frewsxcv frewsxcv deleted the mkirk/geo-feature branch May 3, 2025 14:10
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.

2 participants