Skip to content

r.curvenumber: Add dual HSG support and null-soil fallback#1636

Merged
petrasovaa merged 5 commits intoOSGeo:grass8from
petrasovaa:r_curvenumber-dual
Mar 5, 2026
Merged

r.curvenumber: Add dual HSG support and null-soil fallback#1636
petrasovaa merged 5 commits intoOSGeo:grass8from
petrasovaa:r_curvenumber-dual

Conversation

@petrasovaa
Copy link
Contributor

Resolve dual hydrologic soil groups (A/D, B/D, C/D, D/D encoded as
11-14) to single groups based on drainage condition. Add -d flag to
select drained behavior (uses first group); default is undrained
(uses second group). Fall back to HSG D when soil is null but
landcover is valid. Add tests for both behaviors.

  Resolve dual hydrologic soil groups (A/D, B/D, C/D, D/D encoded as
  11-14) to single groups based on drainage condition. Add -d flag to
  select drained behavior (uses first group); default is undrained
  (uses second group). Fall back to HSG D when soil is null but
  landcover is valid. Add tests for both behaviors.
@petrasovaa petrasovaa marked this pull request as ready for review March 2, 2026 16:04
@petrasovaa petrasovaa requested a review from cwhite911 March 2, 2026 16:05
@petrasovaa petrasovaa requested a review from wenzeslaus March 5, 2026 04:05
Copy link
Contributor

@cwhite911 cwhite911 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@petrasovaa petrasovaa merged commit 5aa1252 into OSGeo:grass8 Mar 5, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants