Skip to content

bugfix: fix 30+ Pi runtime crashes across UI panels and widgets#213

Open
gmmcosta15 wants to merge 3 commits intodevfrom
bugfix/ui-panel-crashes
Open

bugfix: fix 30+ Pi runtime crashes across UI panels and widgets#213
gmmcosta15 wants to merge 3 commits intodevfrom
bugfix/ui-panel-crashes

Conversation

@gmmcosta15
Copy link
Copy Markdown
Collaborator

  • Feature
  • Bug fix
  • Code refactor
  • Documentation

Description

  • cancelPage: fix AttributeError on file_metadata dict access.
  • connectionPage: fix stale slot name.
  • inputshaperPage: fix uninitialized currentItem reference.
  • mainWindow: fix _handle_notify (dict.popitemnext(iter(items()))); fix global_back tab-bar gate; fix logger f-string.
  • filamentTab: fix slot signature mismatch; fix stale _is_printing state; fix bare self.isVisible missing parentheses.
  • sensorsPanel: call removeWidget + deleteLater before clearing tracking dict on reconnect to prevent widget stacking.
  • babystepPage: label syncs with confirmed Klipper value; button reorder; code deduplication.
  • probeHelperPage: only process is_active when key is present in update dict.
  • screensaver: guard DPMS calls with hasattr(_dpms_available) check.
  • confirmPage: deduplicate repeated layout logic.
  • networkWindow, utilitiesTab, notificationPage, basePopup, troubleshootPage: minor crash guards and cleanup.
  • printer.py: fix missing key guards on Klipper state update dicts.

Motivation

30+ crashes observed on Pi hardware during normal operation. Each fix is isolated to its widget. Related to the root PR #200.

@gmmcosta15 gmmcosta15 self-assigned this Apr 13, 2026
@gmmcosta15 gmmcosta15 requested a review from HugoCLSC April 13, 2026 17:12
@gmmcosta15 gmmcosta15 marked this pull request as draft April 13, 2026 17:13
@gmmcosta15 gmmcosta15 marked this pull request as ready for review April 13, 2026 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant