-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Stack overflow in Spring & SFDP layouts #67
Comments
Can you post the output of |
Here you go:
Here's also my version info:
|
Just to be clear - there doesn't seem to be any stack trace generated. If I catch the exception object, it looks like this:
|
using Pkg
pkg"activate --temp"
pkg"dev NetworkLayout"
pkg"add GraphIO, Graphs, GeometryBasics, JSON3"
using GraphIO.EdgeList, Graphs
using NetworkLayout
using GeometryBasics: Point
using JSON3
tmpdir = mktempdir()
dl = download("https://github.com/user-attachments/files/17196286/data.tgz", joinpath(tmpdir, "data.tgz"))
run(`tar -xzf $dl -C $tmpdir`)
gdat = joinpath(tmpdir, "graph.e")
pdat = joinpath(tmpdir, "pin.json")
idat = joinpath(tmpdir, "initialpos.json")
@assert isfile(gdat) && isfile(pdat) && isfile(idat)
println("Loading data")
g = loadgraph(gdat, EdgeListFormat())
pin = identity.(JSON3.read(pdat))
initialpos = identity.(JSON3.read(idat))
NetworkLayout.sfdp(g; pin, initialpos, iterations = 5)
NetworkLayout.spring(g; pin, initialpos, initialtemp = 3.940695496278636, iterations = 5) here's a fully contained example including downloading and unzipping the data in a temp dir. Maybe others wanna try it out too. I still cannot reproduce, neither on 1.10 nor on 1.11... On the first glance i could not find a recursion in the layouts. Does this happen also without pin/initial position or only if you provide those arguments? |
My main use case is
It also succeeds with just
|
I also just checked with |
I have some somewhat-large graph data that I'm attempting to do partial layouts on, and I'm hitting
StackOverflowError
s when I try. I'm attaching the data, and here's the code I'm using for layout:data.tgz
Any thoughts? I did also try converting
g
to an undirected graph, but that doesn't seem to make any difference.The text was updated successfully, but these errors were encountered: