sismic icon indicating copy to clipboard operation
sismic copied to clipboard

Error in visualizing `tests/yaml/deep_history.yaml`

Open rajivr opened this issue 2 years ago • 5 comments

Firstly, thanks a lot for this tool. I accidentally discovered it, and I am really glad I found it.

I am new to sismic. I was trying to visualize tests/yaml/deep_history.yaml, and it looks likes plantuml is throwing an error.

Sismic version is current master (commit e8b585f90d)

Here are the steps that I followed.

$ sismic-plantuml deep_history.yaml > deep_history.uml

$ plantuml deep_history.uml 
Error line 39 in file: deep_history.uml
Some diagram description contains errors

$ plantuml -version
PlantUML version 1.2021.9 (Sun Jul 25 15:43:56 IST 2021)
(GPL source distribution)
Java Runtime: OpenJDK Runtime Environment
JVM: OpenJDK 64-Bit Server VM
Default Encoding: UTF-8
Language: en
Country: US
 
PLANTUML_LIMIT_SIZE: 4096

Dot version: dot - graphviz version 2.47.3 (0)
Installation seems OK. File generation OK

Here is the image that is generated by plantuml.

deep_history

rajivr avatar Aug 30 '21 02:08 rajivr

Hello,

Thanks for having reported this issue! I'll have a look at it within the next few days. :)

AlexandreDecan avatar Aug 30 '21 07:08 AlexandreDecan

For what I see, it seems this "issue" is on PlantUML side. The current implementation does not support concurrent state with outgoing transitions. A similar limitation arises when trying to visualize "nested_parallel.yaml", but not "elevator.yaml" for example.

I have no idea what I could do to fix this, unfortunately... :(

Related discussions:

  • https://forum.plantuml.net/2471/arrow-from-concurrent-state-outside-outer-composite-state
  • https://forum.plantuml.net/13394/statechart-leaving-parallel-transition-within-parallel-region

AlexandreDecan avatar Aug 30 '21 07:08 AlexandreDecan

If PlantUML does not respect the UML 2.5 specification w.r.t. what is possible to specify in certain statecharts, then it is indeed difficult to do anything about it from the side of Sismic. PlantUML should try to fix this problem. (I do not know if they will do this anywhere soon, given that some of the issues are "due to the current implementation" of PlantUML.

tommens avatar Aug 30 '21 09:08 tommens

The first (above) discussion dates back to 2014. The second one was created this year, in March. I hope it will receive more feedback :)

AlexandreDecan avatar Aug 30 '21 10:08 AlexandreDecan

Thanks @tommens and @AlexandreDecan for the feedback and the pointers.

rajivr avatar Aug 31 '21 12:08 rajivr