Skip to content
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

Add fitness points to ttbin decoding #135

Open
scubajorgen opened this issue Oct 3, 2017 · 8 comments
Open

Add fitness points to ttbin decoding #135

scubajorgen opened this issue Oct 3, 2017 · 8 comments

Comments

@scubajorgen
Copy link
Contributor

@derickr Please add support for TomTom Fitness points to the ttbin decoding.
The new TomTom Fitnesspoints seem to be present in the ttbin record with tag 0x4A (The 0x4B record is the new variable length record.):

Format of the record:

  1. Tag 0x4A (1 byte)
  2. Timestamp (epoch seconds UTC) (4 bytes)
  3. Fitness point counter (2 bytes)
  4. Fitness point counter (2 bytes)

It is unclear to me what the difference is between the two fitness point values. They seem to have the same value. Both are counters are reset to 0 at 00:00 hours local time and increase with each workout.

The fitness points are also tracked in the 0x00B1nnnn files (activity tracking). I'll add a new protobuf definition later reflecting this extension.

@derickr
Copy link
Contributor

derickr commented Oct 3, 2017

The question is however, what to do with the fitness point information? The TCX and GPX formats don't really do anything with this of course.

@derickr
Copy link
Contributor

derickr commented Oct 3, 2017

Preliminary implementation in #136 .

@ryanbinns
Copy link
Owner

Does the official TomTom software do anything with this data? If so, what?

@scubajorgen
Copy link
Contributor Author

scubajorgen commented Oct 9, 2017

For sure! Fitness points registered in the logged activities (ttbin files, 0x0091nnnn files) and in the tracked actitivity (0x00b100nn files). The former are displayed with each activity, the latter is displayed in a fancy new screen showing current weeks' fitnesspoints with stars (and new graphs with fitness age (yeah! this 50 year old nerd reached 20 years fitness age (fwiw))).
tt01
tt02

@ryanbinns
Copy link
Owner

I see. It looks pretty, but I'm not sure how we should implement this. ttwatch doesn't have a GUI, and none of the current output formats support adding this data to it. TCX could potentially be extended, since it's simply XML, but it might break other software that reads the files. A separate file could be created, but that could lead to issues with keeping the data together. Any ideas?

@derickr
Copy link
Contributor

derickr commented Oct 12, 2017

As I said in #136 — I'll likely want to add a JSON output that has all the records in it.

@jkopacze
Copy link

jkopacze commented Oct 16, 2017

Hi guys, just to to let you know. The last change in TCX file format broke an integration with Endomondo, basically importing training file now throws an error about not being able to process the file. Going to try to downgrade to version from before #136 merge today.

@derickr
Copy link
Contributor

derickr commented Oct 16, 2017

@jkopacze That seems curious, as my changes did not modify the TCX output format. Can you share the ttbin and tcx files, and if you have, a better error message?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants