theomarzaki / TrafficOrchestrator

Traffic Orchestrator for Central unit Processing of autonomous vehicle merging through the use of Reinforcment Learning
MIT License
2 stars 0 forks source link

Heart Beat Incorporation #24

Closed theomarzaki closed 5 years ago

theomarzaki commented 5 years ago

Hello,

It seems that the error obtained in the integration tests are not a result of mishandling of notify delete message like I had thought. Since trying with req_id 42, the TO can handle the messages properly.

I think that the messages from the V2X are replaced with "\n" messages (heartbeat) causing the TO to behave unexpectedly,

Would it be worth changing the V2X heartbeat to "\t" to avoid this issue - then if the characters received is 1 and \t is the message, then it can be disregarded.

theomarzaki commented 5 years ago

Bug could not recreated in Integration meeting...