[GRASS-git] [OSGeo/grass-addons] 2163cf: r.richdem: clarify resolveflats input; port dephie...

[Andy Wickert]

  Branch: refs/heads/grass8
  Home:   https://github.com/OSGeo/grass-addons
  Commit: 2163cf63360e0ce5bfae95f4815c63ffbb9dda5c
      https://github.com/OSGeo/grass-addons/commit/2163cf63360e0ce5bfae95f4815c63ffbb9dda5c
  Author: Andy Wickert <andy at northernwidget.com>
  Date:   2026-05-31 (Sun, 31 May 2026)

  Changed paths:
    M src/raster/r.richdem/r.richdem.dephier/testsuite/test_r_richdem_dephier.py
    M src/raster/r.richdem/r.richdem.fsm/testsuite/test_r_richdem_fsm.py
    M src/raster/r.richdem/r.richdem.resolveflats/r.richdem.resolveflats.py

  Log Message:
  -----------
  r.richdem: clarify resolveflats input; port dephier/fsm tests to step DEM (#1706)

* r.richdem.resolveflats: clarify input must be filled or breached

Closes awickert/r.richdem#2

Co-Authored-By: Claude Sonnet 4.6 <noreply at anthropic.com>

* r.richdem.dephier tests: port to saddle DEM

Replace the channel DEM (ring=3, outlet at right border) with the
shared saddle DEM (ring=9, pit=1, saddle=4) used by the fill and
breach test suites. Pour-point elevation is 5 (border cell adjacent
to the saddle), not 3. All structural assertions remain valid.

Closes awickert/r.richdem#1

Co-Authored-By: Claude Sonnet 4.6 <noreply at anthropic.com>

* r.richdem.fsm tests: port to saddle DEM

Replace the channel DEM (ring=3, outlet at right border) with the
shared saddle DEM (ring=9, pit=1, saddle=4). Pour-point elevation is
5 (border cell adjacent to the saddle). The wet-scenario assertion
(mean_after < mean_before) still holds: ring cells at elevation 9
with wtd=+1.5 give surface elevation 10.5 > pour-point 5, so water
spills and mean wtd decreases.

Co-Authored-By: Claude Sonnet 4.6 <noreply at anthropic.com>

* r.richdem dephier/fsm tests: rename saddle→step in DEM description

The cell at elevation 4 is not a geomorphological saddle (which would
require lower ground on two opposite sides). It is a step — an
intermediate elevation in the exit path pit(1)→step(4)→border(5).

Co-Authored-By: Claude Sonnet 4.6 <noreply at anthropic.com>

---------

Co-authored-by: Claude Sonnet 4.6 <noreply at anthropic.com>



To unsubscribe from these emails, change your notification settings at https://github.com/OSGeo/grass-addons/settings/notifications