Using a version of up-transport-zenoh which supports being called from within a non-Tokio runtime #18
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
* Add support for tracking subscriptions and forwarding published messages in ustreamer.rs * Add usubscription-static-file to stub out subscriptions before usubscription is written * Add tests for mE (SOME/IP) subscribe/publish and uE (zenoh) subscribe/publish NOTE: This PR also contains changes that @PLeVasseur has done for up-streamer-rust. His will be merged first and then I will rebase based on that merge.
Adding project to Cargo.toml Noted area where we fail. Seems to be when initializing Session based on Runtime. Had to lower the rustc version of up-rust to 1.72.0 and thus put the up-transport libs on test branches using that change, but now appears like it works :) Able to use 1.74.0. Should now start to roll back my bespoke changes to allow us to use 1.72.0. Rolled back to using stock branches. Was able to use Zenoh on its 0.11.0-rc3, but changing its rust-toolchain.toml to bump to 1.74.0 so that we have compatability between zenohd and the up-linux-streamer-plugin Addind documentation on how to build and use the up-linux-streamer-plugin with the own-compiled version of zenoh Add notes on where issues are at Several more attempts to try to read the Config from the Runtime
…io Runtime it can make use of for sending Request messages.
PLeVasseur
pushed a commit
that referenced
this pull request
Aug 12, 2024
* Add support for tracking subscriptions and forwarding published messages in ustreamer.rs * Add usubscription-static-file to stub out subscriptions before usubscription is written * Add examples for mE (SOME/IP) subscribe/publish and uE (zenoh) subscribe/publish * Add information about pub/sub in READMEs * Support zenoh configuration in command line file * Add test for usubscription with bad data Implements [#18]
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.
Hey @matthewd0123 -- check out the latest commit on here, in particular the change to use a branch I made for up-transport-zenoh.
I've confirmed that now all of the flows work for all pub/sub and RPC.