-
Notifications
You must be signed in to change notification settings - Fork 10
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
Add passive device type and optionally specify usb device #35
base: master
Are you sure you want to change the base?
Conversation
- Added possibility to optionally specify device name of usb device.
Thanks for your contribution 👍 Due the conflict caused by #36 I will merge your changes manually. |
Am 12.01.25 um 22:19 schrieb Łukasz Rybak:
Thanks for your contribution 👍 Due the conflict caused by #36 <#36> I will merge your changes manually.
—
Reply to this email directly, view it on GitHub <#35 (comment)>, or unsubscribe <https://
github.com/notifications/unsubscribe-auth/BKCAJSPPKUA7PN4HPTKXEB32KLL4XAVCNFSM6AAAAABU43AR22VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKOBVHEZDAMJTGA>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
Hello lrybak,
when do you expect to merge this?
Do you need help or have questions?
Regards
Sven
|
I will merge your change today or tomorrow. Already reviewed it and all looks clear. |
Merged already but have some issues with aarch64 build. Need to investigate that |
Thanks all, waiting for the build. |
Finally I was able to find out the root cause of the segfaults during image build (segfault) - tonistiigi/binfmt#215 @s-anderss can you validate your changes on edge version: https://github.com/lrybak/addon-repository-edge |
I tried this Edge version.
My output: (it suddenly ends, the addon keeps running)
|
I don't have that type of device, but shouldn't you configure it that way?
|
@lrybak you are right, I've got it working now! (Not passive btw) Edit: Configuration:
Output:
|
Here the debug log (part of, because still running):
Thank you for making this work! |
Good, it found the device. Try to disable debug mode and add it to home assisstant. |
I disabled it and 6 (all) 1-wire devices were added and are reporting the correct temperature. Thanks again! |
@s-anderss Please review on your side as well |
Am 26.01.25 um 11:00 schrieb Łukasz Rybak:
@s-anderss <https://github.com/s-anderss> Please review on your side as well
—
Reply to this email directly, view it on GitHub <#35 (comment)>, or unsubscribe <https://
github.com/notifications/unsubscribe-auth/BKCAJSPEKNJW4AXQYFKPERL2MSW3ZAVCNFSM6AAAAABU43AR22VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMMJUGMYDKOJVGU>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
Hello Łukasz!
Today I tested the latest version and it work for me as expected.
Thanks for merging the changes.
Regards
Sven
|
This device type is common on USB adapters with an DS9097.
https://owfs.org/index_php_page_usb-usb9097.html
In the HomeAssistant environment I need to specify the usb device
(for instance for a DS9490R USB adapter), otherwise it will not be found.