roberto1903 / companion9x

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

c9x consistency with o9x stock #126

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Read EEPROM from TX 
2. compare TX menus with c9x model/general edit

What is the expected output? What do you see instead?
General edit in c9x:
a. missing Volume=7 if voice option is enabled.
b. no same grouping like in o9x

Model edit in c9x:
c. Vario limits
d. Custom function "Play Value" parameter is shifted.
     eg. in o9x "Alt" - in c9x "A1"
e. Setup Telemetry Bar with Altitude:
     in c9x i set Alt from 2m to 1522m
     in o9x it is displayed from 12m to 1532m
     it looks shifted by 10m, but which one?

Please use labels and text to provide additional information.
c9x r1659 - o9x stock r1906

Original issue reported on code.google.com by gbir...@gmail.com on 1 Feb 2013 at 9:51

GoogleCodeExporter commented 8 years ago
point c. is OK
e. please let me know if this is ok (no impact to something else):
--- modeledit.cpp   (revision 1662)
+++ modeledit.cpp   (working copy)
@@ -180,7 +180,7 @@
     case TELEM_GPSALT:
     case TELEM_MAX_ALT:
     case TELEM_MIN_ALT:
-      return (8*Value)-510;
+      return (8*Value)-500;
       break;
     case TELEM_RPM:
     case TELEM_MAX_RPM:

Original comment by gbir...@gmail.com on 3 Feb 2013 at 4:05

GoogleCodeExporter commented 8 years ago
case a. displaing Speaker volume in General edit.

is it enough for now?

Original comment by gbir...@gmail.com on 14 Feb 2013 at 7:10

Attachments:

GoogleCodeExporter commented 8 years ago
But then, all boards do have the volume :)

I think that the HasVolume capacity may be removed, unless we take care of the 
variants? Hmmm. yes, it would make sense!

Original comment by bson...@gmail.com on 14 Feb 2013 at 9:02

GoogleCodeExporter commented 8 years ago
I suppose this issue is closed, isn't it ?
Looking at the reported issues no one seems to exist any longer now.

Original comment by romolo.m...@gmail.com on 14 Apr 2013 at 8:49

GoogleCodeExporter commented 8 years ago
remaining a. was solved. thanks to Romolo.

Original comment by gbir...@gmail.com on 14 Apr 2013 at 7:44