search

zexinliao / opentx

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

Telemetry - FrSky - LiPo Sensor - Cell vs Cells Reversed #167

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
Which board (stock / gruvin9x / sky9n/ Taranis) are you using?
Taranis

What is your openTx FW version?
openTX-r2768
2013-09-21

What is your openTx EEPROM version?
eepr: 215

What steps will reproduce the problem?
1. Attach LiPo sensor
2. Check 'Cell' vs "Cells" values
3.

What is the expected output? What do you see instead?

Cell reports the value of one of the internal cells.  ( ie  3.89v )
Cells report the total value of all cells ( ie  7.7v)
The screen tries to show multiple values for "Cells" but the single value 
(over-voltage) exceeds the space for the first of "n" cells.

The telemetry info for Cell  vs  Cells are reversed.

  In "Cell" I should see the total voltage of the pack (ie 7.9v)
  In "Cells" I should see muliple values, one for each single cell ( ie  3.98v and 3.97v)

Please provide any additional information below.

Original issue reported on code.google.com by Charles....@gmail.com on 18 Oct 2013 at 5:56

GoogleCodeExporter commented 8 years ago 
Sorry but Cell actually is the lowest cell and has the logic of showing the 
more critical one...
Cells instead reports the sum of the cells in case you have no other voltmeter..

In my opinion should stay as it is...

Original comment by romolo.m...@gmail.com on 18 Oct 2013 at 8:48

GoogleCodeExporter commented 8 years ago 
  Wrong.  The prior version of FrSky worked as I described (DFT/hub/lipo sensor/FSH-01)

  Proof is that there are MULTIPLE spaces in the "Cells" location.  If you have 2 cells there will be space for 2 values.  4 spaces for 3 or 4 cells.  6 spaces for 5 - 6 cells, etc.  Thus "CELLS".

   Only one space for "Cell".

   Ask FrSky and/or check with FrSky and their documentation/implementation. 

  "Cell" is the total voltage of the pack.
  "Cells" shows each and every single value in the pack.

  Would you like me to make a video showing it on the FSH-01?

Original comment by Charles....@gmail.com on 19 Oct 2013 at 2:29

GoogleCodeExporter commented 8 years ago 
Opentx has worked this way since ever, as OpenTX runs in radios before than 
Taranis come out, and indeed works as intended in in manual.
http://code.google.com/p/opentx/wiki/OpenTx_FrSky_EN#Telemetry

FrSky implementation on DHT-U and FLD display had no way to set alarms on lower 
cell,

So sorry, but this is a wont fix.

Original comment by romolo.m...@gmail.com on 19 Oct 2013 at 11:04

GoogleCodeExporter commented 8 years ago 
There is nothing to fix.

- Cell is the lowest cell.
- Cels is the sum of all cells
- All cells are individually displayed on the first telemtry screen.

Original comment by bernet.a...@gmail.com on 19 Oct 2013 at 12:27

Attachments: