esciencecenter-digital-skills / geospatial-python

Introduction to Geospatial Raster and Vector Data with Python
https://esciencecenter-digital-skills.github.io/geospatial-python/
Other
3 stars 0 forks source link

Reveiw the vector related eps #68

Closed rogerkuou closed 1 year ago

rogerkuou commented 1 year ago

Thoughts on improving the vector related eps

Ep7 vector data

  1. In the tip "how should I define my bounding box", use an online viewer, for example this one, to determine the bounding box.
  2. The point/line/polygon image seems to be redundant in this ep
  3. The two-step bounding box cropping seems to be redundant. We can put the cx function cropping as an exercise: let the participants read in all the data, read the clip/cx/clip_by_rectdocumentation, and do the same cropping by themselves.
  4. The existing line data seem to be redundant. And fixing the rotated line data was also not super interesting. Maybe this part can be replaced by some more practical techniques, e.g. using sjoin to query the soil type (a larger polygon) for crop parcel.

Ep8 crop raster data with vector data

  1. Visualizing the thumbnail with polygons seems to be redundant. One can directly jump to copping large raster with the bounding box and visualize polygons on the cropped raster.

  2. Explain the rio.clip function and demonstrate clipping with all polygons. Add an exercise to let students choose a subset of the polygons (e.g. choose a certain type of crop), and crop the same raster using the subsetted polygons

  3. The variable names are too long, and need to be improved. Use the term "crop" only for data cropping, but not for field plants.

  4. remove the waterway exercise

Ep9 Zonal stats

  1. Do the Zonal statistics per polygon, but not per type of crop
  2. Visualize the rasterized results
  3. Exercise: discuss why some polygon ids are missing in the rasterized data. (solution: some polygons are in the same pixel)
  4. Demonstrate zonal stats on non-missing polygons.
  5. exercise: design a way to make sure all polygons can get their statistics (solution: iterative way)
rogerkuou commented 1 year ago

Made new PRs in the upper flow: