Hi Jim:

After doing comprehensive tests, we have confirmed that there are no problems 
with the source code update related to temperature this time. We kindly request 
you to update the source code concerning temperature in cps-hid.c (driver: 
USBHID-UPS(8)) and include it in the next official release of NUT.

Additionally, please inform us of the planned timeline and version for the 
upcoming official release of NUT so we can verify it accordingly. Thank you!

Best regards
Yvonne Chen

Cyber Energy Co., Ltd.
Tel: +886-2-8792-9628 #605
Fax: +886-2-8792-9626
Email: [email protected]<mailto:[email protected]>

From: Jim Klimov <[email protected]>
Sent: Thursday, December 5, 2024 12:04 AM
To: Yvonne.Chen <[email protected]>
Cc: [email protected]
Subject: Re: [Nut-upsdev] <Cyber Energy> <usbhid-ups/cps-hid.c>-Add HID Usage 
(Temperature)

Looks good.

Posted the PR as https://github.com/networkupstools/nut/pull/2711, code base to 
test would be available at https://github.com/jimklimov/nut/tree/cps-hid-temp 
currently or as part of https://github.com/networkupstools/nut/tree/master 
after the pull request passes our CI and gets merged.

Thanks,
Jim Klimov


On Wed, Dec 4, 2024 at 9:11 AM Yvonne.Chen 
<[email protected]<mailto:[email protected]>> wrote:
Hi NUT:

We propose adding HID usage for "Temperature" to the NUT source code file 
cps-hid.c (driver: 
USBHID-UPS(8)<https://networkupstools.org/docs/man/usbhid-ups.html>) so that 
UPS temperature information can be displayed in the UPS status list. After 
conducting internal tests, we confirmed that inserting the following code:
{ "ups.temperature", 0, 0, "UPS.PowerSummary.Temperature", NULL, "%s", 0, 
kelvin_celsius_conversion },
after line 236 allows UPS temperature data to be displayed in the status list.

In NUT source code file cps-hid.c , the line 236 is
{ "ups.firmware", 0, 0, "UPS.PowerSummary.CPSFirmwareVersion", NULL, "%s", 
HU_FLAG_STATIC, stringid_conversion },
Please add the following code after line 236 for our new HID usage 
(Temperature).
{ "ups.temperature", 0, 0, "UPS.PowerSummary.Temperature", NULL, "%s", 0, 
kelvin_celsius_conversion },

Once you have confirmed the request to add the source code, please provide a 
test version of the NUT software with the added HID usage (Temperature) code 
for our final testing and verification. Please contact me if you have any 
questions regarding this request. Thank you very much!

Best regards
Yvonne Chen

Cyber Energy Co., Ltd.
Tel: +886-2-8792-9628 #605
Fax: +886-2-8792-9626
Email: [email protected]<mailto:[email protected]>
_______________________________________________
Nut-upsdev mailing list
[email protected]<mailto:[email protected]>
https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/nut-upsdev
_______________________________________________
Nut-upsdev mailing list
[email protected]
https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/nut-upsdev

Reply via email to