-
Notifications
You must be signed in to change notification settings - Fork 2
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
Jupyter docker: new full build with latest of everything #137
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- - xclim >= 0.47.0 - - ravenpy >= 0.13.0 + - xclim >= 0.49.0 + - ravenpy >= 0.14.1 # https://anaconda.org/conda-forge/xscen # A climate change scenario-building analysis framework, built with xclim/xarray. - # PIN to 0.8.2 for xclim 0.47.0 compat - - xscen == 0.8.2 + - xscen >= 0.9.0
…ation.ipynb failing
This was referenced May 10, 2024
…ompute it again And also to avoid breaking the build since the diff exit code is 1.
… lots of output change ValueError: These variables cannot be found in this dataset: ['rotated_pole'] PAVICS-landing-fix-pjei-11/content/notebooks/climate_indicators/PAVICStutorial_ClimateDataAnalysis-5Visualization.ipynb::Cell 5
…, lots of output change
ValueError: group_dqm_train failed on block with coords : Coordinates: RavenPy-fix-pjei-12/docs/notebooks/08_Getting_and_bias_correcting_CMIP6_data.ipynb::Cell 14 RavenPy-fix-pjei-12/docs/notebooks/paper/Perform_a_climate_change_impact_study_on_a_watershed.ipynb::Cell 14
…change, finally all code error gone !!!
tlvu
force-pushed
the
new-docker-build
branch
from
November 11, 2024 21:07
3a4bcc8
to
9c57498
Compare
tlvu
changed the title
New docker build
Jupyter docker: new full build with latest of everything
Nov 13, 2024
This was referenced Nov 13, 2024
tlvu
added a commit
to Ouranosinc/PAVICS-landing
that referenced
this pull request
Nov 14, 2024
Fixes 1/2 of Ouranosinc/pavics-jupyter-env-issues#11 For new Jupyter env in Ouranosinc/PAVICS-e2e-workflow-tests#137 ### Changes * Updates the ensemble analysis tools to use the latest xclim conventions * Updates the pre-commit configuration * Uses `https` for external links where possible * Fixes a small typo in the ReadMe file
tlvu
added a commit
to Ouranosinc/pavics-sdi
that referenced
this pull request
Nov 14, 2024
Fixes 1/2 of Ouranosinc/pavics-jupyter-env-issues#11 For new Jupyter env in Ouranosinc/PAVICS-e2e-workflow-tests#137. - `eccc-geoapi-xclim.ipynb` is fixed
tlvu
added a commit
to CSHS-CWRA/RavenPy
that referenced
this pull request
Nov 14, 2024
…Raven code conventions (#395) Fixes 1/2 of Ouranosinc/pavics-jupyter-env-issues#12 For new Jupyter env in Ouranosinc/PAVICS-e2e-workflow-tests#137. FYI @tlvu ### Changes - Updates the date handling of datasets so that chunking with `dask` is properly performed - Minor fixes to comments (formatting, typos).
@tlogan2000 The new Jupyter env is live on PAVICS. You can notify users to restart their servers to get the new version. |
tlvu
added a commit
to bird-house/birdhouse-deploy
that referenced
this pull request
Nov 14, 2024
## Overview Update Jupyter env to latest version, see Ouranosinc/PAVICS-e2e-workflow-tests#137 for more info. ## Changes **Non-breaking changes** - Update Jupyter env to version `py311-241111`. **Breaking changes** - This new Jupyter env requires the Jenkins host to be updated. Jenkins host upgrade problem found and fix on Ouranos side can be found here Ouranosinc/jenkins-config#16. ## Related Issue / Discussion - Resolves [issue id](url) ## Additional Information Links to other issues or sources. - [ ] Things to do... ## CI Operations <!-- The test suite can be run using a different DACCS config with ``birdhouse_daccs_configs_branch: branch_name`` in the PR description. To globally skip the test suite regardless of the commit message use ``birdhouse_skip_ci`` set to ``true`` in the PR description. Using ``[<cmd>]`` (with the brackets) where ``<cmd> = skip ci`` in the commit message will override ``birdhouse_skip_ci`` from the PR description. Such commit command can be used to override the PR description behavior for a specific commit update. However, a commit message cannot 'force run' a PR which the description turns off the CI. To run the CI, the PR should instead be updated with a ``true`` value, and a running message can be posted in following PR comments to trigger tests once again. --> birdhouse_daccs_configs_branch: master birdhouse_skip_ci: false
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
New full build with latest of everything.
Changes
conda env export
and environment size directly in the docker image for ease of tracking change between releases.libnetcdf
(Nb 3 raise wierd warning errors with newer Jupyter env PAVICS-landing#66 fixed).dask != 2024.11.0
due to bugs with Raven nb.Test
HydroShare_integration.ipynb
: https://github.com/Ouranosinc/PAVICS-e2e-workflow-tests/blob/0b3419bba4aa42ee5a1fa9a8de0c8a6b91bf8547/docker/saved_buildout/jenkins-buildlogs-raven.txtRelated Issue / Discussion
Matching notebook fixes:
Deployment to PAVICS: Jupyter: update to release py311-241111 bird-house/birdhouse-deploy#475
Jenkins-config changes for new notebooks: PR url
Other issues found while working on this one
Previous release: Jupyter docker: new full build with latest of almost everything except xclim and ravenpy to smooth transition #121
Additional Information
Full diff conda env export:
release-py311-240506-update240508...f8c1c58
Full new conda env export:
https://github.com/Ouranosinc/PAVICS-e2e-workflow-tests/blob/0b3419bba4aa42ee5a1fa9a8de0c8a6b91bf8547/docker/saved_buildout/conda-env-export.yml
DockerHub build log
https://github.com/Ouranosinc/PAVICS-e2e-workflow-tests/blob/0b3419bba4aa42ee5a1fa9a8de0c8a6b91bf8547/docker/saved_buildout/docker-buildlogs.txt