Ouranosinc / PAVICS-e2e-workflow-tests

Test user-level workflow.
Apache License 2.0
0 stars 2 forks source link

New Jupyter env docker build for urlpath #112

Closed tlvu closed 1 year ago

tlvu commented 1 year ago

Overview

Full rebuild to add urlpath. Previous rebuild was https://github.com/Ouranosinc/PAVICS-e2e-workflow-tests/pull/106.

Changes

< - dask=2022.11.0=pyhd8ed1ab_0

  • dask=2022.11.1=pyhd8ed1ab_0

< - flox=0.6.3=pyhd8ed1ab_0

  • flox=0.6.4=pyhd8ed1ab_0

< - h5netcdf=1.0.2=pyhd8ed1ab_0

  • h5netcdf=1.1.0=pyhd8ed1ab_0

< - numpy=1.23.4=py38h7042d01_1

  • numpy=1.23.5=py38h7042d01_0

  • urlpath=1.2.0=pyhd8ed1ab_0

Test

Related Issue / Discussion

Additional Information

huard commented 1 year ago

Trevor mentioned another library to deal with URLs (yarl), but I suggest we go ahead with urlpath for the time being. Functionalities seem very similar.

Zeitsperre commented 1 year ago

I haven't had a chance to look at it, but yarl seems be safer (non-mutable objects), and maintained. I'll work it into some existing projects first to get a sense of it. We can move from urlpath eventually.

tlvu commented 1 year ago

@huard a build with urlpath is available on PAVICS as the alpha Jupyter image. You can try your notebooks. I'll try to close this PR for this week. All initial tests seem fine with this new build so far.

tlvu commented 1 year ago

@tlogan2000 @JeremyFyke FYI a new Jupyter env is coming. Delta vs the previous release is fairly small since they differ for 2 weeks only.

All standard checks are passing. I guess it should not break the various Panel dashboard we have but I have not checked. It is deployed currently as alpha image on PAVICS. Schedule to go-live this Friday.

tlvu commented 1 year ago

@tlogan2000 This new Jupyter env is live. I decided to keep the existing "previous" release in the Jupyter image drop-down menu unchanged since the change between this one and the previous release is fairly small.

tlogan2000 commented 1 year ago

Excellent! Thanks @tlvu

JeremyFyke commented 1 year ago

Hi @tlvu - can you confirm which PAVICS version to test apps out against? the fire weather app works in internal testing but appears to error out upon deployment (tested against alpha, beta, current, and panel_dashboard).

tlvu commented 1 year ago

Hi @tlvu - can you confirm which PAVICS version to test apps out against? the fire weather app works in internal testing but appears to error out upon deployment (tested against alpha, beta, current, and panel_dashboard).

@JeremyFyke Fixed in https://github.com/Ouranosinc/pavics_panel_cccs_deploy/pull/11

As for which image, it is the current image. You can always refer to the source of truth here https://github.com/Ouranosinc/pavics_panel_cccs_deploy/blob/4563f94944dafe83783cc3c9c12da6a027891a98/cccs-config/docker-compose-extra.yml#L56-L57