Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Homebridge 2.0 Warnings #23

Closed
awlynn68 opened this issue Aug 13, 2024 · 6 comments
Closed

Homebridge 2.0 Warnings #23

awlynn68 opened this issue Aug 13, 2024 · 6 comments
Labels
enhancement New feature or request

Comments

@awlynn68
Copy link

I recently updated my Homebridge to 2.0-beta. The UPS plugin works, but is constantly throwing the following errors. The listeners number continues to grow with each warning iteration.

8/12/2024, 11:20:57 PM] [UPS] Homebridge: warning: recommended version: API v1.2
[8/12/2024, 11:20:57 PM] [UPS] Homebridge: QBFMX7004179: CPS LX1500GU3 120V UPS with 24V PbAcid battery from CPS
[8/12/2024, 11:20:57 PM] [UPS] On Battery: Configured Name: warning: 2827 listeners
[8/12/2024, 11:20:57 PM] [UPS] On Battery: Mute: warning: 2827 listeners
[8/12/2024, 11:20:57 PM] [UPS] On Battery: Log Level: warning: 2827 listeners
[8/12/2024, 11:20:57 PM] [UPS] Homebridge: warning: TypeError: params.Service: not a Service
at new ServiceDelegate (file:///var/lib/homebridge/node_modules/homebridge-ups/node_modules/homebridge-lib/lib/ServiceDelegate.js:64:13)
at new Battery (file:///var/lib/homebridge/node_modules/homebridge-ups/node_modules/homebridge-lib/lib/ServiceDelegate/Battery.js:55:5)
at new Battery (file:///var/lib/homebridge/node_modules/homebridge-ups/lib/UpsService.js:138:5)
at new UpsAccessory (file:///var/lib/homebridge/node_modules/homebridge-ups/lib/UpsAccessory.js:92:27)
at UpsHost.init (file:///var/lib/homebridge/node_modules/homebridge-ups/lib/UpsHost.js:74:37)
at processTicksAndRejections (node:internal/process/task_queues:95:5)
at UpsPlatform.init (file:///var/lib/homebridge/node_modules/homebridge-ups/lib/UpsPlatform.js:108:7)

The Homebridge plugin JSPN is:
{
"name": "UPS",
"hosts": [
{
"host": "localhost:3493",
"username": "local_mon",
"password": "==================="
}
],
"platform": "UPS",
"_bridge": {
"username": "0E:C4:7A:EA:7C:54",
"port": 53120
}
}

The relevant section of ups.conf is:
[cyberpower]
driver = usbhid-ups
port = auto
vendorid = 0764
pollfreq = 30

@ebaauw
Copy link
Owner

ebaauw commented Aug 13, 2024

HAP-NodeJS v1 has removed BatteryService in favour of Battery.

@ebaauw ebaauw added bug Something isn't working enhancement New feature or request and removed bug Something isn't working labels Aug 13, 2024
ebaauw added a commit to ebaauw/homebridge-lib that referenced this issue Aug 13, 2024
HAP-NodeJS v1 breaking change, see ebaauw/homebridge-ups#23.
@ebaauw
Copy link
Owner

ebaauw commented Aug 13, 2024

Could you please try beta v1.2.12-0?

@awlynn68
Copy link
Author

That fixed the vast majority of the warnings.

Where is the name of the accessory held? I renamed the machine and the network name to eliminate the "-" but these accessories are picking it up from somewhere, even after removing them from the cached accessories list.

Screenshot 2024-08-13 at 17 57 01

@ebaauw
Copy link
Owner

ebaauw commented Aug 14, 2024

That fixed the vast majority of the warnings.

Do you still have any Homebridge UPS related issues?

Where is the name of the accessory held?

In HomeKit. On initial pairing, the name is copied from the accessory (i.c. the Homebridge plugin), but afterwards it lives a life of its own. You can change it in any decent HomeKit app. Same for service names, except that these are now synced with the Configured Name characteristic, which is accessible to the accessory.

@awlynn68
Copy link
Author

I fixed the issue. For some reason remove just the cached UPS did not clear the cached accessory completely. Once I deleted all cached accessories to reverted to the proper name.

I would consider this closed.

@ebaauw
Copy link
Owner

ebaauw commented Aug 14, 2024

Released v1.2.12.

@ebaauw ebaauw closed this as completed Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants