terrastruct / TALA

A diagram layout engine designed specifically for software architecture diagrams
https://terrastruct.com/tala
Other
194 stars 4 forks source link

Tala error with connection between two sequence diagrams #79

Closed judepayne closed 4 months ago

judepayne commented 4 months ago

Hi, Found an error with Tala (causes Tala to crash) for this d2:

seq1: Process One {
  shape: sequence_diagram
  style: {
    stroke: '#a6a6a6'
    stroke-width: 3
    border-radius: 8
    fill: '#e3eefa'
    shadow: true
  }
  # Remember that semicolons allow multiple objects to be defined in one line
  # Actors will appear from left-to-right as a, b, c, d...
  a
  b
  c
  d
  # ... even if the connections are in a different order
  c -> d
  d -> a: close the loop
  b -> d
}
seq2: Process Two {
  shape: sequence_diagram
  style: {
    stroke: '#366631'
    stroke-width: 3
    border-radius: 8
    fill: honeydew
    shadow: true
  }
  x: step 1
  y: step 1
  z: step 3
  x -> y
  y -> z
  z -> x: close the loop
}
seq1.d -> seq2.x: Mgr approval

This works in dagre and elk. It's just the very last line:

seq1.d -> seq2.x: Mgr approval

that causes the issue when introduced.

alixander commented 4 months ago

Appreciate it @judepayne , will be fixed in next release