-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Collection 3 ARD Style Questions #681
Comments
I think this makes sense, and should account for nodata as well as help clean up some of the ugly results you get near the edge of Landsat paths and rows. There are however issues with the current 20 m resolution Sentinel-2 contiguity bands not being fit-for-purpose for masking 10 m Sentinel-2 bands (I'm going to email this out more widely later this week), so there's a chance a different approach might be required for Sentinel-2 down the track.
Yes. Cloud masking these indices would be a great improvement over the current approach with can lead to very misleading visualisations of the indices under cloud (e.g. the MNDWI water index which maps clouds as a strong water signal).
I think this would overcomplicate things - ideally we want to keep the number of options to a minimum where possible. Clouds in the True and False colour layers are easily recognizable visually unlike the indices where clouds can be very difficult to distinguish from non-cloud, so there's less of a pressing need to hide them from the user. This isn't a strong preference through, so happy to be overruled if anyone would like cloud-masked layers to be included.
I lean on the side of the GSKY behaviour here because it gives the user a lot more context; e.g. they might not know what "NDVI" is, but they will know they want a view of the data that focuses on "vegetation". That said, there are advantages in having the spectral bands info included too for indices like MNDWI/NDWI that vary by a single band. Would
"True colour" and "False colour" feel more intuitive, use existing remote sensing terminology, and pair nicely together. I've never heard the term "Simple RGB" used outside of our web services, and there isn't really anything "simpler" about it vs. any of the other layers. If we decided to include band names for the indices, it could be good to include them for these styles too, e.g.
True colour. False colour is incredibly useful, but it raises the bar of interpretability significantly. I feel that users should be greeted with a view of satellite data they immediately recognise, then only be exposed to more complex remote sensing views once they make the choice to see them.
The OWS layer on the right is much more vibrant and dynamic - my preference would be for this one. |
My two bits:
|
@SpacemanPaul Yep, I agree that for 7 something in between would probably be the best... split the difference? (I did some notebook experimenting last year with visualising the layers after applying a log transform to dull down the high albedo areas and it seemed to work quite well too, but I have no idea if that's possible in OWS.) |
Change "Simple RGB" to "True Colour as per advice in #681
It's totally possible!!! We do way weirder things to scale TMAD. |
Update indice style labels in line with #681
As discussed in today's meeting @SpacemanPaul and @pindge, we would like to augment the behaviour of the following ARD C3 styles:
|
For reference:
https://explorer.dev.dea.ga.gov.au/products/ga_ls8c_ard_3
https://explorer.dev.dea.ga.gov.au/products/ga_ls7e_ard_3
https://explorer.dev.dea.ga.gov.au/products/ga_ls5t_ard_3
The text was updated successfully, but these errors were encountered: