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

cuSpatial Polygonize Notebook #1233

Draft
wants to merge 1 commit into
base: branch-23.08
Choose a base branch
from

Conversation

jarmak-nv
Copy link
Contributor

@jarmak-nv jarmak-nv commented Jul 27, 2023

Description

This PR adds a simple raster -> vector polygonization notebook.

  • Each pixel is converted into a square polygon
  • A GeoDataFrame is made from the polygons
  • Data is read from the Raster and converted to cuDF columns

This notebook serves to be a test-showcase for cuProj. It uses Sentinel-2 imagery and each tile generates about 120M polygons.

The cuML section is largely for testing/showcasing another RAPIDS integration but I don't feel it's necessary (esp since it needs work.)

Regardless of status on the reviews, the has been applied until legal approval. It may be better for us to put this in notebook-contrib as well to prevent rasterio from being a notebook dependency.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@jarmak-nv jarmak-nv added the 5 - DO NOT MERGE Hold off on merging; see PR for details label Jul 27, 2023
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@github-actions github-actions bot added the Python Related to Python code label Jul 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5 - DO NOT MERGE Hold off on merging; see PR for details Python Related to Python code
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

None yet

1 participant