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.

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

Updates to product notebook Knowledge Hub links and DEA notebook content #1221

Merged
merged 6 commits into from
Jun 14, 2024

Conversation

robbibt
Copy link
Collaborator

@robbibt robbibt commented Apr 11, 2024

Proposed changes

This PR updates some documentation that was outdated (Beginners_guide/DEA.ipynb that refered to old names for some of our products and didn't include Landsat 9), and makes some minor updates to our DEA Product notebooks to use more visible alert boxes to point users to Knowledge Hub for more information about our products:

image

Closes issues (optional)

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link
Collaborator

@erialC-P erialC-P left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! Happy to approve for today. Would be useful to embed storymap links in product notebooks too where we've got them but this can be a job for another rainy day :)

@robbibt
Copy link
Collaborator Author

robbibt commented Jun 14, 2024

Looks good to me! Happy to approve for today. Would be useful to embed storymap links in product notebooks too where we've got them but this can be a job for another rainy day :)

Yeah, awesome idea. I think we could do a more general re-write/re-design of the very header block of the notebooks - would be neat to make something pretty that includes all the useful stuff like KH, storymap, Explorer etc!

@robbibt robbibt merged commit 523c34f into develop Jun 14, 2024
1 check passed
@robbibt robbibt deleted the doco_updates branch June 14, 2024 04:59
erialC-P pushed a commit that referenced this pull request Jun 17, 2024
* Add updated `xr_regression` function for multi-dimensional linear regression (#1226)

* Add updated xr_regression function

* Add dask support for lazy computation

* Set dtypes

* Update docstring

* Update docstring

* Add MAD outliers

* Update docstring

* Remove lag functionality

* Update docstrings

* Add better error handling

* Update stream gauge corr notebook to use new func

* Adding DEAfrica Wetland Turbidity notebook for Australian study site (#1175)

* Adding DEAfrica Wetland Turbidity notebook for Australian study site

* change all instances of NDTI to NDTI2 to reflect usage at top of notebook

* update notebook to use Collection 3 WO Statistics

* rerun notebook

---------

Co-authored-by: BexDunn <[email protected]>

* Add spatial interpolation with `xr_interpolate` notebook (#1233)

* Add ensemble tide modelling functionality to model_tides

* Update test_coastal.py

* Remove test

* Updates to IDW, xr_interpolate and ensemble tide modelling code"

* Doco updates

* Switch ensemble rankings from high to low = good

* Update docstring

* Fix doco

* Add interpolation notebook

* Remove coastal files from branch

* Add points data

* Review feedback;

* Add p param to IDW

* Fix test

* Updates to product notebook Knowledge Hub links and DEA notebook content (#1221)

* Move KH links into consistent alert box format

* Update DEA notebook

* Minor wording updates

* Minor wording

* Temporarily remove STAC notebook from tests

* Add ensemble tide modelling functionality to `model_tides` (#1231)

* Add ensemble tide modelling functionality to model_tides

* Update test_coastal.py

* Remove test

* Updates to IDW, xr_interpolate and ensemble tide modelling code"

* Doco updates

* Switch ensemble rankings from high to low = good

* Update docstring

* Fix doco

* Add interpolation notebook

---------

Co-authored-by: Matt-dea <[email protected]>
Co-authored-by: BexDunn <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants