cryogars / ice-road-copters

MIT License
8 stars 2 forks source link

SSA rasters #9

Closed brentwilder closed 8 months ago

brentwilder commented 10 months ago

**Updated 12/11/2023

General things

SSA specific things

brentwilder commented 10 months ago

@shadoneel @ZachHoppinen

Thank you Shad for the email questions this morning! To summarize,

What is the canopy height model bug?

What about SSA - what does that stand for? I can weave together what you are working on, but want to make sure im tracking. Will this be default in any ice roads run that happens if we migrate to the new version on kelvin, or is it a flag? How intensive is the computation?

**EDIT: I found a much faster solution to have 1 normalized reflectance raster, leading to 1 SSA raster. This preserves the incidence angle information and reduces computational load x10. I have already output all of my incidence angles for my analysis and so this is really just an extra thing.

brentwilder commented 10 months ago

The biggest bottleneck now is computing calibrated reflectance for each flightline (roughly 5 min per flight line). But still is not bad. The other bottleneck is merging and performing IDW on one LAS file (but this is unavoidable and arguably the fastest possible option).

brentwilder commented 8 months ago

I'm going to test it soon to make sure everything still works after the changes, but thanks so much for the comments. I added logging for the main R functions, and streamlined pipeline to call a function from las2grain.

I didn't add the R packages to the yml file at this time because I feel like it may not be as helpful for folks using it right now. However, I realized the README was a bit frightening with the amount of detail on grain size. I dialed it back a bit and so now it is there and people can look into it more if need be.