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

Feature/1.5.8/zenoh utransport impl #65

Merged

Commits on Jul 8, 2024

  1. Initial draft of ZenohUTransport implementation

    Most of the basic functionality is present, but there are still several
    TODO items remaining:
    
    * Rework invoke_nonblock_callback call
    * Finish all TODO comments in code files
    * Implement cleanupListener
    * Add and finish tests
    * Fix anything that fails tests
    * Switch to Zenoh 1.0 when available
    sashacmc authored and gregmedd committed Jul 8, 2024
    Configuration menu
    Copy the full SHA
    0a12a8d View commit details
    Browse the repository at this point in the history

Commits on Jul 25, 2024

  1. Patch draft implementation to get it building

    * Remove use to gtest outside of test tree.
    * Address error message from zenohcpp library.
    * Switch to our temporary zenohcpp conan recipe.
    gregmedd committed Jul 25, 2024
    Configuration menu
    Copy the full SHA
    0b2e638 View commit details
    Browse the repository at this point in the history

Commits on Jul 26, 2024

  1. Updating ZenohUTransport with revised API

    Updating up-cpp version, rewriting pub/sub test to use L2 APIs, updating
    the transport to use the revised up-cpp interface from up-cpp#240, and
    fixing some general bugs.
    
    Also adds a zenoh config json for use with the tests.
    gregmedd committed Jul 26, 2024
    Configuration menu
    Copy the full SHA
    7388a7e View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    f82c1d1 View commit details
    Browse the repository at this point in the history

Commits on Jul 27, 2024

  1. Updating CI and pointing at pre-release up-cpp

    The pre-release up-cpp has critical bugfixes required for this code to
    build.
    
    Also makes sure the test configs are captured in the build artifacts so
    tests can run in CI.
    gregmedd committed Jul 27, 2024
    Configuration menu
    Copy the full SHA
    ab53e69 View commit details
    Browse the repository at this point in the history
  2. Temporarily removing test pass requirement for PRs

    We know that the current state of this implementaiton does not fully
    work. Making this change is a compromise - it keeps the tests active and
    running so we can see the output as we work on resolving issues, but
    does not prevent us from merging early, incomplete code.
    
    Once this implementation is more stable, we will re-enable this check
    gregmedd committed Jul 27, 2024
    Configuration menu
    Copy the full SHA
    db76fd6 View commit details
    Browse the repository at this point in the history