Skip to content

EEPROM/RTC manager: auto-sync on reset, SN display, per-module sections#10

Merged
kaklik merged 4 commits into
masterfrom
feature/eeprom-rtc-manager-ui
Apr 25, 2026
Merged

EEPROM/RTC manager: auto-sync on reset, SN display, per-module sections#10
kaklik merged 4 commits into
masterfrom
feature/eeprom-rtc-manager-ui

Conversation

@roman-dvorak

Copy link
Copy Markdown
Member

This pull request introduces significant improvements to the EEPROM Manager UI and logic, enhancing support for different module types ("detector" and "battery") and improving user experience. The changes include dynamic UI adjustments based on module type, accurate handling and display of calibration coefficients as float32, addition of serial number reading, and improved RTC synchronization handling. Below are the most important changes grouped by theme:

Module Type Awareness and Dynamic UI:

  • The EepromManagerWidget and related functions now accept a module_type parameter ("detector" or "battery"), allowing the UI and logic to adapt based on the connected hardware. This affects which configuration/calibration fields are shown, ensuring only relevant options appear for each module type. [1] [2] [3] [4] [5] [6] [7] [8]

  • RTC synchronization fields are now only visible for battery modules, while calibration fields are only shown for detector modules. Hidden fields are created for compatibility when not displayed, preserving data integrity when reading/writing. [1] [2]

Calibration and Float32 Handling:

  • Calibration coefficients are now handled as float32 values, both in parsing and display. The UI uses text fields with scientific notation support, and values are rounded and formatted to true float32 precision for accurate round-trip representation. [1] [2] [3] [4] [5]

  • Added a "Now" button to quickly set the current Unix timestamp for calibration, improving usability. [1] [2]

Serial Number Reading:

  • The EEPROM Manager now reads and displays the device's serial number (SN) directly from the appropriate EEPROM chip, with logic to select the correct address based on module type. The SN is shown in the UI as a 128-bit hexadecimal value. [1] [2] [3] [4] [5] [6]

Improved Error Handling and Feedback:

  • Added error messages for missing HID interfaces during device connection, improving diagnostics when hardware is not detected.

  • Improved status messages and error handling throughout the EEPROM Manager and RTC Manager, including clear feedback for file loads, device loads, and RTC operations. [1] [2]

RTC Manager Enhancement:

  • After resetting the RTC, the RTC Manager now automatically applies synchronization, ensuring calibration points are updated and reducing manual steps for the user.

@roman-dvorak

Copy link
Copy Markdown
Member Author

@copilot resolve the merge conflicts in this pull request

Copilot AI and others added 2 commits April 22, 2026 22:38
Co-authored-by: roman-dvorak <5196729+roman-dvorak@users.noreply.github.com>

Copilot AI commented Apr 22, 2026

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

Merge conflicts resolved in 8bbdf2c. There was one conflicting file (dosview/__init__.py) with two conflicts:

  1. HID interface error handling — kept master's cleaner approach using errorOccurred signal and a try block over the feature branch's version with a print-based check.
  2. A comment — kept master's English version over the Czech one in the feature branch.

Also updated tests/test_parser.py to handle the 6-value return from parse_file (master added telemetry and spectral_matrix return values), which had broken 2 tests. All 4 tests now pass.

# Conflicts:
#	dosview/eeprom_widget.py
#	tests/test_parser.py
@kaklik kaklik merged commit 22c57a9 into master Apr 25, 2026
2 of 6 checks passed
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.

3 participants