mgorskified / rush-osd-development

Automatically exported from code.google.com/p/rush-osd-development
0 stars 0 forks source link

Amperag via ADC is missing critical parameter #12

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. know formula for amperage calculation
2. look at existing code for amperage calculation
3. see there is something missing

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

expect to see amperage value, but instead get raw analogRead integer formatted 
as amperage

What version of the product are you using? On what operating system?
2.2

Please provide any additional information below.

basically you forgot them[/b], prove me wrong but i just went over your new 
code and [b]there is nowhere to specify your amperage sensors[i] "sensitivity" 
[/i]as you call it, ie 40mv/A[/b]

this is your [b]only[/b] code for calculating amperage from the arduino's 
analog pin: it is not enough

[code]    if (!Settings[S_MWAMPERAGE]) {
    amperage = analogRead(amperagePin)- Settings[S_AMPOFFSET] /2;
    if (amperage >=999) amperage=999;
    }[/code]

Original issue reported on code.google.com by hthr...@gmail.com on 31 Dec 2013 at 1:21

GoogleCodeExporter commented 9 years ago
Thanks for your report once more.
This is yet waiting for review, the formula will change ASAP.

Original comment by katawentos on 31 Dec 2013 at 1:37

GoogleCodeExporter commented 9 years ago

Original comment by katawentos on 4 Jan 2014 at 3:08