-
-
Notifications
You must be signed in to change notification settings - Fork 26
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
Error in visualizing tests/yaml/deep_history.yaml
#111
Comments
Hello, Thanks for having reported this issue! I'll have a look at it within the next few days. :) |
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: |
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. |
The first (above) discussion dates back to 2014. The second one was created this year, in March. I hope it will receive more feedback :) |
Thanks @tommens and @AlexandreDecan for the feedback and the pointers. |
Sorry, we made no progress on this issue. Yesterday, we discovered that PlantUML introduced a backward incompatible change in their render pipeline for statecharts. This "bug" was reported to them more than one year ago (by someone else), and they did nothing about it, so I doubt you should expect any fix for the above issue given how reactive they are :-/ I'll have a look "soon" at Mermaid.JS to replace/complement PlantUML in Sismic, but last time I looked at it, it was far from supporting all the features we need. |
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 likesplantuml
is throwing an error.Sismic version is current master (commit
e8b585f90d
)Here are the steps that I followed.
Here is the image that is generated by
plantuml
.The text was updated successfully, but these errors were encountered: