Treat LoS special case of exactly on terrain as visible
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
GDALIsLineOfSightVisible_through_mountain needs to be adjusted and code formatting issue (https://github.com/OSGeo/gdal/actions/runs/15240459629/job/42859870241?pr=12460)
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.
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.
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.