Skip to content

Fix README feature matrix to match codebase#1158

Merged
brendancol merged 1 commit into
masterfrom
issue-1155
Apr 1, 2026
Merged

Fix README feature matrix to match codebase#1158
brendancol merged 1 commit into
masterfrom
issue-1155

Conversation

@brendancol
Copy link
Copy Markdown
Contributor

Summary

  • Fix all ~20 broken hydrology file links (moved from xrspatial/*.py to xrspatial/hydro/*.py)
  • Add 10 missing functions to the feature matrix: fill, sink, flow_path (Dinf/MFD), watershed (Dinf/MFD), hand (Dinf/MFD), binary, validate_disaggregation
  • Fix backend support markers for viewshed (CuPy/Dask+CuPy fall back to CPU without RTX) and pycnophylactic (CuPy works via CPU fallback)
  • Add GeoTIFF/COG I/O link to nav anchor bar
  • Update function count from 100+ to 150+

Closes #1155

Test plan

  • Verify hydrology links resolve on GitHub (click a few)
  • Confirm new rows render correctly in the markdown table
  • Check nav anchor links scroll to the right sections

- Update all hydrology links from xrspatial/*.py to xrspatial/hydro/*.py
- Add missing functions: fill, sink, flow_path (Dinf/MFD), watershed
  (Dinf/MFD), hand (Dinf/MFD), binary, validate_disaggregation
- Fix viewshed backend markers (CuPy falls back to CPU without RTX)
- Fix pycnophylactic backend markers (CuPy works via CPU fallback)
- Add GeoTIFF/COG I/O link to nav bar
- Update function count from 100+ to 150+
@github-actions github-actions Bot added the performance PR touches performance-sensitive code label Apr 1, 2026
@brendancol brendancol merged commit cb5bc02 into master Apr 1, 2026
11 checks passed
@brendancol brendancol deleted the issue-1155 branch May 4, 2026 13:05
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.

README feature matrix out of sync with codebase

1 participant