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

update remaining nb. mv bias to sandbox #382

Merged
merged 4 commits into from
Apr 16, 2021
Merged

update remaining nb. mv bias to sandbox #382

merged 4 commits into from
Apr 16, 2021

Conversation

huard
Copy link
Contributor

@huard huard commented Apr 16, 2021

Overview

This PR fixes Finch related notebooks.

Note that some processes only work with GR4JCN because other emulators do not yet support the area, lat, lon, elevation -> hru conversion.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

.cruft.json Outdated Show resolved Hide resolved
@huard huard merged commit 7ffdb38 into master Apr 16, 2021
@huard huard deleted the fix_finch_nb branch April 16, 2021 20:52
tlvu added a commit to bird-house/birdhouse-deploy that referenced this pull request Apr 16, 2021
…for-raven-demo

Update Raven and Jupyter env for Raven demo

Raven release notes PR Ouranosinc/raven#374 + Ouranosinc/raven#382

Jupyter env update PR Ouranosinc/PAVICS-e2e-workflow-tests#71

Other fixes:
* Fix intermittent Jupyter spawning error by doubling various timeouts config (it's intermittent so hard to test so we are not sure which ones of timeout fixed it)
* Fix Finch and Raven "Broken pipe" error when the request size is larger than default 3mb (bumped to 100mb) (fixes Ouranosinc/raven#361 and Finch related comment bird-house/finch#98 (comment))
* Lower chance to have "Max connection" error for Finch and Raven (bump parallelprocesses from 2 to 10). In prod, the server has the CPU needed to run 10 concurrent requests if needed so this prevent users having to "wait" after each other.
Zeitsperre pushed a commit that referenced this pull request Aug 17, 2023
update remaining nb. mv bias to sandbox
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.

2 participants