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

usb device resets w/ remote rtl_tcp? #123

Closed
hume-github opened this issue May 4, 2022 · 5 comments
Closed

usb device resets w/ remote rtl_tcp? #123

hume-github opened this issue May 4, 2022 · 5 comments
Labels
bug Something isn't working

Comments

@hume-github
Copy link

The code looks like it does usb resets regardless of whether or not rtl_tcp is local or remote. Is that intentional?

@allangood allangood added the bug Something isn't working label May 5, 2022
@allangood
Copy link
Owner

allangood commented May 5, 2022

Absolutely not intentional! :)
Thank you for pointing it out!

@randysaz928
Copy link

The code appears to not check for whether remote rtl_tcp is used until pretty late, looks like in the meters filter setup section. By then it has already found or guessed a rtlsdr and tried to reset it. If you dont have a local radio it will also exit before getting to the remote rtl_tcp setup.
Commenting line 309 " sys.exit(1)" should allow it to get to the setup of the meters, and the remote rtl_tcp setup when no local radio is present. Albeit with errors about USB/devices being invalid.

My issue is that I have a local radio on HA that I use rtl433 and a remote rtl_tcp instance for rtlamr. Weather station is in the back yard on 433 near my office and the HA server, and my water meter on 915 is the other direction toward the front of the house..

I suppose it is time for me to learn python... :(

@allangood
Copy link
Owner

Yes, this problem was introduced with the USB reset functionality in version 2.0.0.

You can either use the previous version or wait a few days until I finish the fix.

I am organizing the code a bit and fixing this bug alone would make me redo a lot of things, that is why is taking more than usual to fix it.

@allangood
Copy link
Owner

PR #131 has been merged!

This PR has a lot of fixes and changes to the code!

@allangood
Copy link
Owner

Small correction on #132
Thank you @shbatm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants