-
Notifications
You must be signed in to change notification settings - Fork 63
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
Added support for Modal Models #501
Conversation
Conflicts: org.lflang.diagram/src/org/lflang/diagram/synthesis/LinguaFrancaSynthesis.xtend org.lflang.diagram/src/org/lflang/diagram/synthesis/styles/LinguaFrancaShapeExtensions.xtend org.lflang.diagram/src/org/lflang/diagram/synthesis/util/UtilityExtensions.xtend org.lflang/src/org/lflang/scoping/LFScopeProvider.xtend
Conflicts: org.lflang/src/org/lflang/LinguaFranca.xtext
Conflicts: org.lflang/src/org/lflang/generator/ReactionInstance.xtend org.lflang/src/org/lflang/generator/c/CGenerator.xtend
Conflicts: org.lflang/src/org/lflang/generator/c/CGenerator.xtend
Conflicts: org.lflang/src/lib/c/reactor-c org.lflang/src/org/lflang/federated/FederateInstance.xtend
Conflicts: org.lflang.diagram/src/org/lflang/diagram/synthesis/LinguaFrancaSynthesis.xtend org.lflang.diagram/src/org/lflang/diagram/synthesis/action/AbstractAction.xtend org.lflang.diagram/src/org/lflang/diagram/synthesis/action/CollapseAllReactorsAction.xtend org.lflang.diagram/src/org/lflang/diagram/synthesis/action/ExpandAllReactorsAction.xtend org.lflang.diagram/src/org/lflang/diagram/synthesis/action/MemorizingExpandCollapseAction.xtend org.lflang.diagram/src/org/lflang/diagram/synthesis/styles/LinguaFrancaShapeExtensions.xtend
This PR now includes changes of PR #940 |
Conflicts: org.lflang/src/lib/c/reactor-c org.lflang/src/org/lflang/ASTUtils.java org.lflang/src/org/lflang/validation/LFValidator.java
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks ready to me! A very cool feature that I think will find lots of use.
* This methods needs to be overridden in target specific code generators that support modal reactors. | ||
*/ | ||
protected def void transformConflictingConnectionsInModalReactors(Collection<Connection> transform) { | ||
errorReporter.reportError("The currently selected code generation is missing an implementation for conflicting transforming connections in modal reactors.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
errorReporter.reportError("The currently selected code generation is missing an implementation for conflicting transforming connections in modal reactors.") | |
errorReporter.reportError("This code generator is missing an implementation for transforming conflicting connections in modal reactors.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a real tour de force! I have not reviewed much, but I think we should merge because almost anything we do will lead to more conflicts.
experimental/C/src/ModalModels/BehaviorTrees/robohub_example_advanced.lf
Outdated
Show resolved
Hide resolved
experimental/C/src/ModalModels/BehaviorTrees/robohub_example_advanced.lf
Outdated
Show resolved
Hide resolved
…dvanced.lf Co-authored-by: Edward A. Lee <[email protected]>
Prototype for modal reactors. See also reactor-c PR #6
Currently only supports C code generation + non-federated.
ToDo:
ToDiscuss: