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

rtl_433 MQTT Auto Discovery (next) does not start #159

Closed
GCV-Sleeper-Service opened this issue Sep 26, 2023 · 12 comments
Closed

rtl_433 MQTT Auto Discovery (next) does not start #159

GCV-Sleeper-Service opened this issue Sep 26, 2023 · 12 comments

Comments

@GCV-Sleeper-Service
Copy link

The problem

RTL_433 Next is installed and started. I see messages in MQTT Explorer

When I install rtl_433 MQTT Auto Discovery (next), it does not start. Log output is shown below. Also, in the HAOS log there is following entry
2023-09-25 23:01:35.209 ERROR (MainThread) [homeassistant.components.hassio] Failed to to call /addons/9b13b3f4_rtl433mqttautodiscovery-next/stats - Container addon_9b13b3f4_rtl433mqttautodiscovery-next is not running

What addon are you reporting the bug for?

rtl_433_mqtt_autodiscover

What is the addon version?

0.4.0

What type of MQTT Broker are you using?

Home Assistant Mosquitto MQTT Broker

Addon log messages

s6-rc: info: service s6rc-oneshot-runner: starting
s6-rc: info: service s6rc-oneshot-runner successfully started
s6-rc: info: service fix-attrs: starting
s6-rc: info: service fix-attrs successfully started
s6-rc: info: service legacy-cont-init: starting
s6-rc: info: service legacy-cont-init successfully started
s6-rc: info: service legacy-services: starting
s6-rc: info: service legacy-services successfully started
mqtt found in this Home Assistance instance.
Starting rtl_433_mqtt_hass.py...
usage: rtl_433_mqtt_hass.py [-h] [-d] [-q] [-u USER] [-P PASSWORD] [-H HOST]
                            [-p PORT] [-c CA_CERT] [-r] [-f] [-R RTL_TOPIC]
                            [-D DISCOVERY_PREFIX] [-i DISCOVERY_INTERVAL]
                            [-x EXPIRE_AFTER] [-I IDS [IDS ...]]
rtl_433_mqtt_hass.py: error: unrecognized arguments: -T null
s6-rc: info: service legacy-services: stopping
s6-rc: info: service legacy-services successfully stopped
s6-rc: info: service legacy-cont-init: stopping
s6-rc: info: service legacy-cont-init successfully stopped
s6-rc: info: service fix-attrs: stopping
s6-rc: info: service fix-attrs successfully stopped
s6-rc: info: service s6rc-oneshot-runner: stopping
s6-rc: info: service s6rc-oneshot-runner successfully stopped

Additional information

No response

@catduckgnaf
Copy link

catduckgnaf commented Sep 28, 2023

I am fixing things. Just made this

https://github.com/catduckgnaf/rtl_433_ha

I am working on updating everything to be streamlined. This will work with next or not.

@SunsetGolfer
Copy link

Same problem here 👎
How can I help?

@deviantintegral
Copy link
Collaborator

Thanks for offering!

I reviewed the above repo for changes that would clearly fix the argument parsing I could patch or cherry-pick in, but I didn't find one.

My time is limited at the moment (but will be opening up at the end of the month). I'm more than happy to merge a PR with 1-2 reviews, understanding I may not actually be able to test it myself before merging.

@catduckgnaf
Copy link

Thanks for offering!

I reviewed the above repo for changes that would clearly fix the argument parsing I could patch or cherry-pick in, but I didn't find one.

My time is limited at the moment (but will be opening up at the end of the month). I'm more than happy to merge a PR with 1-2 reviews, understanding I may not actually be able to test it myself before merging.

Hello, I've made some changes since then.

I am working on one or two things, and then really want to actively develop and work on things.
I would be happy to take over maintaining the project, however I plan do make alot more changes, so I guess when I'm ready I'll continue with a Fork instead. I don't think that is the best solution for the community. I have the time, passion, and devices to pick up the torch, would much rather you spend your time helping me get up to speed.

Let me know

