Skip to content

ASCII-Blockkopf-Dekoder, 1326–1333 Umbenennung, 20xx/21xx auf Read#89

Merged
dosordie merged 1 commit into
workfrom
codex/verbessere-ascii-decoder-und-umbenenne-register
Jul 12, 2026
Merged

ASCII-Blockkopf-Dekoder, 1326–1333 Umbenennung, 20xx/21xx auf Read#89
dosordie merged 1 commit into
workfrom
codex/verbessere-ascii-decoder-und-umbenenne-register

Conversation

@dosordie

Copy link
Copy Markdown
Owner

Motivation

  • Blockkopf-/Kennungsregister sollen auch dann druckbare ASCII-Zeichen anzeigen, wenn das Wort ein druckbares Byte zusammen mit 0x00/0xFF oder anderen nicht-druckbaren Bytes enthält.
  • Die deutsche Bezeichnung für Register 1326–1333 soll fachlich korrekter als „Zirkulationspumpe“ statt „Rücklaufpumpe“ angegeben werden.
  • Bereits erkannte Status-/Zähler-/Diagnosewerte im Bereich 20xx/21xx dürfen nicht als schreibbar ausgewiesen werden.

Description

  • Ändert den ASCII-Dekoder MainWindow._decode_printable_ascii_word in foxair_phnix_control.py auf byteweise Prüfung, wobei 0x00 und 0xFF sowie nicht-druckbare Bytes ignoriert werden und druckbare ASCII-Zeichen erhalten bleiben; die Fall‑Back-Ausgabe Reserve bleibt erhalten, wenn kein Zeichen übrig bleibt.
  • Benennt die deutschen Registerbezeichnungen RücklaufpumpeZirkulationspumpe für die Register 1326–1333 in data/foxair_phnix_registers.json und passt die Erwähnung in docs/public/README.txt an.
  • Setzt für die relevanten 20xx/21xx Einträge die mode-Felder von write/r/w auf read/read-only, sodass Status/Diagnose-/Zählerwerte nicht als schreibbar erscheinen (Änderungen nur Bezeichnung/Mode, keine Logik).
  • Betroffene Dateien: foxair_phnix_control.py, data/foxair_phnix_registers.json, docs/public/README.txt.

Testing

  • python -m compileall . wurde ausgeführt und war erfolgreich.
  • PYTHONPATH=. pytest lief erfolgreich und ergab 16 passed, 2 skipped.
  • Ein kurzes Smoke-Skript/AST-Check validierte mehrere ASCII-Dekodierungsbeispiele (0x6135 -> 'a5', 0x7800 -> 'x', gesamter beobachteter 200er-HMI-Block a5c6utQfC8x) sowie die Register-Umbenennung und dass keine 20xx/21xx-Einträge mehr mode: write haben, und lief erfolgreich.

Codex Task

@dosordie dosordie merged commit ff096a3 into work Jul 12, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant