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.