pangeo-data / helm-chart

Pangeo helm charts
https://pangeo-data.github.io/helm-chart/
21 stars 26 forks source link

add graphviz to notebook docker image #43

Closed rabernat closed 6 years ago

rabernat commented 6 years ago

dask experts: what is the recommended way to install graphviz? (I know it has some complicated dependencies.)

I would like to be able to visualize dask graphs from pangeo.pydata.org.

xref: https://github.com/pangeo-data/pangeo/issues/74

rabernat commented 6 years ago

cc @martindurant as someone who might know something about this

mrocklin commented 6 years ago

apt install graphviz

then

pip install graphviz

You could also try

conda install python-graphviz

but I've been generally less happy with it over the years

On Mon, Jun 18, 2018 at 10:27 AM, Ryan Abernathey notifications@github.com wrote:

cc @martindurant https://github.com/martindurant as someone who might know something about this

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/pangeo-data/helm-chart/issues/43#issuecomment-398072823, or mute the thread https://github.com/notifications/unsubscribe-auth/AASszBzuRvAJwffMuKCOHnBi_OW6XEnXks5t97i7gaJpZM4Ur2yZ .

ocefpaf commented 6 years ago

conda install python-graphviz but I've been generally less happy with it over the years

I'm curious to know why and if there is anything we can do to fix that.

mrocklin commented 6 years ago

I don't know much about the conda-forge recipe. The defaults recipe has been broken on-and-off for years. I think the graphviz system library was only compiled to support a couple image formats or something. There was a long github issue on this topic at one point. I spent a couple minutes googling and couldn't find it though.

On Mon, Jun 18, 2018 at 11:25 AM, Filipe notifications@github.com wrote:

conda install python-graphviz but I've been generally less happy with it over the years

I'm curious to know why and if there is anything we can do to fix that.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/pangeo-data/helm-chart/issues/43#issuecomment-398093391, or mute the thread https://github.com/notifications/unsubscribe-auth/AASszAicLxQMV2n2LwNyY3CW1IQROjEDks5t98Z2gaJpZM4Ur2yZ .

martindurant commented 6 years ago

I seem to recall that the conda package needed some PNG extras in order to be useful from dask's point of view. I definitely did this at some point and could probably find those requirements again.

ocefpaf commented 6 years ago

Thanks @martindurant and @mrocklin. I don't want to hijack this thread with an off-topic question so I'll be brief: if you can open issues at https://github.com/conda-forge/python-graphviz-feedstock or, even better, add a test that ensures the options you need are enabled, I can look into fixing these things.

martindurant commented 6 years ago

I have found that for linux, the following combination works libiconv graphviz python-graphviz pango

rabernat commented 6 years ago

@martindurant -- I'd be happy if you could add this via PR.

I would also like to add gsw to both worker and client images (https://anaconda.org/conda-forge/gsw), so if you bother to rebuild the docker images, it would be great to throw that in as well.

rabernat commented 6 years ago

Closed by #45