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

Better multiport support for Rust #1406

Merged
merged 12 commits into from
Nov 24, 2022
Merged

Conversation

oowekyala
Copy link
Collaborator

This is the companion branch to lf-lang/reactor-rs#29.

@oowekyala oowekyala force-pushed the rust.support-better-multiports branch from f1358c4 to 2d9d77a Compare October 11, 2022 11:58
@oowekyala oowekyala force-pushed the rust.support-better-multiports branch from 2d9d77a to 9b21c94 Compare October 20, 2022 13:42
@oowekyala oowekyala force-pushed the rust.support-better-multiports branch from 73fdd6d to c6ea5be Compare November 9, 2022 15:51
@cmnrd
Copy link
Collaborator

cmnrd commented Nov 11, 2022

Is this ready for review? From the rust benchmark test failure I assume this also needs to be accompanied by changes in the benchmark repo, right?

@oowekyala
Copy link
Collaborator Author

oowekyala commented Nov 11, 2022

Thanks for the pointer, I had forgotten about that. I'll open a PR there.

I think the changes are stable now so I will mark this as ready for review

@oowekyala oowekyala marked this pull request as ready for review November 11, 2022 11:37
@jhaye
Copy link
Collaborator

jhaye commented Nov 14, 2022

Seems like this slightly changes the API for ReactionCtx::get(…). The benchmarks should probably be adjusted accordingly.

oowekyala added a commit to lf-lang/benchmarks-lingua-franca that referenced this pull request Nov 14, 2022
@oowekyala oowekyala force-pushed the rust.support-better-multiports branch from 6217b25 to a4bb59e Compare November 14, 2022 14:53
@oowekyala oowekyala force-pushed the rust.support-better-multiports branch from cd76a4c to d82f816 Compare November 15, 2022 18:31
oowekyala added a commit to lf-lang/benchmarks-lingua-franca that referenced this pull request Nov 16, 2022
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.

This looks great! I can take care of merging the PRs in the different repos, as it requires a bit of care to keep everything in sync.

@cmnrd cmnrd merged commit bfa688e into master Nov 24, 2022
@cmnrd cmnrd deleted the rust.support-better-multiports branch November 24, 2022 16:12
@petervdonovan petervdonovan added the enhancement Enhancement of existing feature label Jan 26, 2023
@petervdonovan petervdonovan added the rust Related to the Rust target label Jan 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement of existing feature rust Related to the Rust target
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants