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

Fit file: decode cyclist position (seated/standing) #4327

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

vlcvboyer
Copy link
Contributor

Decode rider position from FIT file and record into GoldenCheetah ride as "POSITION" XDATA.
IDs for positions: 0="seated", 2="standing", 1 to 3="transition" are from FIT SDK. IDs 4 to 9 are kept as spare in case of usage in future FIT developments. In addition, 10 to 11 are used for "aero" (determined by fixRiderPosition when cadence is zero for a 'long' period) and "off" (when speed is zero).
This will allow to include rider position during analysis by adjusting CdA accordingly.
Aerolab may be interessted by this also.
fixRiderPosition will generate the fake "aero" and "off" position and adjust effective position as per FIT SDK (rename "transition" to effective position when confirmed by sensor).

@vlcvboyer
Copy link
Contributor Author

This is the first step to go forward on #3840 (and avoid git conflicts during merge, etc.)
All Cycling Dynamics are already there (in FIT file import and GC rides) but rider position was missing.
I'm working on #3840 assuming that this PR is OK

@vlcvboyer vlcvboyer force-pushed the FitFileDecoderCyclistPositionPR branch from 60b749b to a4bc874 Compare December 10, 2023 14:56
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

Successfully merging this pull request may close these issues.

None yet

1 participant