kawal547 / open9x

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

Larger current range. #189

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Would love to be able to specify larger ranges than 200 amps for the current 
sensing.  I am using FrSky for telemetry on a 6s boat.  A range of 400 would 
fall in line with the ACS759 sensor offered by Allego, combined with an offset 
of -200.

Original issue reported on code.google.com by Arbe...@gmail.com on 2 Feb 2013 at 9:30

GoogleCodeExporter commented 8 years ago
I am annoyed with this issue, at least on stock board. Having a range of 
400.00V would raise overflow problems (starting around 326V). 

I am ok though to do it on ARM boards, as working with 32bits integers is not a 
problem.

Original comment by bson...@gmail.com on 3 Feb 2013 at 6:48

GoogleCodeExporter commented 8 years ago

Original comment by bson...@gmail.com on 3 Feb 2013 at 6:50

GoogleCodeExporter commented 8 years ago
Done. Only on ARM as we need 32bits in all calculations.

Original comment by bson...@gmail.com on 3 Feb 2013 at 9:33

GoogleCodeExporter commented 8 years ago
Thanks for the update and enhancement.  I didn't realize it would be a PITA.  
FYI - I can also just run an ACS758 unidirectional where the offset would be 
much smaller and still read 200 amps.  In case this comes up again, this may be 
a better recommendation than the ACS759 bidirectional.

Original comment by Arbe...@gmail.com on 4 Feb 2013 at 6:08

GoogleCodeExporter commented 8 years ago
I am also wondering if PREC2 is needed internally. If no, there would be no 
problems at all!

Original comment by bson...@gmail.com on 4 Feb 2013 at 6:20