Change [Arad / Master Meter Dialog3G] preamble alignment, overrides, and documentation#3489
Open
avicarmeli wants to merge 2 commits intomerbanan:masterfrom
Open
Change [Arad / Master Meter Dialog3G] preamble alignment, overrides, and documentation#3489avicarmeli wants to merge 2 commits intomerbanan:masterfrom
avicarmeli wants to merge 2 commits intomerbanan:masterfrom
Conversation
Improve robustness of the Arad / Master Meter Dialog3G decoder.
Preamble detection now uses a configurable nibble window inside the
observed 48-bit preamble while payload extraction always aligns to the
start of the full preamble. This keeps decoding stable even if the match
window changes.
Add a diagnostic field "unmatched_preamble" which outputs the preamble
nibbles outside the match window in inverted nibble form:
<before>_..._<after>
This helps analyze variants of the Dialog3G transmission.
Add decoder options:
- mandatory serial input
- gear override
- unit override
These allow forcing decoding parameters when meter configuration or
protocol variants differ from the automatically detected values.
Update protocol documentation to reflect observed messages:
* the trailing FF byte is not constant
* the serial suffix byte sometimes matches the printed meter letter
* the suffix byte together with the following byte likely indicates
meter unit and gear ratio
Add BitBench alignment reference:
PREAMBLE_ALIGN = c196f51385
(invert of 3e690aec7a)
Format string used for analysis:
UID:16h SERIAL:<24d 8h 8h
COUNTER:<32d 8h8h 8h8h 8h8h
SUFFIX:hh
Contributor
Author
|
Tested locally on Windows/MSVC and in GitHub CI. Example frames used during analysis: |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Improve robustness of the Arad / Master Meter Dialog3G decoder.
Preamble detection now uses a configurable nibble window inside the observed 48-bit preamble while payload extraction always aligns to the start of the full preamble. This keeps decoding stable even if the match window changes.
Add a diagnostic field "unmatched_preamble" which outputs the preamble nibbles outside the match window in inverted nibble form:
This helps analyze variants of the Dialog3G transmission.
Add decoder options:
These allow forcing decoding parameters when meter configuration or protocol variants differ from the automatically detected values.
Update protocol documentation to reflect observed messages:
Add BitBench alignment reference:
(invert of 3e690aec7a)
Format string used for analysis:
Example frame used during analysis:
3e690aec7ac84b9aa50900404a7a02000c42bb5e8cc0f8