-
Notifications
You must be signed in to change notification settings - Fork 446
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
Addition of CoreTemp sensor support #4207
base: master
Are you sure you want to change the base?
Conversation
src/FileIO/JsonRideFile.y
Outdated
@@ -105,6 +105,8 @@ static QString protect(const QString string) | |||
#define scanner jc->scanner | |||
|
|||
%} | |||
// generated by the scanner | |||
%define api.header.include {"JsonRideFile_yacc.h"} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change breaks ci builds since it requires a newer bison version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apologies. No idea what made that change, removed.
Spotted a bug while fixing the CI issue, tidied the change. 😄 |
If you want to edit the data remember to update RideEditor::insColumn() and RideEditor::whatColumns() |
This patch adds ANT+ support for CoreTemp body sensors.
As discussed on #4121 this is now updated to use the XDATA fields rather than changing the saved json.
If there is core temperature data then it uses that for "tcore" data instead of using the estimated calculation.