-
Notifications
You must be signed in to change notification settings - Fork 184
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Timestamps don't have timezone information #3
Comments
Look at the bottom of # Custom conversion functions for FieldTypes (specific to FIT SDK 1.2)
# TODO:
# "0x10000000: if date_time is < 0x10000000 then it is system time (seconds
# from device power on)" -- not ofr local_date_time
_convert_date_time = lambda x: datetime.datetime.fromtimestamp(631065600 + x)
# TODO: Handle local tz conversion
_convert_local_date_time = lambda x: datetime.datetime.fromtimestamp(631065600 + x) Hmmmm... I'm not sure TZ is information held in the .FIT file. For now, it just uses your local system time for both date_time field types. Let me think about this a bit. D |
My .FIT from the Garmin Edge 500 files just hold time relative to their current as far as I can tell. Do you have any example files to share? I could support this. |
I'm using python-fitparse to for FIT-to-TCX and would like to get some timezone information in there. I have some fit files from my Garmin Forerunner 610, I'll see if I can send some over to you. |
Hey Tigge, I'm currently in the process of updating the library. Any sample files with and a description of expected behaviour sent to [email protected] would be very much appreciated! Thanks, David |
*any sample files from your Forerunner 610 and a description of expected behaviour... |
This is a hard problem because (as far as I can tell from reading the docs) there is no "device timezone" information supplied anywhere in the FIT file. Decoding used to just assume the device was in the computer's local timezone. As of b43005d this was changed to assuming UTC. Both are invalid assumptions but at least assuming UTC will give you consistency. One method to get a mostly correct result would be to look for a record that has both a Really, the only really 100% correct way would be to require a device timezone to be passed into the constructor. If one isn't supplied, then there's no way to ensure that the library is outputting correct times. At that point maybe it should log a warning and not emit any @dtcooper Thoughts? |
Late jump in but in case it helps: according to the Therefore for correctness and consistency, I guess fitparse should always make its |
Oh, it seems fitparse's data processor is context-less so the timezone offset of a |
In the 'activity' message |
I came across a .fit file recorded by a Garmin Edge 510 which didn't have the 'local_timestamp'. Another way to work around this is to use the latitude and longitude data to get the timezone info (there are a couple of packages available for this). |
I recently submitted a PR to @Tigge's FIT-to-TCX, Tigge/FIT-to-TCX#13, which makes it test that all of the files from this project's test/files can at least be translated to TCX without failures. antfs-dump.63.fit turns out to be a really interesting one, because it's from a non-GPS HRM device. The only "real" timestamp (integer value
These two values ( I implemented this offset-adjusting in Tigge/FIT-to-TCX@4a8e0e8, but perhaps it would be appropriate to include it in Something like a |
I don't know if FIT-files keep timezone information or if one is implied somehow. If that is the case it would be nice to have timezone information in the datetime objects created by fitparse.
The text was updated successfully, but these errors were encountered: