Skip to content

r.horizon: height output feature#7119

Open
zst86 wants to merge 1 commit intoOSGeo:mainfrom
zst86:feature/r.horizon-map-units-output
Open

r.horizon: height output feature#7119
zst86 wants to merge 1 commit intoOSGeo:mainfrom
zst86:feature/r.horizon-map-units-output

Conversation

@zst86
Copy link
Copy Markdown

@zst86 zst86 commented Feb 24, 2026

Add an option to output a second map when running r.horizon in raster mode, which outputs horizon height in map units, useful for rockfall risk assessment. For example, a pixel below a vertical cliff has a 90° horizon in the azimuth direction of the cliff. This feature is intended to produce a second map that outputs the horizon height, so risk assessments can distinguish between a small or large cliff, which affects the kinetic energy of rockfalls.

Add an option to output a second map when running r.horizon in raster mode, which outputs horizon height in map units, useful for rockfall risk assessment
@github-actions github-actions bot added raster Related to raster data processing C Related code is in C module labels Feb 24, 2026
@petrasovaa petrasovaa changed the title r.horizon height output feature r.horizon: height output feature Feb 25, 2026
@petrasovaa
Copy link
Copy Markdown
Contributor

@zst86 it looks like there are conflicts that you need to resolve.

@zst86
Copy link
Copy Markdown
Author

zst86 commented Feb 27, 2026

Thanks Anna, I will look to resolve these conficts next week when I have time.

@wenzeslaus
Copy link
Copy Markdown
Member

This looks to me like your AI agent is using outdated code base to generate the code.

An update to the documentation, example, image, and a scientific reference will be helpful for the review and the users.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C Related code is in C module raster Related to raster data processing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants