nipype icon indicating copy to clipboard operation
nipype copied to clipboard

Remove `dot` dependency

Open mgxd opened this issue 3 years ago • 1 comments
trafficstars

Summary

Nipype still uses pydot for graph rendering:

https://github.com/nipy/nipype/blob/bf48f7722f057a1eb2005390eb0a7ebcd35ee8bf/nipype/pipeline/engine/utils.py#L1381

This has lead to problems (https://github.com/nipreps/fmriprep/issues/2870) and is on its way out in networkx (https://github.com/networkx/networkx/issues/5723), and might just easily be replaced with nx.nx_agraph.write_dot

mgxd avatar Oct 06 '22 19:10 mgxd

Actually this line:

https://github.com/nipy/nipype/blob/bf48f7722f057a1eb2005390eb0a7ebcd35ee8bf/nipype/pipeline/engine/utils.py#L1373

At the very least nipype should set a timeout and allow a TimeoutError to be raised.

effigies avatar Dec 03 '22 02:12 effigies