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

Implement a geoparquet writer #899

Merged
merged 6 commits into from
Nov 5, 2024
Merged

Implement a geoparquet writer #899

merged 6 commits into from
Nov 5, 2024

Conversation

bchapuis
Copy link
Member

No description provided.

@bchapuis bchapuis force-pushed the geoparquet-writer branch 2 times, most recently from 216481e to 59ec9fc Compare October 19, 2024 23:35
@bchapuis bchapuis changed the title Implement a geoparquet writer Implement a geoparquet writer (WIP) Oct 21, 2024
@bchapuis bchapuis force-pushed the geoparquet-writer branch 2 times, most recently from 32b3fb9 to 10c5388 Compare October 21, 2024 20:31
@bchapuis bchapuis changed the title Implement a geoparquet writer (WIP) Implement a geoparquet writer Oct 24, 2024
@bchapuis bchapuis requested a review from sebr72 October 28, 2024 15:58
Copy link
Contributor

@sebr72 sebr72 left a comment

Choose a reason for hiding this comment

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

Looks good. My comments are not compulsory. So I let you decide if you choose to merge.

Copy link

sonarcloud bot commented Nov 5, 2024

@bchapuis bchapuis merged commit f2fb634 into main Nov 5, 2024
8 checks passed
@bchapuis bchapuis deleted the geoparquet-writer branch November 5, 2024 20:54
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