-
Notifications
You must be signed in to change notification settings - Fork 16
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
TCX speed unit ? #74
Comments
Hi @pulmark,
Interesting. I didn't actually know what units TCX uses for speed. Its not made clear in the schema, and I never thought to check (a bit silly of me). It should be very easy to convert... I'll take a closer look in the next couple of days.
You could, if you prefer, just use my v800_downloader clone instead of the official. The original author appears to have abandoned the project, and my clone is simply the latest version with all Bipolar updates applied (just as you have done). There's an open pull request if you want to see the differences. Thanks! 😃 |
Ok, thanks for the info ! Yep, fix speed units is easy, I just did "divide by 3.6" modifications into the TrainingSession::toTCX() method. I have working build environment now. I will investigate your v800_downloader clone. I don't know how you have done it but maybe building a separate library of bipolar parts that are common to V800_downloader and bipolar would be the next step ? |
The Polar speed data in in kilometres-per-hour, but TCX uses metres-per- second (apparently; the relevant schemas are not explicit about it). For completeness, the HRM output is already using the correct units (that is, kilometres-per-hour multiplied by 10 or 128 depending on the situation), and the GPX output does not include speed data at all. So neither of those formats needing correcting.
Yep, would be pretty easy to do - there's only a very small amount of UI logic that's leaked into the protobuf and Polar-specific classes that v800d uses. I've put together a basic project outline already, but its not a high priority right now. Cheers. |
first of all - this is a very nice Project !! And I've seen you are planing to integrate FIT file Export ? |
Thanks :)
Oh. Can you provide more details of what you think is missed? I had a good look, and thought I covered everything, but the specs are pretty ambiguous in their language. Which "extended" format are you referring to? (Garmin Activity Extension or Garmin Course Extension?) and which child element or attribute of the trackpoint? Thanks! 😄
There's no timeline currently. Its one of those things that I'd like to do one day, but there's hasn't been much demand nor interest for it. Out of curiosity, what application would you use the FIT output with? If there's something I can get my hands on to test generated FIT files, I might be more motivated to implement it 😃 Thanks again. |
hi, In "QDomDocument TrainingSession::toTCX(const QString &buildTime) const" "http://www.garmin.com/xmlschemas/ActivityExtension/v2"
|
Out of curiosity, what application would you use the FIT output with? If there's something I can get my hands on to test generated FIT files, I might be more motivated to implement it I want to use it with RubiTracks 4 Pro. There is a Trial Option if you want to test it (mac only) For me this Format (as far as I read the docu) gives the most among of data to be tranfered, therfor im interested in. Beside this Im not realy shure if this is caused by the lack of data in the file. The stride length could be easily calculated from distance and cardence. A Support request to rubitracks, if this stride-length value is supported at all, is not answered yet |
Thanks @MHatGH! Now I can exactly what was missing. I really appreciate the effort you've made reporting the details for me 👍 I'll go ahead an fix that up now 😄 PS: Good use of |
I integrated bipolar master branch into V800_downloader and build it. Now it seems to support cycling power data in HRM files with PPT5.
I noticed that TCX conversion uses km/h for speed data as it should be m/s. I don't know if this is specific to my environment, I use M450 for cycling, I have Powertap G3 hub and download training data directly from head unit via USB connection by using V800_downloader in Linux OS. In addition to power, G3 hub measures also speed and cadence.
For building in Linux OS I cloned V800_downloader and then added bipolar into it by cloning it under V800_downloader (bipolar directory). I then had to edit some project files to build app with QT Creator. For some reason I couldn't get delivered Linux libusb files to work with QT Creator so just modified project files to use libusb development libraries and header files that I installed into my Linux OS.
I run then tests and noticed that HRM files now have power data and it is displayed in PPT5. Then did TCX testing and noticed speed unit error when uploading TCX file into MapMyRide web service. Fixed speed values to be m/s instead of km/h in
QDomDocument TrainingSession::toTCX(const QString &buildTime) const
method. All fine after modification.
Also to add power/speed data into TCX files I modified V800_downloader code to enable Garmin ActivityExtension (XML data - TPX element). This option cannot be set in V800_downloader UI.
The text was updated successfully, but these errors were encountered: