Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update stable with changes from develop #1273

Merged
merged 10 commits into from
Oct 9, 2024
Merged

Update stable with changes from develop #1273

merged 10 commits into from
Oct 9, 2024

Conversation

robbibt
Copy link
Member

@robbibt robbibt commented Oct 9, 2024

Proposed changes

Include a brief description of the changes being proposed, and why they are necessary.

Closes issues (optional)

  • Closes Issue #000

Checklist

(Replace [ ] with [x] to check off)

  • Notebook created using the DEA-notebooks template
  • Remove any unused Python packages from Load packages
  • Remove any unused/empty code cells
  • Remove any guidance cells (e.g. General advice)
  • Ensure that all code cells follow the PEP8 standard for code. The jupyterlab_code_formatter tool can be used to format code cells to a consistent style: select each code cell, then click Edit and then one of the Apply X Formatter options (YAPF or Black are recommended).
  • Include relevant tags in the final notebook cell (refer to the DEA Tags Index, and re-use tags if possible)
  • Clear all outputs, run notebook from start to finish, and save the notebook in the state where all cells have been sequentially evaluated
  • Test notebook on both the NCI and DEA Sandbox (flag if not working as part of PR and ask for help to solve if needed)
  • If applicable, update the Notebook currently compatible with the NCI|DEA Sandbox environment only line below the notebook title to reflect the environments the notebook is compatible with
  • Check for any spelling mistakes using the DEA Sandbox's built-in spellchecker (double click on markdown cells then right-click on pink highlighted words). For example:

sandbox_spellchecker

geoscience-aman and others added 9 commits August 7, 2024 04:19
@geoscience-aman
Fix deprecation warnings and speed up code
b7ad6ba
@geoscience-aman
Complete merge
0f3557d
@geoscience-aman
Change last modified date
1e04274
@erialC-P
Intertidal exposure (#1261)
7e29a84 
* 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
@abradley60 @geoscience-aman
adding global SAR access through microsoft planetary compute (#1263)
0c4268d 
* adding global SAR access through microsoft planetary compute

* Make minor spelling and formatting amendments.

* small changes for PR

---------

Co-authored-by: geoscience-aman <96451725+geoscience-aman@users.noreply.github.com>
@BexDunn
Update USAGE.rst (#1268)
c715abb 
Add Swinburne course for 2024
@robbibt
Minor compatibility change for tide modelling package (#1269)
aec5468
@mitchest
Mitch predict_xr (#1270)
9710bb5 
* 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
@BexDunn
Update USAGE.rst (#1272)
262bd5c 
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
@review-notebook-app Review Notebook App
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@robbibt
Fix broken code on unstable Sandbox image (#1274)
fe36b0e 
* 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
@robbibt robbibt marked this pull request as ready for review October 9, 2024 20:43
@robbibt robbibt merged commit 8d22060 into stable Oct 9, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BexDunn BexDunn Awaiting requested review from BexDunn BexDunn is a code owner

@erialC-P erialC-P Awaiting requested review from erialC-P erialC-P is a code owner

@uchchwhash uchchwhash Awaiting requested review from uchchwhash uchchwhash is a code owner

@Kooie-cate Kooie-cate Awaiting requested review from Kooie-cate Kooie-cate is a code owner

@geoscience-aman geoscience-aman Awaiting requested review from geoscience-aman geoscience-aman is a code owner

@JM-GA JM-GA Awaiting requested review from JM-GA JM-GA is a code owner

@margaretharrison margaretharrison Awaiting requested review from margaretharrison margaretharrison is a code owner

@vnewey vnewey Awaiting requested review from vnewey vnewey is a code owner

@Ariana-B Ariana-B Awaiting requested review from Ariana-B Ariana-B is a code owner

@amanda2099 amanda2099 Awaiting requested review from amanda2099 amanda2099 is a code owner

@supermarkion supermarkion Awaiting requested review from supermarkion supermarkion is a code owner

@erin-telfer erin-telfer Awaiting requested review from erin-telfer erin-telfer is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@robbibt @geoscience-aman @erialC-P @abradley60 @BexDunn @mitchest