search

digitalearthpacific / dep-coastlines

GNU General Public License v3.0
3 stars 1 forks source link

Implement terrain masking #7

Open jessjaco opened 1 year ago

jessjaco commented 1 year ago

Looks like they get the sun azimuth and elevation from the landsat metadata, then create a hillshade using matplotlib

jessjaco commented 1 year ago

We might consider using xrspatial hillshade

jessjaco commented 1 year ago

See example PC notebook here

jessjaco commented 1 year ago

See landsat metadata here

alexgleith commented 1 year ago

I think we should test without the hillshade first perhaps. I think this was only added to DE Africa after testing identified issues.

jessjaco commented 1 year ago

@sopac suggested the same, as terrain is likely not as big an issue in this study area.

jessjaco commented 7 months ago

This is an issue in a few places (around Noumea for one), but it has mostly been eliminated by my model-based filtering of "bare_terrain" as I call it image