-
Notifications
You must be signed in to change notification settings - Fork 272
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
Android sensors #490
Merged
jonasoreland
merged 11 commits into
jonasoreland:master
from
gerhardol:android-sensors-merge
Dec 18, 2016
Merged
Android sensors #490
jonasoreland
merged 11 commits into
jonasoreland:master
from
gerhardol:android-sensors-merge
Dec 18, 2016
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Fixes that Orieenteering is handled as Running rather than Other in .tcx, Google Fit The main benefit is though a cleanup in the handling of the raw dbValues like DB.ACTIVITY.SPORT_RUNNING The enabling of sporttypes had a big uncertainity here, many manual checks (the cleanup was partial, I had done a little separately that is now separated) As there is more or less one-to-one mapping dbValue to Sport, it is still not easy to add new sporttypes, but at least the db value is checked.
Remove Eclipse specific from .gitignore
Cadence changed from integer to real (OK in SQLite, field is not used)
Good to see the elevation stabilizing to get good accuracy Even more important when using barometer elevation, it is calibrated to GPS elevation
…sors Support for cadence/temperature/pressure-elevation affects many files Separate TrackerElevation that selects GPS or barometer depending on configuration Creator set in gpx/tcx export if pressure was available when recording
Strava exports temperature but do not display it Tune Garmin GPX extensions to upload cadence etc too Support Cluetrust GPX extensions (Cluetrust is not fully working to Strava)
very cool!
|
The PR is based on the second to last commit in master, actually on PR #486 (uses some of those changes). Edit: Yes, the patch is big. Most cannot be separated. |
# Conflicts: # .travis.yml # build.gradle
gerhardol
added a commit
to gerhardol/runnerup
that referenced
this pull request
Dec 25, 2016
gerhardol
added a commit
to gerhardol/runnerup
that referenced
this pull request
Dec 25, 2016
Merged
Closed
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Support internal sensors for step(cadence), temperature, pressure(elevation)
This also needed the infrastructure for those measures
The commits are split to explain some of the internal parts, but most changes are in one of the commits