search

robotology / robometry

Telemetry suite for logging data from your robot 🤖
https://robotology.github.io/robometry
Other
14 stars 9 forks source link

BufferManager: introduce the usage of the condition variable #104

Closed Nicogene closed 3 years ago

Nicogene commented 3 years ago

This is used for handling the save thread and this should avoid awaiting the thread save_period seconds to wake up in the closure phase.

It fixes #94

cc @S-Dafarra @lornat75

Nicogene commented 3 years ago

Seems that with this change the fail on macOs happens more frequently, but I expect that it will be fixed after #9.

Nicogene commented 3 years ago

https://github.com/robotology-playground/yarp-telemetry/pull/104/commits/d457e2dd7cde469bbee8c30acbc087f33e8682e3 should test this improvement. With the previous implementation, the execution would remain stuck at save_thread.join() for one hour before closing.