b-robotized / b-robotized/beckhoff_ads_driver

BeckhoffADSHardwareInterface::on_init uses wrong parameter type

Open Beginner friendly
#9 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
17
Forks
11
PR merge metrics
No merged PRs in 30d

Description

`BeckhoffADSHardwareInterface::on_init` is declared with `HardwareComponentParams` instead of `HardwareComponentInterfaceParams`, which means it does not override the virtual method in `hardware_interface::HardwareComponentInterface` and is never called by the ros2_control.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start at BeckhoffADSHardwareInterface::on_init and compare its declared parameter type with the virtual method in hardware_interface::HardwareComponentInterface. Update the declaration so it overrides the expected method, then verify that ros2_control recognizes and calls on_init.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
embedded-iot
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
68/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.