Getting wrong readings time to time

[josepmon]

Dear all,

Right now I've started playing with this sketch in order to connect and Arduino Nano with an OpenUPS and read some values from it (voltage, av. current, av. time to full, capacity, etc.).

I follow the default I2C configuration for this purpose, using Arduino A4 and A5 to connect to OpenUPS SMBUS. Then, by running PackProbe, initially everything works as expected. The scan function detects the OpenUPS address.

Then, when time comes for reading values from OpenUPS, I'm having some incoherence reading time to time. Of course, the OpenUPS is not ready to display all the information that PackProbe defines, but for the parameters that are allowed, time to time I'm getting a wrong value (always -15134).

Example: I'm reading a capacity value of 2600mAh (which is the expected value). Then, at some loop, I'm getting a capacity value of -15134.

I'm finding this randomly in all the parameters read. Any idea of which could be the reason?

Thanks

Josep

[josepmon]

Find below the output obtained through the serial monitor. In this case I'm just reading the 'Design capacity' value with a delay between readings of 5secs. As you can see I'm getting 2600 (right value) most of the times, but some times I get -15134 (wrong value).

Serial Initialized
I2C Inialized
Address: 0x0: -
Address: 0x1: -
Address: 0x2: -
Address: 0x3: -
Address: 0x4: -
Address: 0x5: -
Address: 0x6: -
Address: 0x7: -
Address: 0x8: -
Address: 0x9: -
Address: 0xA: -
Address: 0xB: -
Address: 0xC: -
Address: 0xD: -
Address: 0xE: -
Address: 0xF: -
Address: 0x10: -
Address: 0x11: -
Address: 0x12: -
Address: 0x13: -
Address: 0x14: -
Address: 0x15: -
Address: 0x16: -
Address: 0x17: -
Address: 0x18: -
Address: 0x19: -
Address: 0x1A: -
Address: 0x1B: -
Address: 0x1C: -
Address: 0x1D: -
Address: 0x1E: -
Address: 0x1F: -
Address: 0x20: -
Address: 0x21: -
Address: 0x22: -
Address: 0x23: -
Address: 0x24: -
Address: 0x25: -
Address: 0x26: -
Address: 0x27: -
Address: 0x28: -
Address: 0x29: -
Address: 0x2A: -
Address: 0x2B: -
Address: 0x2C: -
Address: 0x2D: -
Address: 0x2E: -
Address: 0x2F: -
Address: 0x30: -
Address: 0x31: -
Address: 0x32: -
Address: 0x33: -
Address: 0x34: -
Address: 0x35: -
Address: 0x36: -
Address: 0x37: -
Address: 0x38: OK
Address: 0x39: -
Address: 0x3A: -
Address: 0x3B: -
Address: 0x3C: -
Address: 0x3D: -
Address: 0x3E: -
Address: 0x3F: -
Address: 0x40: -
Address: 0x41: -
Address: 0x42: -
Address: 0x43: -
Address: 0x44: -
Address: 0x45: -
Address: 0x46: -
Address: 0x47: -
Address: 0x48: -
Address: 0x49: -
Address: 0x4A: -
Address: 0x4B: -
Address: 0x4C: -
Address: 0x4D: -
Address: 0x4E: -
Address: 0x4F: -
Address: 0x50: -
Address: 0x51: -
Address: 0x52: -
Address: 0x53: -
Address: 0x54: -
Address: 0x55: -
Address: 0x56: -
Address: 0x57: -
Address: 0x58: -
Address: 0x59: -
Address: 0x5A: -
Address: 0x5B: -
Address: 0x5C: -
Address: 0x5D: -
Address: 0x5E: -
Address: 0x5F: -
Address: 0x60: -
Address: 0x61: -
Address: 0x62: -
Address: 0x63: -
Address: 0x64: -
Address: 0x65: -
Address: 0x66: -
Address: 0x67: -
Address: 0x68: -
Address: 0x69: -
Address: 0x6A: -
Address: 0x6B: -
Address: 0x6C: -
Address: 0x6D: -
Address: 0x6E: -
Address: 0x6F: -
Address: 0x70: -
Address: 0x71: -
Address: 0x72: -
Address: 0x73: -
Address: 0x74: -
Address: 0x75: -
Address: 0x76: -
Address: 0x77: -
Address: 0x78: -
Address: 0x79: -
Address: 0x7A: -
Address: 0x7B: -
Address: 0x7C: -
Address: 0x7D: -
Address: 0x7E: -
Design Capacity: 2600
.
Design Capacity: 2600
.
Design Capacity: 2600
.
Design Capacity: 2600
.
Design Capacity: 2600
.
Design Capacity: 2600
.
Design Capacity: -15134
.
Design Capacity: 2600
.
Design Capacity: 2600
.
Design Capacity: 2600
.
Design Capacity: 2600
.
Design Capacity: 2600
.
Design Capacity: 2600
.
Design Capacity: -15134
.
Design Capacity: 2600
.
Design Capacity: 2600
.
Design Capacity: 2600
.
Design Capacity: 2600
.
Design Capacity: 2600
.
Design Capacity: -15134
.
Design Capacity: 2600
.
Design Capacity: 2600
.
Design Capacity: 2600

