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

Declarative Port Graph in C++ Runtime #1848

Merged
merged 23 commits into from
Jul 27, 2023
Merged

Conversation

tanneberger
Copy link
Member

@tanneberger tanneberger commented Jun 15, 2023

Declarative Port Graph

This pull requests adjusts the code generator to use the new port graph interface supplied by reactor-cpp.

Previously Ports and Connections were wired together directly by the code generator in the assembly method. This Pull-Request changes this behavior that connections between ports are not declared in the environment. This allows the environment to optimize a lot of connections away and then wire up the components.

Previously

reactor1.out.bind_to(reactor2.in);

Now

env.draw_connection(reactor1.out, reactor2.in, ConnectionProperties{});

Also different Connection Types can be annotated over this interface like ConnectionType::EnclavedDelayed, ConnectionType::PhysicalConnection and many more.

This PR works together with the changes in lf-lang/reactor-cpp/pull/51

@tanneberger tanneberger requested a review from cmnrd June 15, 2023 17:23
@tanneberger tanneberger self-assigned this Jun 15, 2023
@tanneberger tanneberger added enhancement Enhancement of existing feature cpp Related to C++ target labels Jul 26, 2023
@tanneberger tanneberger requested a review from cmnrd July 26, 2023 11:19
@tanneberger tanneberger changed the title [DRAFT] Declarative Port Graph Declarative Port Graph Jul 26, 2023
@cmnrd cmnrd changed the title Declarative Port Graph Declarative Port Graph in C++ Runtime Jul 26, 2023
Copy link
Collaborator

@cmnrd cmnrd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great! Thanks for all the hard work!

updating reactor-cpp

updating reactor-cpp

update reactor-cpp
@cmnrd cmnrd force-pushed the cpp-connection-optimizations branch from ce53467 to bb42523 Compare July 27, 2023 10:56
@cmnrd cmnrd enabled auto-merge July 27, 2023 10:56
@cmnrd cmnrd added this pull request to the merge queue Jul 27, 2023
Merged via the queue into master with commit 6dfa470 Jul 27, 2023
43 checks passed
@cmnrd cmnrd deleted the cpp-connection-optimizations branch July 27, 2023 14:33
@petervdonovan petervdonovan added refactoring Code quality enhancement and removed enhancement Enhancement of existing feature labels Aug 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cpp Related to C++ target refactoring Code quality enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants