Nut upsuser - Aug 2014 - Cyberpower Value1200E might not need 0.667 battery scaling

I just got a Cyberpower SOHO Value 1200 ELCD UPS and even with nut 
2.7.2, it appears to report battery voltage too low due to the battery 
scaling function (In drivers/cps-hid.c).  Even though the ups has usb 
id: 0764:0501, UPS.PowerSummary.Voltage reports 26.6 for the 24V 
batteries so when the 0.667 battery scale is applied that goes down to 
17.7V.

upsc output:
battery.charge: 100
battery.charge.low: 10
battery.charge.warning: 20
battery.mfr.date: CPS
battery.runtime: 3012
battery.runtime.low: 300
battery.type: PbAcid
battery.voltage: 17.7
battery.voltage.nominal: 24
device.mfr: CPS
device.model: Value1200E
device.type: ups
driver.name: usbhid-ups
driver.parameter.offdelay: 20
driver.parameter.ondelay: 1
driver.parameter.pollfreq: 30
driver.parameter.pollinterval: 2
driver.parameter.port: auto
driver.version: 2.7.2
driver.version.data: CyberPower HID 0.3
driver.version.internal: 0.38
input.transfer.high: 0
input.transfer.low: 0
input.voltage: 235.0
input.voltage.nominal: 230
output.voltage: 239.0
ups.beeper.status: enabled
ups.delay.shutdown: 20
ups.delay.start: 1
ups.load: 11
ups.mfr: CPS
ups.model: Value1200E
ups.productid: 0501
ups.realpower.nominal: 720
ups.status: OL
ups.test.result: No test initiated
ups.timer.shutdown: -60
ups.timer.start: -60
ups.vendorid: 0764

Full usb descriptor from lsusb:
Bus 002 Device 016: ID 0764:0501 Cyber Power System, Inc. CP1500 AVR UPS
Device Descriptor:
   bLength                18
   bDescriptorType         1
   bcdUSB               1.10
   bDeviceClass            0 (Defined at Interface level)
   bDeviceSubClass         0
   bDeviceProtocol         0
   bMaxPacketSize0         8
   idVendor           0x0764 Cyber Power System, Inc.
   idProduct          0x0501 CP1500 AVR UPS
   bcdDevice            0.01
   iManufacturer           3 CPS
   iProduct                1 Value1200E
   iSerial                 0
   bNumConfigurations      1
   Configuration Descriptor:
     bLength                 9
     bDescriptorType         2
     wTotalLength           34
     bNumInterfaces          1
     bConfigurationValue     1
     iConfiguration          0
     bmAttributes         0x80
       (Bus Powered)
     MaxPower               50mA
     Interface Descriptor:
       bLength                 9
       bDescriptorType         4
       bInterfaceNumber        0
       bAlternateSetting       0
       bNumEndpoints           1
       bInterfaceClass         3 Human Interface Device
       bInterfaceSubClass      0 No Subclass
       bInterfaceProtocol      0 None
       iInterface              0
         HID Device Descriptor:
           bLength                 9
           bDescriptorType        33
           bcdHID               1.10
           bCountryCode           33 US
           bNumDescriptors         1
           bDescriptorType        34 Report
           wDescriptorLength     518
          Report Descriptors:
            ** UNAVAILABLE **
       Endpoint Descriptor:
         bLength                 7
         bDescriptorType         5
         bEndpointAddress     0x81  EP 1 IN
         bmAttributes            3
           Transfer Type            Interrupt
           Synch Type               None
           Usage Type               Data
         wMaxPacketSize     0x0008  1x 8 bytes
         bInterval              10
Device Status:     0x0000
   (Bus Powered)

-- 
Matthew Stapleton
Email: matthew4196 at gmail.com

On Aug 7, 2014, at 10:18 PM, Matthew Stapleton <matthew4196 at gmail.com>
wrote:
> I just got a Cyberpower SOHO Value 1200 ELCD UPS and even with nut 2.7.2,
it appears to report battery voltage too low due to the battery scaling function
(In drivers/cps-hid.c).  Even though the ups has usb id: 0764:0501,
UPS.PowerSummary.Voltage reports 26.6 for the 24V batteries so when the 0.667
battery scale is applied that goes down to 17.7V.
I'm wondering if that scale factor is only needed for that one Dynex UPS
model. We should be able to do a string match, which is ugly, but should work
(especially if we default to no scaling).

Logged: https://github.com/networkupstools/nut/issues/142

-- 
Charles Lepple
clepple at gmail

On Aug 7, 2014, at 10:52 PM, Charles Lepple <clepple at gmail.com> wrote:
> On Aug 7, 2014, at 10:18 PM, Matthew Stapleton <matthew4196 at
gmail.com> wrote:
> 
>> I just got a Cyberpower SOHO Value 1200 ELCD UPS and even with nut
2.7.2, it appears to report battery voltage too low due to the battery scaling
function (In drivers/cps-hid.c).  Even though the ups has usb id: 0764:0501,
UPS.PowerSummary.Voltage reports 26.6 for the 24V batteries so when the 0.667
battery scale is applied that goes down to 17.7V.
> 
> I'm wondering if that scale factor is only needed for that one Dynex
UPS model. We should be able to do a string match, which is ugly, but should
work (especially if we default to no scaling).
> 
> Logged: https://github.com/networkupstools/nut/issues/142
It's not just Dynex models - there are some early "UPS VALUE" and
"CP 1000D" units that seem to need the correction factor. I'll try
to add a check that looks to see if the battery voltage makes sense.

-- 
Charles Lepple
clepple at gmail