Skip to content
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

Diagram Synthesis fails wih IllegalArgumentException when enabling "Reactor State Variables" #1930

Closed
cmnrd opened this issue Aug 3, 2023 · 2 comments · Fixed by #1932
Closed
Assignees
Labels
bug Something isn't working diagrams Problems with diagram synthesis
Milestone

Comments

@cmnrd
Copy link
Collaborator

cmnrd commented Aug 3, 2023

Steps to reproduce:

  1. Create a program that contains state variables
target Cpp

main reactor Example {
    state foo: int = 42
}
  1. Enable the "Reactor State Variables" option in VS code
@cmnrd cmnrd added bug Something isn't working diagrams Problems with diagram synthesis labels Aug 3, 2023
@cmnrd cmnrd added this to the 0.5.0 milestone Aug 3, 2023
@soerendomroes
Copy link
Collaborator

It seems to be a problem in the model.
Here
something tries to infer the target of an InitializerImpl which is an EObject but not a Model.

@cmnrd
Copy link
Collaborator Author

cmnrd commented Aug 4, 2023

Oh, right. This actually looks like a bug in FormattingUtil. @petervdonovan could you take a look at this? I assume that the diagram synthesis renders a string from a state variable to print in the diagram, but it fails because it cannot in infer the target.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working diagrams Problems with diagram synthesis
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants