[bhuvan-somisetty]

Description

This PR implements the -i (invert) flag functionality in r.reclass.area:

• In reclass(): passed invert boolean flag and updated the inclusion logic:

  in_range = lower <= hectares <= upper
  if not in_range if invert else in_range:
      rules += "{} = {} {}\n".format(*[f[i] for i in [0, 2, -2]])

  Clumps with area between lower and upper are filtered out when -i is set, preserving clumps smaller than lower or greater than upper.
• Added complete translatable informational and fatal messages for each threshold permutation with invert=True.
• Handled mutual exclusions: added % excludes: -m, -i in parser rules (since -m prunes small clumps during clumping prior to inversion) and raised gs.fatal(_("Flag -i is not supported with method='rmarea'")) when combining -i with vector-based method='rmarea'.
• Documented -i behavior in r.reclass.area.md and r.reclass.area.html.
• Added unit tests for inverted filtering with lower, lower + upper, and validation for unsupported method='rmarea'.

Motivation and context

Fixes #8097

The -i flag was defined in the interface definition of r.reclass.area.py (% flag: i, % description: Invert selection (remove areas within size range)), but was never passed to reclass() or evaluated, leaving it completely inoperative.

How has this been tested?

• Ran ruff check scripts/r.reclass.area/ and ruff format --check scripts/r.reclass.area/ (all passed cleanly).
• Added unit tests in scripts/r.reclass.area/testsuite/test_r_reclass_area.py:
  • test_reclass_lower_invert: checks that clumps below the lower threshold are retained.
  • test_reclass_lower_upper_invert: checks that clumps outside the [lower, upper] range are retained.
  • test_rmarea_invert_unsupported: verifies proper module failure when -i is passed with method=rmarea.

Screenshots (if appropriate)

N/A

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as before)

Checklist

• PR title provides summary of the changes and starts with one of the pre-defined prefixes
• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.

Signed-off-by: bhuvan-somisetty somisettybhuvan5@gmail.com

[petrasovaa]

Thanks! There are couple small improvements still needed.

[bhuvan-somisetty]

Thanks for the review @petrasovaa! I've updated everything based on your feedback:

• Simplified the condition readability using keep.
• Documented that -i is only supported with the reclass method (and not rmarea).
• Updated the tests with distinct ranges (test_reclass_upper_invert) and verified that normal and inverted outputs have zero overlapping cells (n=0).

[petrasovaa]

Thanks!