-
Notifications
You must be signed in to change notification settings - Fork 11
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
0.7.0 #36
Conversation
Aohzan
commented
Mar 21, 2024
•
edited
Loading
edited
- Fix default conf missing error during initialization
- Lint and code improvments
- Bump pyserial-asyncio
- Add service translation
- Handle no USB device found during init
- Replace the jamming sensor by a binary_sensor and add events catching to it
- Allow to specify manual path to USB device
- Disallow multi instance of the integration
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rfpparser.py: why did you remove Delta Dore X2D Starbox related code here ? (askin since I am co-author of these ... and use that in my install..)
it's still a draft MR, I don't know why I removed this code, I will revert that |
ping @Doubledom45 @jaroslawp si vous pouvez review 🙏 |
Slt, ✅👁🗨 |
Yes je m'occupe de ça |
@Doubledom45 j'ai fix le problème qui apparait dans les logs, peux-tu revérifier stp ? sinon je n'ai pas compris le début de ton message, je n'ai pas de sonde Oregon ou autre pour tester cette partie là de mon côté |
Je vais essayer de refaire des tests avec ta version ! |
Modification de l'ID pour Oregon, doit être sur l' adr_channel !
Update rfpparser.py OREGON
Et de plus , il y a 2 valeurs à remontée souvent sur l'OREGON, L'humidité et le température . |
DONC plantage sur la remontée de l'info température ! LOG pour sonde oregon
2eme sonde
LOg pour OWL IL Y A Plusieurs type OWL ici 180i
|
tu as un exemple de code où tu le gères ? |
Le truc c'est que l'on décode pas comme toi ! Je vois que toi tu transformes les values en command, ce qui fait que tu ne peux dissocier les différentes remontées du sensor !
Log du même sensor avec mon add_on au 1er passage ! Ici un peu verbeux, mais simplement pour te montrer !
INFO (MainThread) [custom_components.rfplayer.sensor] RfPlayer Sensor ID : 51204typ J'ai encore pas mal de truc à refaire, mais pour l'instant tourne , il faut que je revoie le config_flow lors du passage sur modif , je pense d'ailleurs trop compliqué et faudrait laisser faire par un envoie de commande ! |
je te propose de merger cette PR si tu ne vois pas de regression, et on peut travailler sur une nouvelle PR avec ton code pour mieux traiter les données ? |