ICube-Robotics / ethercat_driver_ros2

Hardware Interface for EtherCAT module integration with ros2_control
https://icube-robotics.github.io/ethercat_driver_ros2/
Apache License 2.0
125 stars 32 forks source link

Ethercat master abstraction #97

Open yguel opened 6 months ago

yguel commented 6 months ago

Create master abstraction to enable seemless using of different ethercat masters (e.g. etherlab, SOEM).

github-actions[bot] commented 6 months ago

Test Results

186 tests  +59   152 :heavy_check_mark: +47   0s :stopwatch: ±0s   44 suites +15     34 :zzz: +12    44 files   +15       0 :x: ±  0 

Results for commit b8eaa123. ± Comparison against base commit 396cb1a6.

This pull request removes 10 and adds 69 tests. Note that renamed tests count towards both. ``` ethercat_generic_cia402_drive.EcCiA402DriveTest ‑ SlaveSetupDomains ethercat_generic_cia402_drive.EcCiA402DriveTest ‑ SlaveSetupPdoChannels ethercat_generic_cia402_drive.EcCiA402DriveTest ‑ SlaveSetupSyncs ethercat_generic_slave.GenericEcSlaveTest ‑ SlaveSetupDomains ethercat_generic_slave.GenericEcSlaveTest ‑ SlaveSetupPdoChannels ethercat_generic_slave.GenericEcSlaveTest ‑ SlaveSetupSyncs ethercat_interface.copyright ‑ src/ec_master.cpp ethercat_interface.cppcheck ‑ src/ec_master.cpp ethercat_interface.cpplint ‑ /ros/src/ethercat_driver_ros2/ethercat_interface/src/ec_master.cpp ethercat_interface.uncrustify ‑ src/ec_master.cpp ``` ``` ethercat_interface.copyright ‑ include/ethercat_interface/ec_buffer_tools.h ethercat_interface.cppcheck ‑ include/ethercat_interface/ec_buffer_tools.h ethercat_interface.cpplint ‑ /ros/src/ethercat_driver_ros2/ethercat_interface/include/ethercat_interface/ec_buffer_tools.h ethercat_interface.uncrustify ‑ include/ethercat_interface/ec_buffer_tools.h ethercat_master_etherlab.EtherlabSlaveTest ‑ EcReadTPDOToStateInterface ethercat_master_etherlab.EtherlabSlaveTest ‑ EcWriteRPDODefaultValue ethercat_master_etherlab.EtherlabSlaveTest ‑ EcWriteRPDOFromCommandInterface ethercat_master_etherlab.EtherlabSlaveTest ‑ SlaveSetup ethercat_master_etherlab.EtherlabSlaveTest ‑ SlaveSetupDomains ethercat_master_etherlab.EtherlabSlaveTest ‑ SlaveSetupPdoChannels … ```
This pull request removes 1 skipped test and adds 13 skipped tests. Note that renamed tests count towards both. ``` ethercat_interface.cppcheck ‑ src/ec_master.cpp ``` ``` ethercat_interface.cppcheck ‑ include/ethercat_interface/ec_buffer_tools.h ethercat_master_etherlab.cppcheck ‑ include/ethercat_master/ec_master_etherlab.hpp ethercat_master_etherlab.cppcheck ‑ include/ethercat_master/ec_slave_etherlab.hpp ethercat_master_etherlab.cppcheck ‑ src/ec_master_etherlab.cpp ethercat_master_etherlab.cppcheck ‑ src/ec_slave_etherlab.cpp ethercat_master_etherlab.cppcheck ‑ test/test_etherlab_slave.cpp ethercat_master_etherlab.cppcheck ‑ test/test_etherlab_slave.hpp ethercat_master_etherlab.cppcheck ‑ test/test_load_ec_master_etherlab.cpp ethercat_master_mock.cppcheck ‑ include/ethercat_master/ec_master_mock.hpp ethercat_master_mock.cppcheck ‑ include/ethercat_master/ec_slave_mock.hpp … ```

:recycle: This comment has been updated with latest results.