adding global SAR access through microsoft planetary compute #1263
Conversation
abradley60
requested review from
robbibt,
BexDunn,
erialC-P,
uchchwhash,
Kooie-cate,
geoscience-aman,
JM-GA,
margaretharrison,
vnewey,
Ariana-B,
amanda2099,
supermarkion and
erin-telfer
as code owners
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
|
@geoscience-aman this has both of our changes. Can you confirm it looks good? Also, will this be cloned down to users sandbox's on login during the session? Or does another branch get clones? |
Has my changes and looks good to me! I have made a commit with minor formatting improvements. I think it's the |
|
Hey @geoscience-aman @abradley60, the Sandbox sync comes from |
| @@ -2309,11 +2309,663 @@ | |||
| "metadata": {}, | |||
There was a problem hiding this comment.
Line #1. # Search the STAC Catalog
No need for this; only required for datacube access
Reply via ReviewNB
| @@ -2309,11 +2309,663 @@ | |||
| "metadata": {}, | |||
There was a problem hiding this comment.
| @@ -2309,11 +2309,663 @@ | |||
| "metadata": {}, | |||
There was a problem hiding this comment.
Line #2. ds_s1["vh_dB"] = 10 * np.log10(ds_s1.vh)
I think @geoscience-aman hada function for applying this step? If so, could be good to use that here too
Reply via ReviewNB
|
Hey @abradley60, left some tiny comments, if you can fix these up I can approve and we can merge this in! |
* Fix deprecation warnings and speed up code * Change last modified date * Intertidal exposure (#1261) * New files for intertidal exposure notebook * minor editing to Load packages cells * Updated discord and removed slack links * Improved markdown linking to image and gif in Introduction * Incorporated SS and MA reviews * Updated to include RBT reviews * Minor pip install and notebook naming edits. Add notebook to README * adding renamed exposure notebook back into PR * adding global SAR access through microsoft planetary compute (#1263) * adding global SAR access through microsoft planetary compute * Make minor spelling and formatting amendments. * small changes for PR --------- Co-authored-by: geoscience-aman <[email protected]> --------- Co-authored-by: Aman Chopra <[email protected]> Co-authored-by: geoscience-aman <[email protected]> Co-authored-by: ClaireP <[email protected]> Co-authored-by: Alex Bradley <[email protected]>
* Fix deprecation warnings and speed up code * Change last modified date * Intertidal exposure (#1261) * New files for intertidal exposure notebook * minor editing to Load packages cells * Updated discord and removed slack links * Improved markdown linking to image and gif in Introduction * Incorporated SS and MA reviews * Updated to include RBT reviews * Minor pip install and notebook naming edits. Add notebook to README * adding renamed exposure notebook back into PR * adding global SAR access through microsoft planetary compute (#1263) * adding global SAR access through microsoft planetary compute * Make minor spelling and formatting amendments. * small changes for PR --------- Co-authored-by: geoscience-aman <[email protected]> * Update USAGE.rst (#1268) Add Swinburne course for 2024 * Minor compatibility change for tide modelling package (#1269) * Mitch predict_xr (#1270) * add probability array output to predict_xr * predict_xr at proba_max args * predict_xr match arg names * xr_predict deal with multiband prob outout * xr_predict merge output probs * clean up comments and spacing * Update USAGE.rst (#1272) Add new reference, Burton et al 2024 Enhancing long-term vegetation monitoring in Australia: a new approach for harmonising the Advanced Very High Resolution Radiometer normalised-difference vegetation (NVDI) with MODIS NDVI * Fix broken code on `unstable` Sandbox image (#1274) * Updates for pyTMD * Fix contours bug due to groupby squeeze * Try loosening pyTMD requirements * Update tests to pass on both stable and unstable sandbox * Fix pansharpening bug --------- Co-authored-by: Aman Chopra <[email protected]> Co-authored-by: geoscience-aman <[email protected]> Co-authored-by: ClaireP <[email protected]> Co-authored-by: Alex Bradley <[email protected]> Co-authored-by: Bex Dunn <[email protected]> Co-authored-by: Mitchell Lyons <[email protected]>
Proposed changes
Include a brief description of the changes being proposed, and why they are necessary.
Closes issues (optional)
Checklist
(Replace
[ ]with[x]to check off)Load packagesGeneral advice)jupyterlab_code_formattertool can be used to format code cells to a consistent style: select each code cell, then clickEditand then one of theApply X Formatteroptions (YAPForBlackare recommended).NCIandDEA Sandbox(flag if not working as part of PR and ask for help to solve if needed)Notebook currently compatible with the NCI|DEA Sandbox environment onlyline below the notebook title to reflect the environments the notebook is compatible with