Ros serialization #449
Merged
Ros serialization #449
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…he C target) whether CMake is used to compile or not
… requested. Slight bug fix
…n federated execution
…g an instance of their parent. Added copy functions
…ll to see what happens
…or to not use CMake
…arget if using CMake
…and repopulating it
…egardless of target compiler status
Co-authored-by: Marten Lohstroh <[email protected]>
…th their file and target configs instead of keeping a list of resources
Co-authored-by: Marten Lohstroh <[email protected]>
…imported .lf resources
`cmake-include` improvements
CCpp target
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR proposes the following changes:
Implements ROS serialization (and a modular serialization framework) for our C target federated runtime.
Limitation: The port types need to be
std::shared_ptr<**ROS2MessageType**>for the serialization to work correctly.Fixes the
_distribute.shscripts.Limitation: Platform support for remote federates is currently only enabled for Linux.
Modifies the C runtime so that all reactor definitions, including their actions, ports, reactions, and preambles are only selectively generated if they belong to a given federate (previously, all reactor definitions were being generated for all federates, causing linking conflicts in some cases).
Adds a C++ mode to the
CCmakeGeneratorthat will compile all generated.cfiles as if they were.cpp(needed for ROS 2 compatibility).Adds support for multi-threaded compilation in the CGenerator for federated programs, where each federate is compiled (i.e., cmake is invoked for each federate) in a separate thread.
Fixes an issue where the
filetarget property was not working correctly for federated programs.Changes the behavior for treating imported reactors that are used as federates:
.lffile (previously all preambles in all imported federates' files were included for all federates).cmake-includefile is provided for an imported federate in the original.lffile, that file is used to extend theCMakeLists.txtfor that federate, provided that nocmake-includeis provided in the current.lffile. This is particularly useful to handle different dependencies that might interfere with each other for different federates.Fixes issues with the combination of federated banks and multiports and decentralized coordination.
Fixes an issue where the trace table generation was not considering federates.
Changes the federated AST transformation so that the receiver action is now of a special target-specific type (as opposed to mimicking the original port type). This type is reported by
getNetworkBufferType().Moves
FederateInstance.xtendtoorg.lflang.federated.