search

mavlink / mavlink-camera-manager

MAVLink Camera Manager Service
MIT License
102 stars 30 forks source link

Better tasking less locks #318

Closed joaoantoniocardoso closed 7 months ago

joaoantoniocardoso commented 11 months ago

merge after #303

joaoantoniocardoso commented 11 months ago

The threads do not finish..

image

A shutdown mechanism like #314 is still necessary

Even if you add the shutdown mechanism on top of this patch, your test will fail. If you have a while loop inside the drop, you need to allow more workers.

patrickelectric commented 11 months ago

The threads do not finish.. image A shutdown mechanism like #314 is still necessary

Even if you add the shutdown mechanism on top of this patch, your test will fail. If you have a while loop inside the drop, you need to allow more workers.

Indeed the method does not work with this approach, I'll check if there other ways for us to investigate this.

joaoantoniocardoso commented 7 months ago

Superposed by #377