trcc-linux 9.6.3__tar.gz → 9.6.4__tar.gz

{trcc_linux-9.6.3 → trcc_linux-9.6.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: trcc-linux
-Version: 9.6.3
+Version: 9.6.4
 Summary: Linux implementation of Thermalright LCD Control Center
 Project-URL: Homepage, https://github.com/Lexonight1/thermalright-trcc-linux
 Project-URL: Documentation, https://github.com/Lexonight1/thermalright-trcc-linux#readme

{trcc_linux-9.6.3 → trcc_linux-9.6.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "trcc-linux"
-version = "9.6.3"
+version = "9.6.4"
 description = "Linux implementation of Thermalright LCD Control Center"
 readme = "README.md"
 license = "GPL-3.0-or-later"

{trcc_linux-9.6.3 → trcc_linux-9.6.4}/src/trcc/__version__.py

@@ -1,4 +1,4 @@
 """TRCC Linux version information. History in doc/CHANGELOG.md."""
 
-__version__ = "9.6.3"
+__version__ = "9.6.4"
 __version_info__ = tuple(int(x) for x in __version__.split("."))

{trcc_linux-9.6.3 → trcc_linux-9.6.4}/src/trcc/adapters/device/linux/detector.py

@@ -58,7 +58,10 @@ def _resolve_vid_pid(sysfs_base: str) -> tuple[int, int] | None:
                 return v, p
     except (OSError, ValueError):
         pass
-    log.warning("sysfs VID/PID walk failed for %s — skipping device", sysfs_base)
+    # Common for non-USB SCSI generic devices (CD-ROMs, virtual disks) —
+    # they have no idVendor/idProduct in their parent chain.  Not a defect;
+    # we just skip those devices.  Debug-level so it doesn't trip user reports.
+    log.debug("sysfs %s has no USB parent — skipping (non-USB SCSI device)", sysfs_base)
     return None
 
 

{trcc_linux-9.6.3 → trcc_linux-9.6.4}/src/trcc/adapters/device/scsi_protocol.py

@@ -42,7 +42,11 @@ class ScsiProtocol(LazyTransportProtocol):
     def _open_transport(self) -> Any:
         fn = DeviceProtocolFactory._scsi_transport_fn
         if fn is None:
-            log.error("SCSI transport factory not injected")
+            # Pre-boot ScsiProtocol instance (created before the
+            # composition root finished wiring) — graceful no-op; the
+            # caller bails on transport=None and a later call after
+            # boot completes succeeds.  Debug-level: not a defect.
+            log.debug("SCSI transport factory not yet wired — skipping open")
             return None
         log.debug("Opening SCSI transport: %s%s",
                   self._path,

{trcc_linux-9.6.3 → trcc_linux-9.6.4}/src/trcc/adapters/infra/data_repository.py

@@ -381,9 +381,14 @@ class DataManager:
             user_dir=os.path.join(DataManager._data_dir(), name),
             archive_name=f'{name}.7z',
             check_fn=has_themes,
-            fetch_fn=lambda a: DataManager._fetch_archive(a),
+            fetch_fn=DataManager._fetch_archive,
         )
 
+    @staticmethod
+    def _fetch_web_archive(archive_name: str) -> str | None:
+        """Bound fetcher for the 'web' subdirectory — named, not a lambda."""
+        return DataManager._fetch_archive(archive_name, 'web')
+
     @staticmethod
     def ensure_web(width: int, height: int) -> bool:
         """Extract cloud theme previews from .7z archive if not already present."""
@@ -394,7 +399,7 @@ class DataManager:
             user_dir=os.path.join(DataManager._data_dir(), 'web', res_key),
             archive_name=f'{res_key}.7z',
             check_fn=_has_any_content,
-            fetch_fn=lambda a: DataManager._fetch_archive(a, 'web'),
+            fetch_fn=DataManager._fetch_web_archive,
         )
 
     @staticmethod
@@ -407,7 +412,7 @@ class DataManager:
             user_dir=os.path.join(DataManager._data_dir(), 'web', res_key),
             archive_name=f'{res_key}.7z',
             check_fn=has_themes,
-            fetch_fn=lambda a: DataManager._fetch_archive(a, 'web'),
+            fetch_fn=DataManager._fetch_web_archive,
         )
 
     @staticmethod

{trcc_linux-9.6.3 → trcc_linux-9.6.4}/src/trcc/adapters/infra/dc_writer.py

@@ -596,4 +596,6 @@ def read_carousel(filepath: str) -> CarouselConfig | None:
         return None
 
 
-_metric_to_hardware_ids = lambda metric: METRIC_TO_IDS.get(metric, (0, 0))  # noqa: E731
+def _metric_to_hardware_ids(metric: str) -> tuple[int, int]:
+    """Map a metric name to its (hardware_kind, hardware_idx) pair."""
+    return METRIC_TO_IDS.get(metric, (0, 0))

