Skip to content

Comments

r.mapcalc: Use mask and not MASK in the doc#4999

Merged
petrasovaa merged 2 commits intoOSGeo:mainfrom
wenzeslaus:r_mapcalc-mask-not-MASK
Jan 29, 2025
Merged

r.mapcalc: Use mask and not MASK in the doc#4999
petrasovaa merged 2 commits intoOSGeo:mainfrom
wenzeslaus:r_mapcalc-mask-not-MASK

Conversation

@wenzeslaus
Copy link
Member

Uses mask or raster mask instead of MASK in the documentation. Where the raster called MASK needs to be accessed directly in an example, MASK is used as the assumed name mentioning that explicitly.

Updates r3.mapcalc in the same way which should be harmless since the wording does not explicitly mention the possibility of a different name. The text looks like a direct copy-paste from r.mapcalc, so that is preserved. The only improvement is that r3.mask is now linked instead of r.mask (in the 3D doc).

Uses mask or raster mask instead of MASK in the documentation. Where the raster called MASK needs to be accessed directly in an example, MASK is used as the assumed name mentioning that explicitly.

Updates r3.mapcalc in the same way which should be harmless since the wording does not explicitly mention the possibility of a different name. The text looks like a direct copy-paste from r.mapcalc, so that is preserved. The only improvement is that r3.mask is now linked instead of r.mask (in the 3D doc).
@github-actions github-actions bot added raster Related to raster data processing HTML Related code is in HTML module docs labels Jan 27, 2025
@petrasovaa petrasovaa added this to the 8.5.0 milestone Jan 28, 2025
@petrasovaa petrasovaa enabled auto-merge (squash) January 28, 2025 16:08
@petrasovaa petrasovaa merged commit 3650646 into OSGeo:main Jan 29, 2025
23 checks passed
@wenzeslaus wenzeslaus deleted the r_mapcalc-mask-not-MASK branch January 29, 2025 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs HTML Related code is in HTML module raster Related to raster data processing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants