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

Updates to database access PR #1153

Merged
merged 374 commits into from
Feb 7, 2025
Merged

Updates to database access PR #1153

merged 374 commits into from
Feb 7, 2025

Conversation

robbibt
Copy link
Member

@robbibt robbibt commented Dec 4, 2023

Proposed changes

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

Closes issues (optional)

  • Closes Issue #000
  • 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

Benjamin Glitsos and others added 26 commits November 30, 2023 16:55
Corrected 07_Parallel_processing_with_Dask to 09_Parallel_processing_…
bf2091b 
…with_Dask
Fixed the sandbox URL to go directly to the sandbox
2eb1d9a
Fixed 01_Jupyter_notebooks.ipynb link
303bc52
Fixing links in Jupyter notebooks
4bb618c
Fixing links in Jupyter notebooks
59baa3e
Fixing links by converting them to relative links in Jupyter notebooks
bb89c66
Changed some URLs to absolute URLs to fix links
0c86d35
Changed notebook URLs to absolute URLs
0aaa827
Fixing links
ae69315
Fixing links
4fd579a
Fixing a link
42ac247
Changed data product links to prevent 403 redirects
cc6e177
Changed more product links to avoid 403 redirects
4671f8e
Removed anchors from links to data products
dfbb52e
Fixing links
6da3963
Fixing links
8d25118
Fixed more links
4633f93
Fixed many notebooks links
e24d770
Fixing links
75d280e
Fixing links
171f7d0
Fixed more URLs
1f13a5a
@robbibt
Simplify tests
cbde9b2
@robbibt
Remove indexing script, rename, remove DEA Tools install
ca8ad03
@robbibt
Run on test branch
57a54c1
@robbibt
Restore DEA Tools install
c1b0f06
@robbibt
Add remainder of DEA products and How to notebooks, increase durations
82f3d66
@robbibt
Copy link
Member Author

robbibt commented Dec 4, 2023

Need to fix: Downloading_data_with_STAC.ipynb:
image

@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

BexDunn and others added 13 commits October 9, 2024 13:29
@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
@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
Deprecate tide modelling tools in favour of eo-tides (#1276)
0204f80
@GL-S
small fixes in input for calculate_indices() rgb()
64fb9e1 
added line to prevent warnings from dask.
Changed collection name in calculate_indices(), and reset band names in rgb() to default to avoid errors
@GL-S
test
adddf58
@GL-S
revert last test commit
7d8d503
@GL-S
Revert "revert last test commit"
329da96 
revert last test commit
This reverts commit 7d8d503.
@GL-S
revert test commit
33a0d74 
This reverts commit adddf58.
@GL-S
product and collection names fixed
c637954
@GL-S
small fixes in input for calculate_indices() rgb()
11fcb73 
small fixes in input for calculate_indices() rgb()
@jennaguffogg
Update product and band names in ML notebook `Machine_learning_with_O…
2473a68 
…DC` (#1276) (#1278)

* update product and band names used in notebook

* update geoMAD product and band names

* change hyperlink to match updated geoMAD product
@robbibt
Merge develop
da4eff2
@robbibt
Fix data sync
05b1eea
@robbibt robbibt marked this pull request as ready for review February 7, 2025 04:52
@robbibt robbibt merged commit c9b5b13 into damien/test-rds-connection Feb 7, 2025
1 check passed
@robbibt robbibt deleted the testing_tests branch February 7, 2025 04:52
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

@Matt-dea Matt-dea Awaiting requested review from Matt-dea

@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

@treefern treefern Awaiting requested review from treefern

@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.