Skip to content
This repository has been archived by the owner on Jul 12, 2022. It is now read-only.

Wrong graphviz requirement #7

Open
vinci1it2000 opened this issue Oct 16, 2019 · 3 comments
Open

Wrong graphviz requirement #7

vinci1it2000 opened this issue Oct 16, 2019 · 3 comments
Assignees
Labels
bug Something isn't working invalid This doesn't seem right

Comments

@vinci1it2000
Copy link
Member

The graphviz requirement defined in the file is wrong. Please update so the issue JRCSTU/co2wui#146 can be closed.

The right library is python-graphviz<=0.8.4 form conda-forge channel. Please update in all recipes and check in the image that you have installed the right library and version. I tested with @ashenafimenza and @dimitriskomnos.

@vinci1it2000 vinci1it2000 added the bug Something isn't working label Oct 16, 2019
@ankostis ankostis added the invalid This doesn't seem right label Oct 16, 2019
@ankostis
Copy link
Member

You seem to be confusing:

  • python-graphviz (python) and
  • graphviz the native OS suite.

The former, depends on the later, you can't have one or the other, you need both in conda.

@ankostis
Copy link
Member

ankostis commented Oct 16, 2019

The right library is python-graphviz<=0.8.4 form conda-forge channel.

Wrong diagnosis, but correct prescription :-)

We have the same lib in the exe:

(base) C:\Users\vagrant\Documents>conda list|grep graphviz
graphviz                  2.38.0            h6538335_1011    conda-forge
python-graphviz           0.13                       py_0    conda-forge

When downgraded to 0.8.4 as you suggested, it works:

conda install python-graphviz=0.8.4
Collecting package metadata (current_repodata.json): done
Solving environment: done
...

(base) C:\Users\vagrant\Documents>conda list|grep graphviz
graphviz                  2.38.0            h6538335_1011    conda-forge
python-graphviz           0.8.4                    py37_1
  • Still researching why 0.8.4 doesn't burp with UNCs....
  • It doesn't need the -c conda-forge channel, that's the default.

@stefanocorsi you may simply dowgrade this in the conda package.
It's yours now, close this.

@ankostis ankostis reopened this Oct 16, 2019
@ankostis
Copy link
Member

Nice catch Vinz. Your personal persistence leads us to an unexpected solution!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working invalid This doesn't seem right
Projects
None yet
Development

No branches or pull requests

3 participants