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

Missing info after Garmin Connect update #16

Closed
ldegiorgi opened this issue Aug 1, 2014 · 4 comments
Closed

Missing info after Garmin Connect update #16

ldegiorgi opened this issue Aug 1, 2014 · 4 comments
Assignees
Labels
Milestone

Comments

@ldegiorgi
Copy link

Hi,

just to start I want to thank you for your exceptional job!
To upload my data to Garmin Connect which is my main repository (other than SportTracks), I compared the TCX resulting from the export from FlowExport and your tool.
In particular, the data coming out from your TCX miss the following information: Max Speed and Average FC. This data is, instead, in the FlowExport TCX (there is missing the Calories data, but this is another story).

Is it right? this data is coming from the watch or calculated? I have the two TCX to look at.

Best regards

Luca

@pcolby
Copy link
Owner

pcolby commented Aug 1, 2014

Hi Luca,

Yeah, there were a couple of optional TCX fields that I couldn't figure out how to get from the protobuf data, but I in doing some recent HRM-export work, I think I've figured most those out now, and will add them very soon.

Specifically, I should now be able to add:

  • MaximumSpeed
  • AverageHeartRateBpm
  • MaximumHeartRateBpm

What did you mean by "Average FC"? (I'm sure its something obvious... I just need some sleep)

Calories should be there - it is in my TCX files. Do you not see any <Calories> entries in the TCX file(s)?

pc.

@pcolby pcolby added the bug label Aug 1, 2014
@pcolby pcolby added this to the 0.1.2 milestone Aug 1, 2014
@pcolby pcolby self-assigned this Aug 1, 2014
@pcolby pcolby added enhancement and removed bug labels Aug 1, 2014
@ldegiorgi
Copy link
Author

You're great!

Average FC means, for me, what you call AverageHeartRateBpm.

Thank you

Il giorno 01/ago/2014, alle ore 13:49, Paul Colby notifications@github.com ha scritto:

Hi Luca,

Yeah, there were a couple of optional TCX fields that I couldn't figure out how to get from the protobuf data, but I in doing some recent HRM-export work, I think I've figured most those out now, and will add them very soon.

Specifically, I should now be able to add:

MaximumSpeed
AverageHeartRateBpm
MaximumHeartRateBpm
What did you mean by "Average FC"? (I'm sure its something obvious... I just need some sleep)

Calories should be there - it is in my TCX files. Do you not see any entries in the TCX file(s)?

pc.


Reply to this email directly or view it on GitHub.

pcolby added a commit that referenced this issue Aug 1, 2014
pcolby added a commit that referenced this issue Aug 1, 2014
pcolby added a commit that referenced this issue Aug 1, 2014
This also reduces the chance of bad data resulting in broken / invalid
TCX files.

Relates to issue #16.
pcolby added a commit that referenced this issue Aug 2, 2014
@pcolby
Copy link
Owner

pcolby commented Aug 3, 2014

Included in 0.2, which was just released.

@pcolby pcolby closed this as completed Aug 3, 2014
@ldegiorgi
Copy link
Author

You're great!

Il giorno 03/ago/2014, alle ore 09:31, Paul Colby notifications@github.com ha scritto:

Included in 0.2, which was just released.


Reply to this email directly or view it on GitHub.

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

No branches or pull requests

2 participants