-
Notifications
You must be signed in to change notification settings - Fork 134
develop to stable - originally created for updates landcover code and notebooks #1345
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
Conversation
* add WIT app files desired from lauren_WIT_app * Grammatical edits to markdown cells * edit licence section to add ccby info and removed witapp link from pubs * updates from review comments --------- Co-authored-by: Claire Phillips <[email protected]> Co-authored-by: BexDunn <[email protected]>
* update-LC-planetary-NB * removed unused Python package * formatting
…ion() * new notebook and function * changed area and added viz * consistent colours * formatting * mask with vector * fixed output table * fixed text * consitency figures size method * same colours of LC * last fixes * more robust code * Update Land_cover_export_data.ipynb * edits to markdown text for lc export notebook * small typos * Update Land_cover_pixel_drill.ipynb * adjusted order markdown cells * Update README.rst new notebook * move analysis params to top, load based on polygon * refine and simplify * rename * small typos fixed * applied formatting style --------- Co-authored-by: Jenna Guffogg <[email protected]> Co-authored-by: cbur24 <[email protected]>
…cs-token Documented a personal access token as a code comment (minor change)
* first attempt at continental-scale animation notebook * testing notebook can execute in small sandbox. Minor fixes to markdown text. * test and reduce years to speed up processing * Minor formatting and code updates --------- Co-authored-by: Jenna Guffogg <[email protected]> Co-authored-by: robbibt <[email protected]>
* first attempt at a new climate notebook * add era5 * refactor markdown cell order * refine markdown * fixes to climate notebook * alternative * remove 2nd nb and refine chunks and params of era5 --------- Co-authored-by: GL-S <[email protected]>
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
* Update USAGE.rst Add recent citation from DE-Africa * sort --------- Co-authored-by: Gianluca S. <[email protected]>
* make copy tidal and pull image optional for reusing db connection provisioning * specify input type * suppress chattering messages * explicit on condition * use strings in condition * always typos --------- Co-authored-by: James Miller <[email protected]> Co-authored-by: ClaireP <[email protected]> Co-authored-by: Emma Ai <[email protected]>
Update rst to include WIT app so it will appear in interactive apps.
Add WIT app to README.rst
* begin retesting ML notebooks * continue ML updates * clean up remaining ML nbs * add missing import
* Update packaging to use uv, hatch * Remove ciso8601 * Pin odc-algo, remove pyTMD, add odc-stac to STAC * Remove setup.py, update deptry * Add additional notebook dependencies * Set ruff linting fix to false for safety * Simplify jupyter * Fix test install * Pin old PyTMD, remove eo-tides * Update readme * Updates from review * Wording * Update README_tools.md * Remove warp sub-dependency
* Simplify package publication using uv * Simplify environment creation * Upgrade dask-ml * Test Python 3.10 * Try to pin numba * Fix deptry, test publish * Switch to Python 3.11 * Run full repo tests on changes to pyproject.toml * Simplify action
…talled (#1354) * Raise error if datacube not installed * Updates to relative package imports * Update spatial and temporal * Update maps.py, dask.py, wit_app.py
* Apply Ruff formatting * Apply Ruff linting to fix code issues * Revert relative imports * Fix type formatting
* Apply Ruff formatting * Apply Ruff linting to fix code issues * Revert relative imports * Fix type formatting * Update repo to use eo-tides, temporarily delete HLTC notebook
* update WIT app tools files and name * Update rst to new WIT app name update name of WIT interactive app
…ncy (#1360) * update create_local_dask_cluster to work without datacube * fix cluster creation in nb * fix docstrings * remove doc string reference to /scripts, and ad kwargs * update doc string with kwargs arg * rm int from kwargs arg * Make configure_rio a param * add default memory limit to client * add more info to docstring * constrain memory limit by number of workers * docstrings --------- Co-authored-by: Robbi Bishop-Taylor <[email protected]>
* Final tidy up before DEA Tools 0.4.0 release * Fix docstring formatting * Try fixing package level * Update * Fix typo * Try settting target wheel * Move readme, alphabetise imports, add sunriset * Update readme * Add stac dependency * Minor formatting * Remove STAC optional dep, make tests only run once * Update readmes * Update code owners * Fix code owners * Add odc.io * add Ipython warning (#1361) --------- Co-authored-by: Chad Burton <[email protected]>
Forecasting/modelling notebook
* first commit of random sampling tools * more efficient algo working * write nb and docs for func * rename nb * add section on sapling large rasters * remove unused libraries * add tags * update tags * remove large-scale testing * refine description * typos * update readme.rst * add tests for random_samples_xr * tidy up function and make more consistent * test nb again * Use spatial_dims, minor formatting * Docstring * Rename * Update validation.py --------- Co-authored-by: robbibt <[email protected]>
* more-flexible-inputs * first-draft-notebook * fixed-doc-functions * read-me-file * index-updated * finished-notebook * first-draft-testing * test_notebooks * improved-test * test-script * finalised-test-script * changed-description-delete-output * screenshot-example * functions-can-work-with-cli * fixed-funct-names-test * update-readme * Updates to continental mosaics PR (#1364) * Updates to COG mosaic and VRT code * Fix formatting * typo * Improve logging * Update tests, fix commented out lines * Update with beter logging, URLs * Revert path change * naming-and-structure * revert-accidental-changes * Implement temporary workaround for S3 access issue * Temporarily ignore notebook * Update test_notebooks_pullrequest.yml * Fix formatting * extra-details_update-names --------- Co-authored-by: Robbi Bishop-Taylor <[email protected]>
…for VRTs (#1368) * colour-schemes-in-deatools-package * udpated-tests
I was wondering if we could do a merge from develop to stable. Currently, the default notebooks in the Sandbox still use old versions of landcover functions, which do not work with LC 2.0. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@GL-S can you check the codeowners are correct? Github is flagging that it can't find some of the users? Unknown owner on line 5: make sure @margaret-dea exists and has write access to the repository
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure how to do that. Considering this PR is a bit old, is it worth it to just delete this and start a new one?
Maybe it will fix the issue
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, sounds like a plan :) Maybe rather than deleting it just close it with a comment and start again?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @GL-S thanks for doing this. Can you please check that you've got the most up-to-date version of develop in this PR so that any new changes are captured, and that any missing files from this PR are intentional. I'm sure it's the case, however it's a really big PR and I don't want to upset stable! I note there are a few files removed, including a licence one and a number of notebooks and tools packages. I'd love a few more eyes on it as it seems like a lot of deletions. Probably because we haven't pushed to stable in a while :) thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did we mean to remove the high and low tide imagery notebook here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this one supposed to be removed @cbur24 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
did this file mean to be removed? @GL-S
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@GL-S ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Closing because this was opened months ago. Starting a new one |
Merging develop into stable so that the knowledge hub documentation reflects the most recent updates to the landcover code and notebooks