Catalyst.jl icon indicating copy to clipboard operation
Catalyst.jl copied to clipboard

Published 20 hours ago • verified publisher icon SciML

Drop Graphviz and requires

Open isaacsas opened this issue 2 years ago • 5 comments

We need to find an actively maintained / developed graph plotting library to switch to though. GraphMakie.jl seems active, but that would pull in Makie (unless we split out the graph functionality to a sub-library like CatalystGraphs).

isaacsas avatar May 27 '22 23:05 isaacsas

A graph viz sub package isn't a bad idea.

ChrisRackauckas avatar May 27 '22 23:05 ChrisRackauckas

Looks like GraphViz may be fixed now too: https://github.com/JuliaPackaging/Yggdrasil/issues/1428, if so we can drop requires for it.

isaacsas avatar May 28 '22 00:05 isaacsas

This will have to wait on https://github.com/JuliaPackaging/Yggdrasil/issues/4961 as I can't play with using the jll currently.

isaacsas avatar May 29 '22 21:05 isaacsas

https://github.com/JuliaPackaging/Yggdrasil/issues/4961 is fixed, what about moving the __init__ function into an extension?

longemen3000 avatar May 17 '23 23:05 longemen3000

Last I checked, on some platforms the Graphviz artifact is fine but on others it still has issues (i.e. last time I looked it wasn't working via artifacts on macs). Do you have any issues the current approach is causing?

Honestly though, I'd prefer to just completely drop Graphviz entirely and switch to a native-Julia plotting approach if/when there are reasonably maintained libraries. When we set this up there didn't seem to be anything, but perhaps that has changed in the last year.

isaacsas avatar May 18 '23 12:05 isaacsas