Andrew Bell
Andrew Bell
I don't understand the doxygen CI failure. I ran doxygen locally and searched for "warning" and found none.
Fine by me when you're satisfied. Thanks.
Do you do an interactive rebase? What happens with the merge commits? Just curious.
I just do squash merges. Is the other way better?
Let me know if there are things I can do that would make things easier for you.
What we're calculating in DEM mode is the "minimum height at which the cell would be observable". I believe we are correctly calculating this. However, we are bounding the height...
Seems like we should eliminate the spinlock support with this issue. Is that OK?
There is a famous Linus rant about using spinlocks outside of kernel/embedded code. (Linus likes to rant on occasion.) I think the recommendation is to use a regular mutex.
I ran clang-format locally, so I'm not sure what the issue is with the CI failure. Maybe different clang-format versions?
Before I threaded things, it was a wee bit slower than the original. Most of that was the merging of the output vector, which required the RasterIO code to potentially...