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

SW-4972: merge switching to static transform publisher contribution foxy #125

Conversation

Samahu
Copy link
Contributor

@Samahu Samahu commented May 5, 2023

Related Issues & PRs

Summary of Changes

  • Send static transforms only once with a static transform broadcaster

Validation

  • [optional] Launch sensor node (xml | py) and replay / record nodes.

Samahu and others added 11 commits February 20, 2023 13:39
* Update to the latest ouster sdk
* Forward multicast funcitonality + Other improvements and fixes
* Add service_msgs dependency to package.xml
* Correct sensor_mtp.launch for ros2 launch file format
* Move to most recent SDK update
* Declare and fill defaults for mtp paramters + fix uninitialzed compute_to_scan
* Launch file rename and README corrections
…ith latched topics (#102)

* Working port of latched metadata topic on ros2
* Update replay and record launch files to providing metadata file an optional parameter
* Remove extra white space in replay record command
* Undo changes to the metadata-qos-override
* minor code syntax improvements
* Add missing metadata topic when bag file isn't specified
* Use concise syntax and formatting
* Reverse logic for easier read
* Apply node transition if it exists
…parameters (#115)

* deprecate tf_prefix from os_cloud (#96)
Co-authored-by: Guillaume Doisy <[email protected]>

* Squashed commit of the following:
commit 6280bfa1178bdee4fe695cb4752efd5ff15279db
Author: Ussama Naal <[email protected]>
Date:   Fri Apr 28 07:54:34 2023 -0700
    Merge branch 'deprecate_tf_prefix'

commit 35f2fd2
Author: Guillaume Doisy <[email protected]>
Date:   Mon Apr 3 18:12:44 2023 +0100
    deprecate tf_prefix from os_cloud

* Update ChangeLog and package version
* Propagate the parameters to launch files
* Add a TODO note
---------

Co-authored-by: Guillaume Doisy <[email protected]>
…ocess (#108)

* Remove the use of static vars within components
* Resolve conflicts and update changelog and version
* Fix a typo 'instance'
* use separate params for tf frames
* send static transforms once
* Revert "use separate params for tf frames"
This reverts commit 4361969.
…ontribution' into SW-4972-merge-switching-to-static-transform-publisher-contribution-foxy
@Samahu Samahu added the enhancement New feature or request label May 5, 2023
@Samahu Samahu self-assigned this May 5, 2023
@Samahu Samahu changed the base branch from master to ros2-foxy May 5, 2023 19:25
@Samahu Samahu changed the title Sw 4972 merge switching to static transform publisher contribution foxy SW-4972: merge switching to static transform publisher contribution foxy May 5, 2023
@Samahu Samahu marked this pull request as ready for review May 5, 2023 19:48
@Samahu Samahu merged commit 43248fd into ros2-foxy May 5, 2023
1 check passed
@Samahu Samahu deleted the SW-4972-merge-switching-to-static-transform-publisher-contribution-foxy branch May 5, 2023 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

High TF Broadcast Rate
3 participants