py-why / dowhy

DoWhy is a Python library for causal inference that supports explicit modeling and testing of causal assumptions. DoWhy is based on a unified language for causal inference, combining causal graphical models and potential outcomes frameworks.

Home Page:https://www.pywhy.org/dowhy

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Graphviz installation : --include-path not recognized anymore

sinhaharsh opened this issue · comments

README provides the following command to install graphviz,

pip install pygraphviz --install-option="--include-path=/usr/include/graphviz" \
--install-option="--library-path=/usr/lib/graphviz/"

However this command doesn't work anymore as per pypa/setuptools#2740
It should be modified to

pip install --global-option=build_ext --global-option="-I/usr/local/include/graphviz/" --global-option="-L/usr/local/lib/graphviz" pygraphviz

Thank you @sinhaharsh . I wonder if we should just link to the pygraphviz INSTALL instructions at https://pygraphviz.github.io/documentation/stable/install.html

Thoughts?

A link to pygraphviz install instructions already exists as a note
Screenshot 2024-04-29 at 1 15 58 PM

Ah, I missed that somehow. Thanks @sinhaharsh . Would you like to submit a PR with your README change? This will help record your contribution.

It seems the note is present at the documentation page for Contributing-Code. However, no such note is present in the README and on the installation page.

There are only 3 occurrences for the following code across the repository.

Sure. I will send a PR for these changes.