Jetty update #3
Jetty update #3
Conversation
|
https://github.com/devinrsmith/deephaven-extra-images is relevant too - it may be the template we provide for building your own images off of Deephaven. |
There was a problem hiding this comment.
Doesn't look like it's working for me, server says it's started up but getting a socket error when navigating to http://localhost:10000/ide/
| docker-compose pull | ||
| docker-compose up --build -d | ||
| docker compose pull | ||
| docker compose up --build -d |
There was a problem hiding this comment.
There's a line at the bottom of the README that we should remove since we removed the examples:
The 3D examples in this readme are from Rashida Nasrin Sucky the data is available from kaggle
server/Dockerfile
Outdated
| RUN ./pack-plugins.sh @deephaven/js-plugin-matplotlib | ||
|
|
||
| FROM ghcr.io/deephaven/server:latest | ||
| RUN pip install --no-cache-dir deephaven-plugin-matplotlib |
There was a problem hiding this comment.
| RUN pip install --no-cache-dir deephaven-plugin-matplotlib | |
| RUN pip install --no-cache-dir deephaven-plugin-matplotlib matplotlib |
Matplotlib should be installed in the Dockerfile as well
There was a problem hiding this comment.
@mofojed doesn't deephaven-plugin-matplotlib depend on matplotlib? We shouldn't need to re specify dependency IMO.
There was a problem hiding this comment.
That said, it's not bad to be explicit here since we do rely directly on matplotlib.
I think we'll want to simplify this not to rely on another data image.
We'll also need to wait for the official release, but this is the construction we can use for plugins.