Summary: | Decoder has trouble decoding valid DMX-512A data | ||
---|---|---|---|
Product: | libsigrokdecode | Reporter: | gabse |
Component: | PD: dmx512 | Assignee: | Nobody <nobody> |
Status: | RESOLVED FIXED | ||
Severity: | major | CC: | Gerhard.Sittig, graham.g0uus, uwe |
Priority: | Normal | ||
Version: | unreleased development snapshot | ||
Target Milestone: | --- | ||
Hardware: | All | ||
OS: | All | ||
Attachments: | Screenshot of the decoder with with my annotations |
Description
gabse
2019-11-21 22:33:49 CET
Yes, sample *.sr files which demonstrate the issue would be great. Ideally, as pull-request for the sigrok-dumps repo on GitHub, if possible: https://github.com/sigrokproject/sigrok-dumps The more *.sr files the better, we like to collect many of those in sigrok-dumps in order to be able to properly (regression-)test protocol decoders, among other things. Had a look at the available literature and at the decoder implementation. There are some conditions where the support in the current version can get improved. It would be helpful to have captures available, to trigger these conditions and verify changes in the decoder's implementation. @gabse: Can you share the recording that you made and which fails to decode properly, as you illustrated? From the image I can only guess that it's a very short inter frame mark but having data available helps gain certainty. There are also error annotations in start/stop bit positions where the ability to zoom in and out helps to better understand the issue. There have also been (informal) reports about incorrect automatic detection of the signal's polarity. It would be great to have captures of these setups, too. (I understand it's the differential RS-485 transmission which results in users' desire to intercept either line.) As Uwe pointed out, having captures available increases test coverage and allows to extend decoders to support currently unsupported conditions. Hello, I have created a Pull request on the sigrok-dumps repo with the DMX dump i had trouble to decode. I will also add a few dumps of other DMX Interfaces if time allows. Maybe also with inverted Signal lines if that helps to further improve the decoder. Pull request: https://github.com/sigrokproject/sigrok-dumps/pull/11 Hello, I finally had the time to take some dumps of different DMX Signals. I have created a Pull request in the sigrok-dumps Git repository. I think it would be better to let the user decide if the input channel is inverted or not, or maybe also a differential pair, instead of the automated detection, which seems to be a partial cause for the bug. Fixed in dca19fbfdf0650dba693c0ac6213f6cdb748a8c3, thanks! This is a new decoder which now stacks on top of UART (needs 250000 as baudrate). If you could grab a few more .sr files that would be great. Ideally with "special" cases (not all-0 or 0-255), e.g. certain lighting setups or whatnot, and cases where certain protocol requirements are violated (the new PD now has features/options to warn about those). *** Bug 1483 has been marked as a duplicate of this bug. *** |