[Nut-upsdev] <Cyber Energy> <usbhid-ups/cps-hid.c>-Add HID Usage (Temperature)

[Yvonne.Chen]

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: Yvonne.Chen at cyberenergy.com<mailto:Yvonne.Chen at cyberenergy.com>

From: Jim Klimov <jimklimov+nut at gmail.com>
Sent: Thursday, December 5, 2024 12:04 AM
To: Yvonne.Chen <Yvonne.Chen at Cyberenergy.com>
Cc: nut-upsdev at alioth-lists.debian.net
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 <Yvonne.Chen at cyberenergy.com<mailto:Yvonne.Chen at cyberenergy.com>> 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: Yvonne.Chen at cyberenergy.com<mailto:Yvonne.Chen at cyberenergy.com>
_______________________________________________
Nut-upsdev mailing list
Nut-upsdev at alioth-lists.debian.net<mailto:Nut-upsdev at alioth-lists.debian.net>
https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/nut-upsdev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://alioth-lists.debian.net/pipermail/nut-upsdev/attachments/20250113/0b5ea579/attachment-0001.htm>