@fortyfortynine
Copy link

I am fixing things. Just made this

https://github.com/catduckgnaf/rtl_433_ha

I am working on updating everything to be streamlined. This will work with next or not.

How do I use your script? I was able to use your template and now the log is showing my devices but a little lost on how to use the script.

@catduckgnaf
Copy link

I am fixing things. Just made this
https://github.com/catduckgnaf/rtl_433_ha
I am working on updating everything to be streamlined. This will work with next or not.

How do I use your script? I was able to use your template and now the log is showing my devices but a little lost on how to use the script.

I have no idea why they don't want to merge my fix, at least for the time being. But I will be working on an intergration.
Quite honestly the script is mostly useless. It doesn't discover anything but weather sensors. Another reason I wanted to take ownership

unverbraucht added a commit to unverbraucht/rtl_433-hass-addons that referenced this issue Oct 23, 2023
@fortyfortynine
Copy link

I am fixing things. Just made this
https://github.com/catduckgnaf/rtl_433_ha
I am working on updating everything to be streamlined. This will work with next or not.

How do I use your script? I was able to use your template and now the log is showing my devices but a little lost on how to use the script.

I have no idea why they don't want to merge my fix, at least for the time being. But I will be working on an intergration. Quite honestly the script is mostly useless. It doesn't discover anything but weather sensors. Another reason I wanted to take ownership

I’ll give a try adding my sensors in configuration.yaml. I was using another add-on (acurite2mqtt) but it doesn’t support adding any thing using protocol higher than 223. Trying to use rtl_433 so I can get a Govee water leak sensor using 231 added to Home Assistant.

@catduckgnaf
Copy link

I am fixing things. Just made this
https://github.com/catduckgnaf/rtl_433_ha
I am working on updating everything to be streamlined. This will work with next or not.

How do I use your script? I was able to use your template and now the log is showing my devices but a little lost on how to use the script.

I have no idea why they don't want to merge my fix, at least for the time being. But I will be working on an intergration. Quite honestly the script is mostly useless. It doesn't discover anything but weather sensors. Another reason I wanted to take ownership

I’ll give a try adding my sensors in configuration.yaml. I was using another add-on (acurite2mqtt) but it doesn’t support adding any thing using protocol higher than 223. Trying to use rtl_433 so I can get a Govee water leak sensor using 231 added to Home Assistant.

Its sad, but you are so right, most people seem to not use the script and so what you are saying. I am simply going to work on redoing this. I'd rather learn to code verse fight pointless gatekeeping

@fortyfortynine
Copy link

I am fixing things. Just made this
https://github.com/catduckgnaf/rtl_433_ha
I am working on updating everything to be streamlined. This will work with next or not.

How do I use your script? I was able to use your template and now the log is showing my devices but a little lost on how to use the script.

I have no idea why they don't want to merge my fix, at least for the time being. But I will be working on an intergration. Quite honestly the script is mostly useless. It doesn't discover anything but weather sensors. Another reason I wanted to take ownership

I’ll give a try adding my sensors in configuration.yaml. I was using another add-on (acurite2mqtt) but it doesn’t support adding any thing using protocol higher than 223. Trying to use rtl_433 so I can get a Govee water leak sensor using 231 added to Home Assistant.

Its sad, but you are so right, most people seem to not use the script and so what you are saying. I am simply going to work on redoing this. I'd rather learn to code verse fight pointless gatekeeping

Thank you! I’ll keep my eye on your integration for when it’s published.

@unverbraucht
Copy link
Contributor

I fixed this in in PR #162. I'm not aware of any way how you can try this out in HA because I don't know how to specify branches when adding repos. If you want to feel free to download the version from my PR and install it locally as described in https://developers.home-assistant.io/docs/add-ons/tutorial

@unverbraucht
Copy link
Contributor

Probably caused by #164 and not #162 :)

@deviantintegral
Copy link
Collaborator

I just finished rolling a new release, and now that config.json's fixes are on main this should be fixed. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants