search

bearing / dosenet-raspberrypi

Raspberry Pi specific software for dosimeters.
MIT License
7 stars 21 forks source link

Fix bug with timestamps sent to memory #72

Closed bplimley closed 7 years ago

bplimley commented 7 years ago

DataHandler.send_to_queue() takes cpm and cpm_err as inputs, and assigns a timestamp using time.time().

The problem is, if the data is loaded from the queue and then fails to send, it goes back into the queue with the wrong timestamp. Instead, the timestamp should be an input argument to send_to_queue() and send_to_memory().

tybtab commented 7 years ago

I think I solved this with the 'new' protocol issue.

tybtab commented 7 years ago

Also 'new' protocol is still not default. Should I make it 'default'?

bplimley commented 7 years ago

It looks like this is fixed in your pull request, Thanks. I'm still holding off on making new default until a couple more issues are resolved, such as #68. Sorry I forgot to assign you earlier.