tuyndv / mytracks

Automatically exported from code.google.com/p/mytracks
0 stars 0 forks source link

BTLE support with selection in menu #1472

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Mercurial clone with changes to be reviewed:
https://fred.nadeau@code.google.com/r/frednadeau-mytracks2/ 

Revisions to be pulled from that clone (or "all"):
8f44979f1fec

Purpose of code changes on the clone:
Add Heart Rate Monitor via Bluetooth LE device
I was unhappy with the following clones:
https://code.google.com/r/ivakinfn-mytracks/
https://code.google.com/r/dgupta-btle/

My work is based on theirs. Biggest change is in the setting menu(I couldn't 
find a way to link the BTLE device in both cases, if you know of one, let me 
know.)

Code should also support CSC(Cycling Speed & Cadence), disabled for now(hard 
coded), see issue 653 .

When reviewing my code changes, please focus on:
New bluetoothLe sensor manager and setting activity.

Did you run the unit tests?
In Eclipse, select the 'MyTracksTest' project, right click, 'Run As' ->
'Android JUnit Test'.
Got a few failure(20). From what I understand, they are language issue, ex: Got 
"Distance totale : 1,10 km (0,7 mi)" Expect: "Total distance: 1.10 km (0.7 mi)"
I did not add any BTLE unit test as of today, will be added soon.

Did you allow others to review code on your clone?
From your clone, 'Administer' -> 'Source' -> 'Code reviews'
-> check 'Enable code reviews'
-> check 'Allow non-members to review code'
I sure did

Issue possibly affected by this change:
 - issue 1468
 - issue 1426
 - issue 1356
 - issue 1341

Original issue reported on code.google.com by fred.nad...@gmail.com on 11 Mar 2014 at 5:11

GoogleCodeExporter commented 9 years ago
Update:

Revisions to be pulled from that clone (or "all"):
a3f1837b02eb

An issue was found when leaving/returning to application.

Original comment by fred.nad...@gmail.com on 12 Mar 2014 at 1:37

GoogleCodeExporter commented 9 years ago
Tested functional with Mio LINK

Original comment by wyatt.n...@gmail.com on 26 Apr 2014 at 1:17

GoogleCodeExporter commented 9 years ago
Issue 1538 has been merged into this issue.

Original comment by jshih@google.com on 13 Oct 2014 at 7:49

GoogleCodeExporter commented 9 years ago
Issue 1554 has been merged into this issue.

Original comment by jshih@google.com on 13 Oct 2014 at 7:49

GoogleCodeExporter commented 9 years ago
We plan to utilize the Android Fit Platform to access BTLE sensor data.

Original comment by jshih@google.com on 13 Oct 2014 at 10:21

GoogleCodeExporter commented 9 years ago
Sounds great, many people are looking forward to it.

Original comment by wyatt.n...@gmail.com on 14 Oct 2014 at 2:23