search

leducp / KickCAT

A C++ open source EtherCAT master stack
Other
59 stars 13 forks source link

Refactor Mailbox Configuration #176

Closed nicolas71640 closed 4 days ago

nicolas71640 commented 4 weeks ago

Change the way of SyncManagers configuration. Originally we configure the SyncManagers statically : The AL gives the configuration. This can be avoided as the master should be the one to configure the mailbox sync managers thanks to the eeprom. And the slave only need to check the coherency of this configuration

This PR also export the SII parser to common folder

nicolas71640 commented 3 weeks ago

fix #179 #178

github-actions[bot] commented 3 weeks ago

File Coverage Lines Branches
All files 40% 59% 21% :x:
lib/include/kickcat/Error.h 79% 75% 83% :white_check_mark:
lib/include/kickcat/Mailbox.h 86% 87% 85% :white_check_mark:
lib/include/kickcat/protocol.h 88% 88% 0% :white_check_mark:
lib/include/kickcat/CoE/EsiParser.h 0% 0% 0% :x:
lib/include/kickcat/CoE/OD.h 0% 0% 0% :x:
lib/master/include/kickcat/Bus.h 60% 100% 20% :x:
lib/master/include/kickcat/DebugHelpers.h 83% 100% 66% :white_check_mark:
lib/master/include/kickcat/Gateway.h 75% 100% 50% :white_check_mark:
lib/slave/include/kickcat/AbstractESC.h 0% 0% 0% :x:
lib/slave/include/kickcat/ESC/EmulatedESC.h 0% 0% 0% :x:
simulation/network_simulator.cc 0% 0% 0% :x:

Minimum allowed coverage is 75%

Generated by :monkey: cobertura-action against ee55932751a348df6726419c57f6ee1b87667550

Rdk-T commented 3 weeks ago

Before merged the CTT suite shall be run / tested on hardware