Skip to content

Fix duplicate notebook numbers and restructure to standard format#1181

Merged
brendancol merged 2 commits into
masterfrom
issue-1180
Apr 11, 2026
Merged

Fix duplicate notebook numbers and restructure to standard format#1181
brendancol merged 2 commits into
masterfrom
issue-1180

Conversation

@brendancol
Copy link
Copy Markdown
Contributor

Closes #1180.

Summary

  • Fix duplicate notebook numbers: two notebooks shared prefix 48, four shared prefix 50. Now sequential 48-54, ordered by commit date.
  • Restructure 13 notebooks (38, 40-41, 44-49, 51-54) to match the standard cell sequence: title, "What you'll build" with anchor links, imports, data section, References.
  • Replace ax.imshow(data.values) with data.plot.imshow() in all affected notebooks.
  • Swap red/green colormap in 48_Sieve_Filter for teal/blue/orange.
  • Remove em dashes from prose.

Test plan

  • All 55 notebooks validate as JSON
  • Spot-check a few restructured notebooks render correctly in JupyterLab
  • Verify renamed notebooks sort correctly in file browser

Two notebooks shared prefix 48 and four shared prefix 50.
Renumber to a gap-free 48-54 sequence, ordered by commit date.
Notebooks 38, 40-41, 44-49, 51-54 were missing the standard cell
sequence (title, "What you'll build", imports intro, data section,
References). Each notebook now follows the established structure:

- Cell 0: h1 title in "Xarray-Spatial {module}: {tools}" format
- Cell 1: "What you'll build" with anchor-linked section list
- Cells 2-7: imports one-liner, imports, data header, data gen, description, plot
- Final cell: References with real URLs

Also fixed:
- ax.imshow(data.values) replaced with data.plot.imshow() throughout
- Red/green colormap in 48_Sieve_Filter swapped for teal/blue/orange
- Em dashes removed from prose
- Patch legends added to categorical overlays
@github-actions github-actions Bot added the performance PR touches performance-sensitive code label Apr 11, 2026
@brendancol brendancol merged commit 26e14bd into master Apr 11, 2026
11 checks passed
@brendancol brendancol deleted the issue-1180 branch May 4, 2026 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performance PR touches performance-sensitive code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix duplicate notebook numbers in user guide (two at 48, four at 50)

1 participant