gdal icon indicating copy to clipboard operation
gdal copied to clipboard

Treat LoS special case of exactly on terrain as visible

Open Ryanf55 opened this issue 7 months ago • 2 comments

What does this PR do?

Treat the special case of requesting line of sight from above to a point exactly at the same altitude as the DEM as visible.

That change breaks another test, I'll fix it then move this out of draft.

What are related issues/pull requests?

Closes #12458

Tasklist

  • [ ] AI (Copilot or something similar) supported my development of this PR
  • [ ] Make sure code is correctly formatted (cf pre-commit configuration)
  • [ ] Add test case(s)
  • [ ] Add documentation
  • [ ] Updated Python API documentation (swig/include/python/docs/)
  • [ ] Review
  • [ ] Adjust for comments
  • [ ] All CI builds and checks have passed
  • [ ] ADD YOUR TASKS HERE

Environment

Provide environment details, if relevant:

  • OS: Ubuntu 22
  • Compiler: GCC 11

Ryanf55 avatar May 25 '25 17:05 Ryanf55

GDALIsLineOfSightVisible_through_mountain needs to be adjusted and code formatting issue (https://github.com/OSGeo/gdal/actions/runs/15240459629/job/42859870241?pr=12460)

rouault avatar May 25 '25 22:05 rouault

GDALIsLineOfSightVisible_through_mountain needs to be adjusted and code formatting issue (https://github.com/OSGeo/gdal/actions/runs/15240459629/job/42859870241?pr=12460)

I am aware, I'll chase the failure down shortly after the holiday.

Ryanf55 avatar May 25 '25 23:05 Ryanf55

The GDAL project highly values your contribution and would love to see this work merged! Unfortunately this PR has not had any activity in the last 28 days and is being automatically marked as "stale". If you think this pull request should be merged, please check

  • that all unit tests are passing

  • that all comments by reviewers have been addressed

  • that there is enough information for reviewers, in particular link to any issues which this pull request fixes

  • that you have written unit tests where possible In case you should have any uncertainty, please leave a comment and we will be happy to help you proceed with this pull request. If there is no further activity on this pull request, it will be closed in 2 weeks.

github-actions[bot] avatar Jun 23 '25 03:06 github-actions[bot]

While we hate to see this happen, this PR has been automatically closed because it has not had any activity in the last 6 weeks. If this pull request should be reconsidered, please follow the guidelines in the previous comment and reopen this pull request. Or, if you have any further questions, just ask! We love to help, and if there's anything the GDAL project can do to help push this PR forward please let us know how we can assist.

github-actions[bot] avatar Jul 08 '25 03:07 github-actions[bot]