neurobooth / neurobooth-os

Python package for digital phenotyping data synchronization and acquisition
http://neurobooth.github.io
BSD 3-Clause "New" or "Revised" License
6 stars 6 forks source link

Cleaner shutdown for server_acq #410

Closed boubre closed 4 months ago

boubre commented 4 months ago

This is a quick patch for something that keeps bothering me during testing. Namely, terminating servers mid-recording crashes ACQ because it doesn't first try to stop recording. This tends to happen when a bug from my untested code crashes STM.

Side note: servers should really be an object to better handle all the associated state information (device manager, logger, etc...). But will leave this refactor to the server rework.