Hello Mr jonescg
I have updated the CellTester to v3.0 to implement current sensing.
It will now take input from a Jaycar XC-4610 and log this along with the voltage.
I have added time info to the log to show how long the charge or discharge has been running.
It will now calculate the Ah capacity for both charge & discharge cycles.
On completion of the test it will print all Ah measurements.
Also the frequency of logging can now be set.
I have removed the external scaling adjustments so you will have to set the MIN and MAX to be CELL voltages and then adjust the pot to match these.
During testing I found that the ground connection for the analogue inputs is critical in that there should be no (or very little) current flowing in the ground between the Uno and the sensor as a voltage drop of 4.8mV will be 1 count in the A2D result.
Can I suggest that you run the ground from the relay board back to a separate ground on the Uno. The voltage pot and the current sensor should be able to share a single ground.
The zero amps point of the sensor is read during startup so there is no current flowing into or out of the battery.
As for the stray '16' I thought it might be that the # meant something to the print function but it doesn't seem to - you could try removing it just to see.
What version of the Arduino IDE are you running, mine is 1.6.7.
Finally about the relays not turning off at the end of the test, I have added code to continuously turn them off once the tests have completed - see if it works correctly now!
Well I think I have covered everything but you should check the #defines below the line"// FOR NORMAL USE this code is compiled" as I have added some conditionals for my testing with the ATmega.
Hope this helps
neilg
CellTester_v3.zip