Skip to content

Backports - #99

Merged
mjcarroll merged 5 commits into
lyricalfrom
ahcorde/lyrical/backports
Oct 20, 2026
Merged

mjcarroll merged 5 commits into
lyricalfrom
ahcorde/lyrical/backports

Conversation

@ahcorde

@ahcorde ahcorde commented Oct 20, 2026

Copy link
Copy Markdown
Contributor
  • Use NodeInterfaces for tf2_ros constructors
  • Use extras_require for turtle_tf2_py test dependencies
  • Include what you use
  • Update Lyrical CI
  • Python warning

mjcarroll and others added 5 commits October 20, 2026 10:18
geometry2 0.46.2 removed the deprecated tf2_ros constructors that took a
node pointer or separate node interfaces (ros2/geometry2#940), which breaks
the rolling binary build for this package.

Pass the node itself instead: rclcpp::node_interfaces::NodeInterfaces has an
implicit constructor taking a node reference.

Assisted-by: Claude Opus 5 <noreply@anthropic.com>
Signed-off-by: Michael Carroll <mjcarroll.oss@gmail.com>
Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
@ahcorde
ahcorde requested a review from mjcarroll October 20, 2026 08:27
@ahcorde ahcorde self-assigned this Oct 20, 2026
@mjcarroll
mjcarroll merged commit a19b276 into lyrical Oct 20, 2026
8 checks passed
@mjcarroll
mjcarroll deleted the ahcorde/lyrical/backports branch October 20, 2026 21:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants