Ross Barnowski

Results 296 comments of Ross Barnowski

Thanks for the suggestion @Nagael , it does indeed work as intended on all platforms (at least according to CI). I don't think the code duplication is a big deal...

The naming here is confusing: note that the `graphviz` package on PyPI is *not* GraphViz project providing the C-libraries that `pygraphviz` wraps, but rather [a different project providing it's own...

Glad that worked for you @brando90 It seems that the Installation documentation could be made more clear. Two points stand out from the discussion above: 1. Adding a note to...

Good point, especially the part about conda-forge because if you are going to use conda, you definitely *should* use the conda-forge channel (at least as of today) rather than the...

FWIW #333 updates the docs with more up-to-date conda instructions, along with CI jobs to regularly test the conda-forge channel. I just force-pushed to retrigger the doc build so it...

> is this issue solved by doing conda install -y pygraphviz -c conda-forge? `conda-forge` provides a working solution for conda users, but the most general solution would be to provide...

@aguytech is this issue resolved in your second comment or is there still something that needs to be addressed?

Thanks @laspir for the excellent minimal reproducer + analysis! It does indeed seem like some of the attributes are not being mapped to the appropriate objects (node_attr on the subgraph...

I'm not especially familiar with this specific packaging workflow and I can't reproduce, probably because I don't have a system-level pytest instance installed. FWIW to build/test wheels I would do...

> pep517 build procedure it is not packaging specific thing. Yes, I understand, but I have no problems with the build. I can't think of any reason why `--no-isolation` would...