-
Notifications
You must be signed in to change notification settings - Fork 34
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
MU-Nachrichten werden z.T. als MC erkannt, z.B. Siro und andere #72
Comments
Ich habe mal was versucht: 4ed6c3b Problem ist vermutlich, dass ich den Takt erst später errechne und erstmal annehme, dass einer der Pulse = Takt ist. Ich habe das ganze jetzt mal auf einen langen Puls geändert. Ob das so wirklich besser ist weiss ich nicht. Diese Signalfolge wird nun als MU gewertet: |
Hallo @sidey79,
|
Ist das Problem noch vorhanden oder zwischenzeitlich gelöst? |
Dieses Problem besteht immer noch. Hier eine FS20- und eine FHT-Nachricht: 2018.02.18 17:38:08 4: sduino868IP/msg READ: �Mc;LL=-610;LH=579;SL=-402;SH=366;D=FFF5D5F5DDFFF7BED58;C=326;L=73;R=245;� |
Ok verstehe. Das Problem ist nur, wenn ich die MU Nachrichten aus deinem Log teste, kommen die natürlich als MU raus. |
@elektron-bbs Hier siehst du wie man DMC aktivieren kann in der Firmware. (signalDecoder.cpp) |
Das ist doch auch das Thema hier: "MU-Nachrichten werden z.T. als MC erkannt" |
Vielleicht stehe ich auf dem Schlauch... Ich habe diverse Signale gegen die aktuelle Lib getestet und konnte den Fehler nicht mehr nachstellen. Habe aber jetzt eine Signalfolge, mit der sich der Fehler nachstellen lässt. |
Ich habe leider noch nichts finden können, wie ich dieses Fehlverhalten unterbinden, bin aber dran was zu suchen. |
In ManchesterpatternDecoder isManchester wird geprüft, ob longlow + longhigh und longhigh+longlow als Sequenz vorkommen. Dies ist notwendig, um eine bessere MC Erkennung zu erreichen. In der Regel, sollten MC Signale auch beide Sequenzen enthalten. Auch wenn dies nicht 100% garantiert werden kann. z.B bei FFFF ist dies nicht der Fall. Dieser Fall wird aber auch heute bereits nicht erkannt. fixes #76, #72
Ich denke ich habe einen Weg gefunden, diese Signale nicht mehr als MC zu erkennen. |
In ManchesterpatternDecoder isManchester wird geprüft, ob longlow + longhigh und longhigh+longlow als Sequenz vorkommen. Dies ist notwendig, um eine bessere MC Erkennung zu erreichen. In der Regel, sollten MC Signale auch beide Sequenzen enthalten. Auch wenn dies nicht 100% garantiert werden kann. z.B bei FFFF ist dies nicht der Fall. Dieser Fall wird aber auch heute bereits nicht erkannt. fixes #76, #72
Für die Übersicht, lieber ein neuer Faden.
Immer noch sporadische fälschliche Erkennung nach cfb3738 Anpassung (signalDecoder.cpp).
Zum Teil FHTTK oder FHT80b. da besteht noch Potential ;-)
...
The text was updated successfully, but these errors were encountered: