RI-SE / ATOS

ROS2 based platform for coordinating tests of automated vehicles and their surrounding systems.
Mozilla Public License 2.0
16 stars 4 forks source link

Fix obc state enum #635

Closed Robert108 closed 8 months ago

SebastianSvanland commented 9 months ago

Hmm, shouldn't we start adding these things to the ROS messages? That way we ensure that e.g. OBC states enum will match in both backend and frontend and for all other python modules.

SebastianSvanland commented 8 months ago

@Robert108 Do you think it's a good idea? We should fix this bug asap

Robert108 commented 8 months ago

@SebastianSvanland Yes that is a good idea! But it could be worth waiting with introducing enum definitions to when we rewrite OBC. I have a feeling we will do a overhaul of these messages then anyway.