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 rider position (seated/standing) #4328
Comments
Hi developpers, |
hey I have been fixing bugs and working on generic parsing of message types into XDATA. Will be comitting sometime in the next couple of weeks and will come back to look at your PR. My changes are quite large, so may need to redo your work- but will try and support that as I am the one causing the problem. Also, apologies for the delay in looking at this. |
Ok no problem
Thanks for this update
Please di not hesitate to send me reminder if I do not see any rebasing or
rewriting few weeks after your merge on master
Best regards
Le jeu. 9 nov. 2023, 16:13, Mark Liversedge ***@***.***> a
écrit :
… hey
I have been fixing bugs and working on generic parsing of message types
into XDATA. Will be comitting sometime in the next couple of weeks and will
come back to look at your PR.
My changes are quite large, so may need to redo your work- but will try
and support that as I am the one causing the problem.
Also, apologies for the delay in looking at this.
—
Reply to this email directly, view it on GitHub
<#4328 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADG36T7MPEFLEKRXOMTH67DYDTXILAVCNFSM6AAAAAAUR245H6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBUGAZDAMZZGE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Hey- the code is committed can you fix the conflicts so I can merge? |
When using pedal powermeters rider position is part of FIT records.
Getting it will allow to change CdA estimate during ride analysis.
Associated PR is #4327
The text was updated successfully, but these errors were encountered: