uw-cryo / coincident

Search and analysis of STV Precursor Coincident Datasets
https://coincident.readthedocs.io
MIT License
3 stars 1 forks source link

Create Altimeter Point Plotting Function #21

Open Jack-Hayes opened 2 days ago

Jack-Hayes commented 2 days ago

Hey @scottyhq ! Just wanted to get your thoughts on the below idea after I spent a lot of time messily plotting PCD candidate sites: Idea to create a quick plotting function for LiDAR extent, stereo extent, and GEDI and ICESat-2 point overlaps. Probably need a separate package 'plot' (ie. coincident.plot.plot_altimeter_points() ?)

Take the results of cascading_search() (and the input GeoDataframe/GeoSeries intocascading_search()) and plot all overlaps onto a small figure with a quick method call.

ie. If you know a 3dep site (variable name gf_wesm) has overlaps, plug it into this function:

This way you can do a quick check if your overlap area contains a satisfactory amount of filtered/quality-flagged ICESat-2 and GEDI points unaffected by clouds and such.

scottyhq commented 1 day ago

Probably need a separate package 'plot' (ie. coincident.plot.plot_altimeter_points()

I've gone back and forth on including built-in plots beyond basic geopandas.plot/explore and xarray.plot versus just focusing on search and I/O and leaving all visualizations up to users. On one hand, everyone has different aesthetic preferences and it tends to require many dependencies, on the other, as you mention it's ultimately nice to have a quick visual check of results! Your plan sounds good if you want to open a draft PR!

Not sure if building all SlideRule dependencies into this library is desirable

Yes, I think we should add it https://github.com/uw-cryo/coincident/issues/6, I'll wrap that up tomorrow!