robcog-iai / UROSBridge

Unreal ROS Bridge topic / service communication via websocket
http://robcog.org/
BSD 3-Clause "New" or "Revised" License
52 stars 37 forks source link

Thread not properly killed on end play #16

Closed andreihaidu closed 6 years ago

andreihaidu commented 6 years ago
LogStats: Warning: MetaData mismatch. Did you assign a stat to two groups? New //STATGROUP_Threads//ROSBridgeHandlerRunnable///Thread_3244_0///####STATCAT_Advanced#### old //STATGROUP_Threads//ROSBridgeHandlerRunnable///Thread_1ed4_0///####STATCAT_Advanced####

It should not affect usability.

See https://answers.unrealengine.com/questions/166882/logstatswarning-metadata-mismatch.html

Steps to reproduce:

Start world using the ROS handler (FROSBridgeHandler) in an actor. Stop world. Start again. Warning appears. It seems that the previous thread is still living in the editor, and when the new one is created the same name is used, hence the warning.

aaronsnoswell commented 6 years ago

I've seen this too. Does this only occur when in editor?

andreihaidu commented 6 years ago

workaround by appending an incrementing index to thread name, it gets rid of the warning, however the thread might still be idling in the background. (https://github.com/robcog-iai/UROSBridge/blob/master/Source/UROSBridge/Private/ROSBridgeHandler.cpp#L299)