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.

Sign up for GitHub

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

Jump to bottom

Garmin Activity Extension #31

Closed
pcolby opened this issue Aug 12, 2014 · 3 comments
Closed

Garmin Activity Extension #31

pcolby opened this issue Aug 12, 2014 · 3 comments
Assignees
@pcolby
Labels
feature ready for release TCX
Milestone
0.4.0

Comments

@pcolby
Copy link
Owner

pcolby commented Aug 12, 2014

See https://github.com/pcolby/bipolar/wiki/TCX
pcolby added a commit that referenced this issue Sep 10, 2014
@pcolby
Copy initial test files for GPX and TCX extensions
341c0e9 
These files will be used to test the following extensions:
* Cluetrust GPX Extension (#29)
* Garmin (GPX) TrackPoint Extension (#30)
* Garmin (TCX) Activity Extension (#31)
@pcolby pcolby self-assigned this Sep 10, 2014
pcolby added a commit that referenced this issue Sep 10, 2014
@pcolby
Add placeholder test code for GPX / TCX extensions
bd69c12 
These test functions will cover the following extensions:
* Cluetrust GPX Extension (#29)
* Garmin (GPX) TrackPoint Extension (#30)
* Garmin (TCX) Activity Extension (#31)
pcolby added a commit that referenced this issue Sep 10, 2014
@pcolby
Implement Gamin Activity Extension TPX element (#31)
16d543f 
This is one of two extension elements supported by the Gamin Activity
Extension (http://www8.garmin.com/xmlschemas/ActivityExtensionv2.xsd),
the other being the LX element, which will be added shortly.
pcolby added a commit that referenced this issue Sep 10, 2014
@pcolby
Implement Gamin Activity Extension LX element (#31)
d708d25 
This is the last element we needed to implement to support the Garmin
Activity Extension.

Definitely should do some refactoring of the TrainingSession class
sometime - its grown into a small beast.
@pcolby pcolby added this to the 0.3.2 milestone Oct 8, 2014
pcolby added a commit that referenced this issue Jan 5, 2015
@pcolby
Include power data in TCX output (#31, #43, #47)
32603fb 
TCX has no support for power data per se, however, both the Garmin
Activity Extension (#31) and the Garmin Course Extension (#47) support
power data.  This change includes power data for both extensions.

Note, the Garmin Course Extension is currently not unit tested (will
add unit tests shortly).  Also, there are a few code paths added in
this change that cannot be tested until we get a data set that includes
*both* power data and lap data (#46).
@pcolby
Copy link
Owner Author

pcolby commented Jan 9, 2015

Doh! Just realised I've implemented v1, but need v2 for power support.. 😧

@pcolby
Copy link
Owner Author

pcolby commented Jan 9, 2015

Okay, it did implement v2... I was just confusing with the LX and TPX elements in my head. All good now (must have been a long day yesterday).

pcolby added a commit that referenced this issue Jan 9, 2015
@pcolby
Enable extensions in UI (#29,#30,#31,#32,#43,#47)
9e5de66
@pcolby
Copy link
Owner Author

pcolby commented Jan 10, 2015

Included in Bipolar 0.4.0.

@pcolby pcolby closed this as completed Jan 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pcolby pcolby

Labels
feature ready for release TCX
Projects
None yet
Milestone
0.4.0
Development

No branches or pull requests
1 participant
@pcolby