Skip to content

Fix test for ros/jazzy#7

Open
tpoignonec wants to merge 3 commits intob-robotized:masterfrom
tpoignonec:tpoignonec/fix_test
Open

Fix test for ros/jazzy#7
tpoignonec wants to merge 3 commits intob-robotized:masterfrom
tpoignonec:tpoignonec/fix_test

Conversation

@tpoignonec
Copy link

@tpoignonec tpoignonec commented Jan 13, 2025

Hi,

The test does not build as it uses the constructor with only the URDF as opposed to

  explicit ResourceManager(
    const std::string & urdf,
    rclcpp::node_interfaces::NodeClockInterface::SharedPtr clock_interface,
    rclcpp::node_interfaces::NodeLoggingInterface::SharedPtr logger_interface,
    bool activate_all = false, const unsigned int update_rate = 100);

as defined here.

This PR fixes the issue for jazzy.

@mamueluth is this a package targeted for jazzy initially? Otherwise, the appropriate branch can be created before merging this.

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.

1 participant