biobankAccelerometerAnalysis icon indicating copy to clipboard operation
biobankAccelerometerAnalysis copied to clipboard

GENEActive not parsing the tz/dst correctly

Open angerhang opened this issue 2 years ago • 3 comments

At the moment, GENEActive devices are reading local time as if they are true unix time, which is incorrect. This will create issues for daylight savings and different time zones. Something like AX3 needs to be done for GENEActive devices.

angerhang avatar Jun 28 '22 09:06 angerhang

@chanshing do we have an estimated timeline on this on this or I will do the change myself? I need to get this done soon to start training my model?

angerhang avatar Sep 26 '22 16:09 angerhang

@chanshing do we have an estimated timeline on this on this or I will do the change myself? I need to get this done soon to start training my model?

Sorry Hang, just reading this. I assumed you would fix it as you're probably more familiar with the GENEActiv device than me.

chanshing avatar Sep 28 '22 01:09 chanshing

I have to reopen this because bba currently doesn't deal with daylight saving and time zone properly. I am having a hot fix in my code to manually move the data back by one hour. But need to raise this issue for future awareness.

angerhang avatar Apr 04 '23 19:04 angerhang