{trcc_linux-9.6.3 → trcc_linux-9.6.4}/src/trcc/adapters/system/windows/sources/_base.py

@@ -29,6 +29,7 @@ from __future__ import annotations
 
 import logging
 from abc import ABC, abstractmethod
+from operator import attrgetter
 from typing import TYPE_CHECKING, ClassVar
 
 if TYPE_CHECKING:
@@ -98,7 +99,7 @@ class WindowsSensorSource(ABC):
         """
         return sorted(
             (sub() for sub in cls._registry.values()),
-            key=lambda src: src.priority,
+            key=attrgetter('priority'),
         )
 
     # ── Abstract API ───────────────────────────────────────────────────────

{trcc_linux-9.6.3 → trcc_linux-9.6.4}/src/trcc/core/models/theme.py

@@ -218,6 +218,11 @@ class ThemeConfig:
     mask_y: int = 0                    # YvalMB
 
 
+def _default_theme_indices() -> list[int]:
+    """6 carousel slots, each pointing to "unset" (-1) until the user assigns."""
+    return [-1, -1, -1, -1, -1, -1]
+
+
 @dataclass(slots=True)
 class CarouselConfig:
     """Carousel/slideshow configuration."""
@@ -225,7 +230,7 @@ class CarouselConfig:
     enabled: bool = False              # isLunbo
     interval_seconds: int = 3          # myLunBoTimer (minimum 3)
     count: int = 0                     # lunBoCount
-    theme_indices: list[int] = field(default_factory=lambda: [-1, -1, -1, -1, -1, -1])
+    theme_indices: list[int] = field(default_factory=_default_theme_indices)
     lcd_rotation: int = 1              # myLddVal (1-3): split mode style, NOT rotation
 
 

{trcc_linux-9.6.3 → trcc_linux-9.6.4}/src/trcc/services/theme.py

@@ -67,6 +67,11 @@ def _copy_flat_files(src: Path, dest: Path) -> None:
             shutil.copy2(str(f), str(dest / f.name))
 
 
+def _theme_sort_key(theme: ThemeInfo) -> tuple[bool, str]:
+    """Sort: built-in themes first (False < True), then alphabetical by name."""
+    return (theme.name.startswith(('User', 'Custom')), theme.name)
+
+
 class ThemeService:
     """Theme lifecycle: discover, load, save, export, import.
 
@@ -261,10 +266,7 @@ class ThemeService:
                             seen_names.add(item.name)
                             themes.append(theme)
 
-        themes.sort(key=lambda t: (
-            t.name.startswith(('User', 'Custom')),
-            t.name,
-        ))
+        themes.sort(key=_theme_sort_key)
         return themes
 
     @staticmethod

{trcc_linux-9.6.3 → trcc_linux-9.6.4}/src/trcc/ui/gui/__init__.py

@@ -141,7 +141,10 @@ def launch(verbosity: int = 0, decorated: bool = False,
     # QThread) starts the OS-provided MetricsLoop after device enumeration.
 
     # ── IPC raise + signals ───────────────────────────────────────────────
-    signal.signal(signal.SIGINT, lambda *_: qapp.quit())
+    def _on_sigint(*_args):
+        """SIGINT handler — quit the Qt event loop cleanly."""
+        qapp.quit()
+    signal.signal(signal.SIGINT, _on_sigint)
     platform.wire_ipc_raise(qapp, window)
     # Issue #143 ("USB communication lost" after GUI exit) is handled at
     # the kernel level — udev rule sets power/autosuspend_delay_ms=30000

{trcc_linux-9.6.3 → trcc_linux-9.6.4}/src/trcc/ui/gui/uc_sensor_picker.py

@@ -77,7 +77,7 @@ class SensorRow(QWidget):
         if not self._cb_off.isNull():
             self._cb.setIcon(QIcon(self._cb_off))
             self._cb.setIconSize(QSize(CHECKBOX_SIZE, CHECKBOX_SIZE))
-        self._cb.clicked.connect(lambda: self.clicked.emit(self.sensor.id))
+        self._cb.clicked.connect(self._on_checkbox_clicked)
 
         # Name label
         self._name = QLabel(sensor.name, self)
@@ -102,6 +102,10 @@ class SensorRow(QWidget):
         if not px.isNull():
             self._cb.setIcon(QIcon(px))
 
+    def _on_checkbox_clicked(self) -> None:
+        """Checkbox slot — re-emit clicked with this row's sensor id."""
+        self.clicked.emit(self.sensor.id)
+
     def update_value(self, value: float | None):
         """Update the displayed value."""
         if value is None: