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

New Jupyter env build with new xlrd, pre-commit, pin dask, distributed, cf_xarray, latest of everything else #101

Merged
merged 12 commits into from
Apr 7, 2022

Conversation

tlvu
Copy link
Contributor

@tlvu tlvu commented Apr 1, 2022

Relevant Changes

>   - pre-commit=2.17.0=py38h578d9bd_0
>   - xlrd=2.0.1=pyhd8ed1ab_3

<   - xclim=0.32.1=pyhd8ed1ab_0
>   - xclim=0.34.0=pyhd8ed1ab_0

<   - cfgrib=0.9.9.1=pyhd8ed1ab_1
>   - cfgrib=0.9.10.1=pyhd8ed1ab_0

<   - cftime=1.5.1.1=py38h6c62de6_1
>   - cftime=1.6.0=py38h3ec907f_0

<   - intake-xarray=0.5.0=pyhd8ed1ab_0
>   - intake-xarray=0.6.0=pyhd8ed1ab_0

<   - pandas=1.3.5=py38h43a58ef_0
>   - pandas=1.4.1=py38h43a58ef_0

<   - regionmask=0.8.0=pyhd8ed1ab_1
>   - regionmask=0.9.0=pyhd8ed1ab_0

<   - rioxarray=0.9.1=pyhd8ed1ab_0
>   - rioxarray=0.10.3=pyhd8ed1ab_0

<   - xarray=0.20.2=pyhd8ed1ab_0
>   - xarray=2022.3.0=pyhd8ed1ab_0

<   - zarr=2.10.3=pyhd8ed1ab_0
>   - zarr=2.11.1=pyhd8ed1ab_0

New Packages Pins

dask and distributed was causing PAVICStutorial_ClimateDataAnalysis-3Climate-Indicators.ipynb to run excessively slow.

cf_xarray caused climex.ipynb to be slow.

See #100 for more info why these had to be pinned.

Related Issue / Discussion

Test Results

Additional Information

tlvu added 11 commits March 17, 2022 15:08
…problem

dask and distributed for slow PAVICStutorial_ClimateDataAnalysis-3Climate-Indicators.ipynb.

cf_xarray for slow WPS_example.ipynb.
As requested by Jeremy Fyke.
… /opt/conda/envs/birdy/fonts"

The warning appears only once and may or may not re-appear again after a
"Restart kernel". It is completely intermittent.

See issue #97.
…ance problems"

This reverts commit 1387921.

Problem seems to be done by itself.
…ests:220401.1"

This reverts commit 406ad0e.

The performance problem is still there if we do not pin dask,
distributed and cf_xarray.
… performance problems""

This reverts commit 9447d53.

Testing error, the performance problem is still there if we do not keep
those pins.
@tlvu tlvu requested a review from tlogan2000 April 1, 2022 22:02
@tlvu tlvu changed the title New Jupyter env build with new xlrd, pre-commit, pin dask, districuted, cf_xarray, latest of everything else New Jupyter env build with new xlrd, pre-commit, pin dask, distributed, cf_xarray, latest of everything else Apr 1, 2022
@tlvu
Copy link
Contributor Author

tlvu commented Apr 4, 2022

@tlogan2000 this is deployed as "beta" image in production, could you do a quick bokeh visualization performance test to spot for regressions that we've seen in the past?

tlvu added a commit to bird-house/birdhouse-deploy that referenced this pull request Apr 7, 2022
jupyter env: new xlrd, pre-commit, pin dask, distributed, cf_xarray, latest of everything else

  Deploy new Jupyter env from PR Ouranosinc/PAVICS-e2e-workflow-tests#101 on PAVICS.

  Detailed changes can be found at Ouranosinc/PAVICS-e2e-workflow-tests#101.

  Relevant changes:
  ```diff
  >   - pre-commit=2.17.0=py38h578d9bd_0
  >   - xlrd=2.0.1=pyhd8ed1ab_3
  
  <   - xclim=0.32.1=pyhd8ed1ab_0
  >   - xclim=0.34.0=pyhd8ed1ab_0
  
  <   - cfgrib=0.9.9.1=pyhd8ed1ab_1
  >   - cfgrib=0.9.10.1=pyhd8ed1ab_0
  
  <   - cftime=1.5.1.1=py38h6c62de6_1
  >   - cftime=1.6.0=py38h3ec907f_0
  
  <   - intake-xarray=0.5.0=pyhd8ed1ab_0
  >   - intake-xarray=0.6.0=pyhd8ed1ab_0
  
  <   - pandas=1.3.5=py38h43a58ef_0
  >   - pandas=1.4.1=py38h43a58ef_0
  
  <   - regionmask=0.8.0=pyhd8ed1ab_1
  >   - regionmask=0.9.0=pyhd8ed1ab_0
  
  <   - rioxarray=0.9.1=pyhd8ed1ab_0
  >   - rioxarray=0.10.3=pyhd8ed1ab_0
  
  <   - xarray=0.20.2=pyhd8ed1ab_0
  >   - xarray=2022.3.0=pyhd8ed1ab_0
  
  <   - zarr=2.10.3=pyhd8ed1ab_0
  >   - zarr=2.11.1=pyhd8ed1ab_0
  ```
@tlvu tlvu merged commit c74ad1e into master Apr 7, 2022
@tlvu tlvu deleted the new-docker-build branch April 7, 2022 13:15
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.

Intermittent warnings "Unable to revert mtime: /opt/conda/envs/birdy/fonts" confuse new Jupyter users
2 participants