[wagnerbra]

Hi @josepmon .
I am having the same issue. Here are some readings. In fact, here, the oscillation bounces as different numbers.

###############430###############
Manufacturer Name: B-T Inc.
Device Name: B-T Inc.
Chemistry LION
Design Capacity: 9300
Design Voltage: 9300
Manufacture Date (Y-M-D): 2018-9-6
Serial Number: 19750
Specification Info: 49
Cycle Count: 49
Voltage: 0.05
Full Charge Capacity: 49
Remaining Capacity: 67
Relative Charge(%): 1
Absolute Charge(%): 1
Minutes remaining for full charge: -1
Battery Mode (BIN): 0b11111111111111111111111111111111
Battery Status (BIN): 0b11111111111111111111111111111111
Charging Current: 1000
Charging Voltage: 1000
Temp: 22.35
Current (mA): 0
.
###############431###############
Manufacturer Name: B-T Inc.
Device Name: B-T Inc.
Chemistry B-T Inc.
Design Capacity: 16904
Design Voltage: 16904
Manufacture Date (Y-M-D): 2013-0-8
Serial Number: 10963
Specification Info: 49
Cycle Count: 49
Voltage: 0.05
Full Charge Capacity: 49
Remaining Capacity: 67
Relative Charge(%): 1
Absolute Charge(%): 1
Minutes remaining for full charge: -1
Battery Mode (BIN): 0b1
Battery Status (BIN): 0b1
Charging Current: 1000
Charging Voltage: 1000
Temp: -173.15
Current (mA): 0
.
###############432###############
Manufacturer Name: B-T Inc.
Device Name: BT-70791CG
Chemistry BT-70791CG
Design Capacity: 16906
Design Voltage: 16906
Manufacture Date (Y-M-D): 2013-0-10
Serial Number: 10963
Specification Info: 10963
Cycle Count: 2
Voltage: 11.76
Full Charge Capacity: 10351
Remaining Capacity: 67
Relative Charge(%): 1
Absolute Charge(%): 1
Minutes remaining for full charge: 1
Battery Mode (BIN): 0b1
Battery Status (BIN): 0b1011010000
Charging Current: 1000
Charging Voltage: 1000
Temp: 22.35
Current (mA): 0
.

Did you solve your problem? How?

[josepmon]

Hi @wagnerbra ,

I haven't been working on that since long time ago, so the only thing I can give you right now is the approach I finally used in my OpenUPS.cpp library:

// This function reads and returns an integer parameter from OpenUPS.
// An issue was found while reading several parameters in a row. Time to time, the value obtained 
// was wrong (always the same number -15134). This has been solved by including
// the following:
//   - Reading a third byte (PEC byte?).
//   - Perform a brief delay of 1ms after the communications.

int OpenUPS::OpenUPSReadParameterInt(byte cmd)
{
	i2c_start(this->deviceAddress<<1 | I2C_WRITE);
	i2c_write(cmd);
  	i2c_rep_start(this->deviceAddress<<1 | I2C_READ);
  
	byte b1 = i2c_read(false);
	byte b2 = i2c_read(false);
	byte b3 = i2c_read(true); //MANDATORY FOR GOOD READING OF THE FOLLOWING PARAMETER
	
	i2c_stop(); 
	delay(1); //MANDATORY FOR GOOD READING OF THE FOLLOWING PARAMETER
	
	return (int)b1|((( int)b2)<<8);
}

In short I'm planning to move back to these function. If I discover anything new I will publish it here. Please, in case you find any other approach to solve this issue, do not hesitate to contact me.

Best regards,

Josep

[ArminJo]

Try to use the improved version at https://github.com/ArminJo/Smart-Battery-Module-Info_For_Arduino