Migrate pharmacy-deserts example off datashader

Open
#3,462 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
1-2 days
Newbie friendliness
72/100
Issue type
Refactor
Clarity
Clearly specified
Activity status
Quiet
Tech stack
jupyter-notebook, matplotlib, python
Domain
data

Research direction

Open examples/pharmacy-deserts.ipynb and inspect the cells importing datashader and using Canvas, shade, stack, and inferno. Replace those operations with arr.xrs.rasterize and matplotlib as described, update prose references, then re-execute the notebook. Done means it runs top to bottom without a datashader import and produces the expected plots.

Written by the indexing model from the issue text.

Description

area:zonal documentation user-guide-example

Reason or Problem

examples/pharmacy-deserts.ipynb imports datashader and uses Canvas().points()/Canvas().polygons() with ds.mean/ds.max aggregation, plus shade/stack and the inferno colormap for rendering.

Proposal

  • Remove the datashader imports.
  • Replace the Canvas point/polygon rasterization with arr.xrs.rasterize(...), mapping the ds.mean/ds.max aggregation onto the column= and merge= parameters.
  • Replace shade/stack and the inferno colormap with matplotlib (matplotlib has an inferno colormap).
  • Update any prose that mentions datashader.
  • Re-execute the notebook so outputs regenerate cleanly.

Done when

  • The notebook runs top to bottom with no datashader import and produces the expected plots.

Part of removing datashader as a dependency; final packaging removal is tracked in #3450.

Dominant language
Python
Stars
972
Forks
92
Avg merge
2d 12h
Merged PRs (30d)
7

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from xarray-contrib/xarray-spatial

All issues in xarray-contrib/xarray-spatial

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.