subset.ipynb: fix to pass Jenkins against other servers than prod #177
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
Use `boreas.ouranos.ca` instead of `pavics.ouranos.ca` for `wfs_url = 'https://boreas.ouranos.ca/geoserver/wfs'` so we do not replace it with other hosts, due to existing issue Ouranosinc/pavics-sdi#183. Write `poly_file = Path('/tmp/mtl_raw.geojson')` to `/tmp/` for notebooks works on tutorial-notebooks folder on Jupyter env that is read-only.
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
huard
approved these changes
Apr 13, 2021
tlvu
added a commit
to bird-house/birdhouse-deploy
that referenced
this pull request
Apr 13, 2021
…h-subset-nb testdata: new test file for new finch subset.ipynb For notebook https://github.com/bird-house/finch/blob/e2298701dcd20b24dae021c666ebb684d3640b4b/docs/source/notebooks/subset.ipynb To fix error in `docker logs finch`: ``` OSError: [Errno -102] NetCDF: Can't read file: b'https://lvupavicsdev.ouranos.ca/twitcher/ows/proxy/thredds/dodsC/birdhouse/testdata/xclim/NRCANdaily$ nrcan_canada_daily_tasmin_1990.nc' During handling of the above exception, another exception occurred: Traceback (most recent call last): File "/opt/conda/envs/finch/lib/python3.9/site-packages/pywps/app/Process.py", line 248, in _run_process self.handler(wps_request, wps_response) # the user must update the wps_response. File "/code/finch/processes/wps_base.py", line 42, in _handler_wrapper raise ProcessError(f"Finch failed with {err!r}") pywps.app.exceptions.ProcessError: Finch failed with OSError -102, NetCDF: Can t read file ``` Together with bird-house/finch#177 + latest Finch docker + latest xESMF, we have a working Jenkins build http://jenkins.ouranos.ca/job/PAVICS-e2e-workflow-tests/job/minor-fixes-launchnotebook-nbval-lax-as-option/12/console
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.
Use
boreas.ouranos.cainstead ofpavics.ouranos.caforwfs_url = 'https://boreas.ouranos.ca/geoserver/wfs'so we do not replace it with other hosts, due to existing issue Ouranosinc/pavics-sdi#183.Write
poly_file = Path('/tmp/mtl_raw.geojson')to/tmp/for notebooks to work ontutorial-notebooksfolder on Jupyter env that is read-only.Together with bird-house/birdhouse-deploy#146 + latest Finch + latest xESMF, we have a working Jenkins build http://jenkins.ouranos.ca/job/PAVICS-e2e-workflow-tests/job/minor-fixes-launchnotebook-nbval-lax-as-option/12/console