Skip to content

Restore compact init progress; route Display Reboot-Fake and dialog statuses into existing progress/status area#84

Merged
dosordie merged 1 commit into
workfrom
codex/korrigiere-pr-#83-und-entferne-zusatzlabel
Jun 29, 2026
Merged

Restore compact init progress; route Display Reboot-Fake and dialog statuses into existing progress/status area#84
dosordie merged 1 commit into
workfrom
codex/korrigiere-pr-#83-und-entferne-zusatzlabel

Conversation

@dosordie

Copy link
Copy Markdown
Owner

Motivation

  • Der neu eingeführte separate init_progress_text_label verschob das Layout und zeigte Init-Details außerhalb des vorhandenen Fortschritts-/Statusbereichs, was nicht gewünscht ist.
  • Der Display-Init-Button (Alle bekannten Register lesen) nutzt in der Praxis den Reboot-Fake-Pfad und dieser muss im bestehenden Fortschrittsbalken sichtbar werden, inklusive Busy-/Versuchs-/Timeout-Zuständen.
  • Dialoge wie WP-Steuerung und AT-Kompensation brauchten sichtbare Lese-/Schreib- und Timeout-Statusmeldungen und Timer-Schreibvorgänge blieben teilweise in einem hängenden „schreibe ...“-Zustand.

Description

  • Entfernt das separate Label self.init_progress_text_label und die zugehörigen Layoutänderungen, so dass die Init-Zeile wieder kompakt bleibt neben init_read_btn und init_progress_bar.
  • Leitet Init-/Display-Statustexte über init_progress_bar.setFormat(...) (via _set_init_progress_text) und hält Busy-Zustände mit setRange(0,0), statt ein neues Label zu benutzen.
  • Fügt _display_reboot_progress/_display_reboot_finish hinzu und bindet den Display-Modbus Reboot-Fake-Pfad (5112 / 0BC3 / upload) an diese Funktionen, sodass Status wie Display: Reboot-Fake startet ..., Display: Versuch 1/3, Display: 0BC3=0x8000 setzen ..., Display: warte auf Parameterpakete ..., Display: Paket 1001 empfangen, Display: fertig und Display: Timeout im vorhandenen Balken erscheinen.
  • Stellt sicher, dass display_fake_reboot_state in _init_read_progress_counts und _update_init_read_button_state berücksichtigt wird, damit der Init-Button korrekt blockiert/freigegeben wird und wiederholtes Klicken nicht hängen bleibt.
  • Aktualisiert die Display-HMI-Snapshot-Verarbeitung, um während eines Reboot-Fake-Laufs empfangene Parameterpakete in den Fortschrittsbalken zu melden.
  • Ergänzt WPControlDialog und ATCompensationDialog um ein kleines status_label sowie set_write_status, show_read_success und show_read_timeout und setzt lesbare Texte beim Lesen/Schreiben (z. B. Lese WP-Steuerung ..., WP-Steuerung Schreiben gesendet.).
  • Leitet Read-Responses und Write-ACKs gezielt an die WP-/AT-Dialoge weiter (bei passenden Labels) und sorgt dafür, dass Timer-/SG-/Silent-Statusmeldungen nach Abschluss finalisiert und bei Standard-Writes nach einigen Sekunden auf Bereit. zurückgesetzt werden.

Testing

  • python -m compileall . wurde erfolgreich ausgeführt and es gab keine Syntax-Fehler.
  • PYTHONPATH=. pytest wurde ausgeführt: 16 Tests bestanden, 2 Tests wurden übersprungen.
  • Projekt-Integritätsprüfung (git diff --check / Kompilierung) wurde ausgeführt ohne neue Warnungen in den geänderten Dateien.

Codex Task

@dosordie dosordie merged commit c456965 into work Jun 29, 2026
1 check passed
@dosordie dosordie deleted the codex/korrigiere-pr-#83-und-entferne-zusatzlabel branch July 9, 2026 07:14
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