search

seflless / diagrams

Generate Flowcharts, Network Sequence Diagrams, GraphViz Dot Diagrams, and Railroad Diagrams
MIT License
764 stars 114 forks source link

CLI hangs on rendering flowchart diagrams, but works with sequence #24

Open ricpelo opened 5 years ago

ricpelo commented 5 years ago

It seems to be related with #14, but it's the opposite: now diagrams CLI hangs on rendering flowchart diagrams but works with sequence.

System info

Steps to reproduce

Install CLI

$ npm install -g diagrams

Create flowchart diagram source (input.flowchart)

cat > input.flowchart <<EOF
st=>start: Start:>http://www.google.com[blank]
e=>end:>http://www.google.com
op1=>operation: My Operation
sub1=>subroutine: My Subroutine
cond=>condition: Yes
or No?:>http://www.google.com
io=>inputoutput: catch something...
para=>parallel: parallel tasks
st->op1->cond
cond(yes)->io->e
cond(no)->para
para(path1, bottom)->sub1(right)->op1
para(path2, top)->op1
EOF

Run command to render SVG

$ diagrams flowchart input.flowchart flowchart.svg

It hangs forever.

ricpelo commented 5 years ago

I'd just realize that the issue is in the sample input.flowchart. The following diagram source works OK as expected:

cat > input.flowchart <<EOF
st=>start: Start:>http://www.google.com[blank]
e=>end:>http://www.google.com
op1=>operation: My Operation
sub1=>subroutine: My Subroutine
cond=>condition: Yes
or No?:>http://www.google.com
io=>inputoutput: catch something...
st->op1->cond
cond(yes)->io->e
cond(no)->para
EOF

It seems to be a problem with the para nodes. After removing them from the diagram source, the issue is gone.

akatsuki105 commented 4 years ago

I also have this problem. My flowchart contains para too. The version is 0.11.0.

sommella commented 4 years ago

Same for me, any news on this issue?

Francois-Laberge-Bose commented 4 years ago

@Akatsuki-py @skonvols2k Hey guys, sorry but I don't have time at the moment for looking into this. I'm a new parent at the moment and getting no sleep, nor spare time. If one of you has time to see if maybe upgrading dependencies or something helps, I can maybe make time to redeploy.

Hope you guys are having a decent time with all of the WFH.

JoshuaCrestone commented 4 years ago

I've created a pull request that solves this issue by upgrading flowchart.js and making Raphael an explicit dependency. Congratulations on being a parent!