Compare commits
3 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| e3b20089f0 | |||
| 54c0971ac3 | |||
| 9ae2e22b44 |
@@ -5,6 +5,31 @@ All notable changes to RigDoctor are recorded here. Format follows
|
|||||||
(`MAJOR.MINOR.PATCH`, pre-1.0). `__version__` and `pyproject.toml` must match the git
|
(`MAJOR.MINOR.PATCH`, pre-1.0). `__version__` and `pyproject.toml` must match the git
|
||||||
release tag (so the auto-updater, D18, can compare versions).
|
release tag (so the auto-updater, D18, can compare versions).
|
||||||
|
|
||||||
|
## [0.4.0] - 2026-05-21
|
||||||
|
### Added
|
||||||
|
- **Alerts (M8)**: desktop notifications (via `notify-send`) for **overheat** (GPU/CPU past a
|
||||||
|
threshold), **GPU-lost** (nvidia-smi timeout), and a **new version available** (fired once
|
||||||
|
per version). Edge-triggered with a cooldown so it doesn't spam. Degrades gracefully if
|
||||||
|
`notify-send` isn't installed.
|
||||||
|
- **Notifications page**: configure alerts (enable/disable, GPU/CPU temperature thresholds)
|
||||||
|
with a "Send test" button; changes apply live and persist to `config.toml`.
|
||||||
|
- **App icon**: ships a RigDoctor icon and shows it in the dock/launcher. The GUI
|
||||||
|
**self-registers** the icon + `.desktop` on launch (and sets the Wayland app-id), so a
|
||||||
|
self-update + relaunch picks it up — no need to re-run the installer.
|
||||||
|
|
||||||
|
## [0.3.2] - 2026-05-21
|
||||||
|
### Changed
|
||||||
|
- Replaced the per-page "Run with admin" buttons with a **single password prompt at launch**
|
||||||
|
(`pkexec`): the GUI collects root-only data (SMART + dmidecode board/BIOS/RAM) once and
|
||||||
|
caches it for the session, so Health and Inventory always show the full picture. Falls back
|
||||||
|
to non-root if cancelled/unavailable; disable via `elevate_on_launch = false`.
|
||||||
|
|
||||||
|
## [0.3.1] - 2026-05-21
|
||||||
|
### Fixed
|
||||||
|
- Changelog/release notes now **render Markdown** instead of showing raw `#`/`**` markup —
|
||||||
|
the in-app changelog uses `QTextEdit.setMarkdown()` and the update prompt renders notes as
|
||||||
|
rich text (closes #1).
|
||||||
|
|
||||||
## [0.3.0] - 2026-05-21
|
## [0.3.0] - 2026-05-21
|
||||||
### Added
|
### Added
|
||||||
- **System inventory (M5)**: CPU, GPU (model/driver/VBIOS/VRAM/PCIe), motherboard/BIOS, RAM
|
- **System inventory (M5)**: CPU, GPU (model/driver/VBIOS/VRAM/PCIe), motherboard/BIOS, RAM
|
||||||
|
|||||||
+1
-1
@@ -12,7 +12,7 @@ Status: ⬜ not started · 🟦 designing · 🟨 in progress · ✅ done
|
|||||||
| M3 | Crash-capture logger | Essential | none (opt: smartmontools) | all (NVIDIA first) | P0 | 🟨 |
|
| M3 | Crash-capture logger | Essential | none (opt: smartmontools) | all (NVIDIA first) | P0 | 🟨 |
|
||||||
| M4 | Health report (log scan) | Essential | none (opt: smartmontools) | all (NVIDIA first) | P0 | 🟨 |
|
| M4 | Health report (log scan) | Essential | none (opt: smartmontools) | all (NVIDIA first) | P0 | 🟨 |
|
||||||
| M2 | Live monitor (TUI) | Monitoring | none (stdlib curses) | all | P1 | ⬜ |
|
| M2 | Live monitor (TUI) | Monitoring | none (stdlib curses) | all | P1 | ⬜ |
|
||||||
| M8 | Alerting | Monitoring | libnotify (opt) | all | P2 | ⬜ |
|
| M8 | Alerting | Monitoring | libnotify (opt) | all | P2 | 🟨 |
|
||||||
| M5 | System inventory | Diagnostics | none (opt: lm-sensors, dmidecode) | all | P1 | 🟨 |
|
| M5 | System inventory | Diagnostics | none (opt: lm-sensors, dmidecode) | all | P1 | 🟨 |
|
||||||
| M6 | Gaming env checks | Diagnostics | none | all | P2 | ⬜ |
|
| M6 | Gaming env checks | Diagnostics | none | all | P2 | ⬜ |
|
||||||
| M10 | Desktop GUI | Desktop UI | **python3-pyside6** | all | P2 | 🟨 |
|
| M10 | Desktop GUI | Desktop UI | **python3-pyside6** | all | P2 | 🟨 |
|
||||||
|
|||||||
+16
-2
@@ -16,7 +16,8 @@ SCRIPT_DIR=$(CDPATH= cd -- "$(dirname -- "$0")" && pwd)
|
|||||||
uninstall() {
|
uninstall() {
|
||||||
echo "Removing RigDoctor user-local install…"
|
echo "Removing RigDoctor user-local install…"
|
||||||
rm -rf "$VENV"
|
rm -rf "$VENV"
|
||||||
rm -f "$BIN_DIR/rigdoctor" "$BIN_DIR/rigdoctor-gui" "$DESKTOP_FILE"
|
rm -f "$BIN_DIR/rigdoctor" "$BIN_DIR/rigdoctor-gui" "$DESKTOP_FILE" \
|
||||||
|
"$DATA_HOME/icons/hicolor/scalable/apps/rigdoctor.svg"
|
||||||
echo "Done. (Config and logs under ~/.config/rigdoctor and ~/.local/share/rigdoctor were kept.)"
|
echo "Done. (Config and logs under ~/.config/rigdoctor and ~/.local/share/rigdoctor were kept.)"
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -81,6 +82,17 @@ mkdir -p "$BIN_DIR"
|
|||||||
ln -sf "$VENV/bin/rigdoctor" "$BIN_DIR/rigdoctor"
|
ln -sf "$VENV/bin/rigdoctor" "$BIN_DIR/rigdoctor"
|
||||||
ln -sf "$VENV/bin/rigdoctor-gui" "$BIN_DIR/rigdoctor-gui"
|
ln -sf "$VENV/bin/rigdoctor-gui" "$BIN_DIR/rigdoctor-gui"
|
||||||
|
|
||||||
|
# Install the app icon (for the dock/launcher); fall back to a stock icon.
|
||||||
|
ICON_NAME=utilities-system-monitor
|
||||||
|
ICON_SRC=$("$VENV/bin/python" -c "import os, rigdoctor.gui as g; print(os.path.join(os.path.dirname(g.__file__), 'assets', 'rigdoctor.svg'))" 2>/dev/null || true)
|
||||||
|
if [ -n "$ICON_SRC" ] && [ -f "$ICON_SRC" ]; then
|
||||||
|
ICON_DST="$DATA_HOME/icons/hicolor/scalable/apps/rigdoctor.svg"
|
||||||
|
mkdir -p "$(dirname "$ICON_DST")"
|
||||||
|
cp "$ICON_SRC" "$ICON_DST"
|
||||||
|
ICON_NAME=rigdoctor
|
||||||
|
command -v gtk-update-icon-cache >/dev/null 2>&1 && gtk-update-icon-cache -qtf "$DATA_HOME/icons/hicolor" 2>/dev/null || true
|
||||||
|
fi
|
||||||
|
|
||||||
mkdir -p "$DESKTOP_DIR"
|
mkdir -p "$DESKTOP_DIR"
|
||||||
cat > "$DESKTOP_FILE" <<EOF
|
cat > "$DESKTOP_FILE" <<EOF
|
||||||
[Desktop Entry]
|
[Desktop Entry]
|
||||||
@@ -88,10 +100,12 @@ Type=Application
|
|||||||
Name=RigDoctor
|
Name=RigDoctor
|
||||||
Comment=Hardware monitoring & crash diagnostics for Linux gamers
|
Comment=Hardware monitoring & crash diagnostics for Linux gamers
|
||||||
Exec=$VENV/bin/rigdoctor-gui
|
Exec=$VENV/bin/rigdoctor-gui
|
||||||
Icon=utilities-system-monitor
|
Icon=$ICON_NAME
|
||||||
Terminal=false
|
Terminal=false
|
||||||
Categories=System;Monitor;Utility;
|
Categories=System;Monitor;Utility;
|
||||||
|
StartupWMClass=rigdoctor
|
||||||
EOF
|
EOF
|
||||||
|
command -v update-desktop-database >/dev/null 2>&1 && update-desktop-database "$DESKTOP_DIR" 2>/dev/null || true
|
||||||
|
|
||||||
echo
|
echo
|
||||||
echo "RigDoctor $("$VENV/bin/rigdoctor" --version 2>/dev/null | awk '{print $2}') installed."
|
echo "RigDoctor $("$VENV/bin/rigdoctor" --version 2>/dev/null | awk '{print $2}') installed."
|
||||||
|
|||||||
+4
-1
@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
|
|||||||
|
|
||||||
[project]
|
[project]
|
||||||
name = "rigdoctor"
|
name = "rigdoctor"
|
||||||
version = "0.3.0"
|
version = "0.4.0"
|
||||||
description = "Modular hardware monitoring & crash diagnostics for Linux gamers."
|
description = "Modular hardware monitoring & crash diagnostics for Linux gamers."
|
||||||
readme = "README.md"
|
readme = "README.md"
|
||||||
requires-python = ">=3.11"
|
requires-python = ">=3.11"
|
||||||
@@ -21,3 +21,6 @@ rigdoctor-gui = "rigdoctor.gui.app:main"
|
|||||||
|
|
||||||
[tool.setuptools.packages.find]
|
[tool.setuptools.packages.find]
|
||||||
where = ["src"]
|
where = ["src"]
|
||||||
|
|
||||||
|
[tool.setuptools.package-data]
|
||||||
|
rigdoctor = ["gui/assets/*.svg"]
|
||||||
|
|||||||
@@ -1,3 +1,3 @@
|
|||||||
"""RigDoctor — modular hardware monitoring & crash diagnostics for Linux gamers."""
|
"""RigDoctor — modular hardware monitoring & crash diagnostics for Linux gamers."""
|
||||||
|
|
||||||
__version__ = "0.3.0"
|
__version__ = "0.4.0"
|
||||||
|
|||||||
@@ -295,6 +295,18 @@ def cmd_uninstall(args) -> int:
|
|||||||
return 0
|
return 0
|
||||||
|
|
||||||
|
|
||||||
|
def cmd_collect_priv(args) -> int:
|
||||||
|
"""Internal: emit root-only data (SMART + dmidecode) as JSON, run via pkexec at launch."""
|
||||||
|
from dataclasses import asdict
|
||||||
|
|
||||||
|
from .core.health import check_smart
|
||||||
|
from .core.inventory import _dmidecode
|
||||||
|
|
||||||
|
data = {"smart": [asdict(f) for f in check_smart()], "dmidecode": _dmidecode()}
|
||||||
|
print(json.dumps(data))
|
||||||
|
return 0
|
||||||
|
|
||||||
|
|
||||||
def cmd_inventory(args) -> int:
|
def cmd_inventory(args) -> int:
|
||||||
from .core import inventory
|
from .core import inventory
|
||||||
|
|
||||||
@@ -390,6 +402,9 @@ def build_parser() -> argparse.ArgumentParser:
|
|||||||
rep.add_argument("--json", action="store_true", help="output JSON instead of text")
|
rep.add_argument("--json", action="store_true", help="output JSON instead of text")
|
||||||
rep.set_defaults(func=cmd_report)
|
rep.set_defaults(func=cmd_report)
|
||||||
|
|
||||||
|
cp = sub.add_parser("collect-priv", help=argparse.SUPPRESS) # internal: run via pkexec
|
||||||
|
cp.set_defaults(func=cmd_collect_priv)
|
||||||
|
|
||||||
inv = sub.add_parser("inventory", help="system inventory (M5): export hardware/OS details")
|
inv = sub.add_parser("inventory", help="system inventory (M5): export hardware/OS details")
|
||||||
inv.add_argument("--json", action="store_true", help="output JSON")
|
inv.add_argument("--json", action="store_true", help="output JSON")
|
||||||
inv.add_argument("--markdown", action="store_true", help="output Markdown (for forum/bug reports)")
|
inv.add_argument("--markdown", action="store_true", help="output Markdown (for forum/bug reports)")
|
||||||
|
|||||||
@@ -138,6 +138,10 @@ DEFAULTS: dict = {
|
|||||||
"log_max_bytes": 20_000_000, # rotate a log segment past this size
|
"log_max_bytes": 20_000_000, # rotate a log segment past this size
|
||||||
"log_backups": 10, # keep this many rotated segments (bounds disk use)
|
"log_backups": 10, # keep this many rotated segments (bounds disk use)
|
||||||
"update_check_minutes": 30, # re-check for updates this often while running (0 = off)
|
"update_check_minutes": 30, # re-check for updates this often while running (0 = off)
|
||||||
|
"elevate_on_launch": True, # GUI asks for the password once at launch (SMART/dmidecode)
|
||||||
|
"alerts_enabled": True, # desktop notifications on overheat / GPU-lost / new version
|
||||||
|
"gpu_temp_alert": 90.0, # °C — alert when GPU reaches this
|
||||||
|
"cpu_temp_alert": 95.0, # °C — alert when CPU reaches this
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
@@ -153,3 +157,27 @@ def load_config() -> dict:
|
|||||||
except Exception:
|
except Exception:
|
||||||
pass
|
pass
|
||||||
return cfg
|
return cfg
|
||||||
|
|
||||||
|
|
||||||
|
def _toml_value(value) -> str:
|
||||||
|
if isinstance(value, bool):
|
||||||
|
return "true" if value else "false"
|
||||||
|
if isinstance(value, (int, float)):
|
||||||
|
return repr(value)
|
||||||
|
return '"' + str(value).replace("\\", "\\\\").replace('"', '\\"') + '"'
|
||||||
|
|
||||||
|
|
||||||
|
def save_config(values: dict) -> None:
|
||||||
|
"""Write a flat config.toml (stdlib has no TOML writer)."""
|
||||||
|
CONFIG_DIR.mkdir(parents=True, exist_ok=True)
|
||||||
|
lines = ["# RigDoctor config — edit in the app (Notifications) or here."]
|
||||||
|
lines += [f"{key} = {_toml_value(value)}" for key, value in values.items()]
|
||||||
|
CONFIG_FILE.write_text("\n".join(lines) + "\n")
|
||||||
|
|
||||||
|
|
||||||
|
def update_config(**changes) -> dict:
|
||||||
|
"""Merge changes into the current effective config and persist them."""
|
||||||
|
cfg = load_config()
|
||||||
|
cfg.update(changes)
|
||||||
|
save_config(cfg)
|
||||||
|
return cfg
|
||||||
|
|||||||
@@ -0,0 +1,91 @@
|
|||||||
|
"""Desktop alerts (M8): notify on overheat / GPU-lost / new version via notify-send.
|
||||||
|
|
||||||
|
Edge-triggered: an alert fires when a condition becomes true (not every sample), and
|
||||||
|
can fire again only after it has cleared and a cooldown has passed — so a hot GPU or a
|
||||||
|
1-Hz sample loop doesn't spam notifications. Degrades to a no-op if notify-send is absent.
|
||||||
|
"""
|
||||||
|
|
||||||
|
from __future__ import annotations
|
||||||
|
|
||||||
|
import shutil
|
||||||
|
import subprocess
|
||||||
|
import time
|
||||||
|
|
||||||
|
from .sample import Sample
|
||||||
|
|
||||||
|
APP_NAME = "RigDoctor"
|
||||||
|
_ICON = "utilities-system-monitor"
|
||||||
|
|
||||||
|
|
||||||
|
def available() -> bool:
|
||||||
|
return shutil.which("notify-send") is not None
|
||||||
|
|
||||||
|
|
||||||
|
def notify(title: str, message: str, urgency: str = "normal") -> bool:
|
||||||
|
"""Send a desktop notification (best-effort). urgency: low|normal|critical."""
|
||||||
|
if not available():
|
||||||
|
return False
|
||||||
|
try:
|
||||||
|
subprocess.run(
|
||||||
|
["notify-send", "-a", APP_NAME, "-u", urgency, "-i", _ICON, title, message],
|
||||||
|
timeout=10,
|
||||||
|
check=False,
|
||||||
|
)
|
||||||
|
return True
|
||||||
|
except (subprocess.SubprocessError, OSError):
|
||||||
|
return False
|
||||||
|
|
||||||
|
|
||||||
|
class AlertMonitor:
|
||||||
|
"""Evaluate samples and raise edge-triggered desktop alerts."""
|
||||||
|
|
||||||
|
def __init__(self, gpu_temp: float = 90.0, cpu_temp: float = 95.0, cooldown: float = 300.0):
|
||||||
|
self.gpu_temp = gpu_temp
|
||||||
|
self.cpu_temp = cpu_temp
|
||||||
|
self.cooldown = cooldown
|
||||||
|
self.enabled = True
|
||||||
|
self._active: dict[str, bool] = {}
|
||||||
|
self._last: dict[str, float] = {}
|
||||||
|
|
||||||
|
def _fire(self, key: str, title: str, message: str, urgency: str = "critical") -> None:
|
||||||
|
if self._active.get(key):
|
||||||
|
return # already alerting; wait until it clears
|
||||||
|
now = time.time()
|
||||||
|
if now - self._last.get(key, 0.0) < self.cooldown:
|
||||||
|
return
|
||||||
|
self._active[key] = True
|
||||||
|
self._last[key] = now
|
||||||
|
notify(title, message, urgency)
|
||||||
|
|
||||||
|
def _clear(self, key: str) -> None:
|
||||||
|
self._active[key] = False
|
||||||
|
|
||||||
|
def check(self, sample: Sample) -> None:
|
||||||
|
if not self.enabled:
|
||||||
|
return
|
||||||
|
gpu_t = next(
|
||||||
|
(r.value for r in sample.readings
|
||||||
|
if r.source == "gpu" and r.metric == "temp" and r.label == "" and r.value is not None),
|
||||||
|
None,
|
||||||
|
)
|
||||||
|
if gpu_t is not None:
|
||||||
|
if gpu_t >= self.gpu_temp:
|
||||||
|
self._fire("gpu_temp", "GPU overheating", f"GPU at {gpu_t:.0f} °C")
|
||||||
|
else:
|
||||||
|
self._clear("gpu_temp")
|
||||||
|
|
||||||
|
cpu_temps = [r.value for r in sample.readings
|
||||||
|
if r.source == "cpu" and r.metric == "temp" and r.value is not None]
|
||||||
|
if cpu_temps:
|
||||||
|
cpu_t = max(cpu_temps)
|
||||||
|
if cpu_t >= self.cpu_temp:
|
||||||
|
self._fire("cpu_temp", "CPU overheating", f"CPU at {cpu_t:.0f} °C")
|
||||||
|
else:
|
||||||
|
self._clear("cpu_temp")
|
||||||
|
|
||||||
|
lost = any(r.source == "gpu" and r.metric == "status" and r.label == "query-timeout"
|
||||||
|
for r in sample.readings)
|
||||||
|
if lost:
|
||||||
|
self._fire("gpu_lost", "GPU not responding", "nvidia-smi query timed out — the GPU may have dropped")
|
||||||
|
else:
|
||||||
|
self._clear("gpu_lost")
|
||||||
@@ -0,0 +1,51 @@
|
|||||||
|
"""Session privilege elevation.
|
||||||
|
|
||||||
|
At GUI launch the app asks for the password once (pkexec) and collects the data that
|
||||||
|
needs root — SMART health + dmidecode (board/BIOS/RAM) — caching it for the session so
|
||||||
|
Health and Inventory can always show the full picture without per-action prompts.
|
||||||
|
"""
|
||||||
|
|
||||||
|
from __future__ import annotations
|
||||||
|
|
||||||
|
import json
|
||||||
|
import os
|
||||||
|
import shutil
|
||||||
|
import subprocess
|
||||||
|
import sys
|
||||||
|
|
||||||
|
_privileged: dict | None = None
|
||||||
|
|
||||||
|
|
||||||
|
def privileged() -> dict | None:
|
||||||
|
"""Cached root-collected data ({"smart": [...], "dmidecode": {...}}), or None."""
|
||||||
|
return _privileged
|
||||||
|
|
||||||
|
|
||||||
|
def set_privileged(data: dict | None) -> None:
|
||||||
|
global _privileged
|
||||||
|
_privileged = data
|
||||||
|
|
||||||
|
|
||||||
|
def available() -> bool:
|
||||||
|
return shutil.which("pkexec") is not None and os.geteuid() != 0
|
||||||
|
|
||||||
|
|
||||||
|
def _cli() -> list[str]:
|
||||||
|
candidate = os.path.join(os.path.dirname(sys.executable), "rigdoctor")
|
||||||
|
return [candidate] if os.path.exists(candidate) else [sys.executable, "-m", "rigdoctor"]
|
||||||
|
|
||||||
|
|
||||||
|
def collect_via_pkexec(timeout: float = 120.0) -> dict | None:
|
||||||
|
"""Run one elevated collection (single password prompt). None if unavailable/cancelled."""
|
||||||
|
if not available():
|
||||||
|
return None
|
||||||
|
try:
|
||||||
|
proc = subprocess.run(
|
||||||
|
["pkexec", *_cli(), "collect-priv"],
|
||||||
|
capture_output=True, text=True, timeout=timeout,
|
||||||
|
)
|
||||||
|
if proc.returncode == 0 and proc.stdout.strip():
|
||||||
|
return json.loads(proc.stdout)
|
||||||
|
except (subprocess.SubprocessError, OSError, ValueError):
|
||||||
|
pass
|
||||||
|
return None
|
||||||
@@ -234,11 +234,21 @@ def check_live_temps() -> list[Finding]:
|
|||||||
|
|
||||||
|
|
||||||
def run_health_checks() -> list[Finding]:
|
def run_health_checks() -> list[Finding]:
|
||||||
"""Run all checks and return findings sorted by severity (worst first)."""
|
"""Run all checks and return findings sorted by severity (worst first).
|
||||||
|
|
||||||
|
SMART needs root; if the session collected it via launch elevation, use that
|
||||||
|
instead of re-running smartctl (which would just report "needs root").
|
||||||
|
"""
|
||||||
|
from . import elevation
|
||||||
|
|
||||||
findings: list[Finding] = []
|
findings: list[Finding] = []
|
||||||
findings += check_nvidia_driver()
|
findings += check_nvidia_driver()
|
||||||
findings += check_journal()
|
findings += check_journal()
|
||||||
findings += check_journal_persistence()
|
findings += check_journal_persistence()
|
||||||
|
priv = elevation.privileged()
|
||||||
|
if priv is not None and priv.get("smart") is not None:
|
||||||
|
findings += [Finding(**d) for d in priv["smart"]]
|
||||||
|
else:
|
||||||
findings += check_smart()
|
findings += check_smart()
|
||||||
findings += check_live_temps()
|
findings += check_live_temps()
|
||||||
findings.sort(key=lambda f: _ORDER.get(f.severity, 9))
|
findings.sort(key=lambda f: _ORDER.get(f.severity, 9))
|
||||||
|
|||||||
@@ -171,7 +171,10 @@ def _dmidecode() -> dict:
|
|||||||
|
|
||||||
|
|
||||||
def collect() -> list[Section]:
|
def collect() -> list[Section]:
|
||||||
dmi = _dmidecode()
|
from . import elevation
|
||||||
|
|
||||||
|
priv = elevation.privileged()
|
||||||
|
dmi = priv["dmidecode"] if (priv and priv.get("dmidecode") is not None) else _dmidecode()
|
||||||
return [_system(), _cpu(), _firmware(dmi), _memory(dmi), _gpu(), _storage(), _display()]
|
return [_system(), _cpu(), _firmware(dmi), _memory(dmi), _gpu(), _storage(), _display()]
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
@@ -23,6 +23,7 @@ def targets(purge: bool = False) -> list[Path]:
|
|||||||
home / ".local" / "bin" / "rigdoctor",
|
home / ".local" / "bin" / "rigdoctor",
|
||||||
home / ".local" / "bin" / "rigdoctor-gui",
|
home / ".local" / "bin" / "rigdoctor-gui",
|
||||||
share / "applications" / "rigdoctor.desktop",
|
share / "applications" / "rigdoctor.desktop",
|
||||||
|
share / "icons" / "hicolor" / "scalable" / "apps" / "rigdoctor.svg",
|
||||||
]
|
]
|
||||||
if purge:
|
if purge:
|
||||||
items += [config.CONFIG_DIR, config.STATE_DIR, config.DATA_DIR]
|
items += [config.CONFIG_DIR, config.STATE_DIR, config.DATA_DIR]
|
||||||
|
|||||||
@@ -3,18 +3,28 @@
|
|||||||
from __future__ import annotations
|
from __future__ import annotations
|
||||||
|
|
||||||
import sys
|
import sys
|
||||||
|
from pathlib import Path
|
||||||
|
|
||||||
|
from PySide6.QtGui import QIcon
|
||||||
from PySide6.QtWidgets import QApplication
|
from PySide6.QtWidgets import QApplication
|
||||||
|
|
||||||
from ..config import load_config
|
from ..config import load_config
|
||||||
|
from . import desktop
|
||||||
from .main_window import MainWindow
|
from .main_window import MainWindow
|
||||||
from .theme import STYLESHEET
|
from .theme import STYLESHEET
|
||||||
|
|
||||||
|
ICON = Path(__file__).parent / "assets" / "rigdoctor.svg"
|
||||||
|
|
||||||
|
|
||||||
def main(argv: list[str] | None = None) -> int:
|
def main(argv: list[str] | None = None) -> int:
|
||||||
|
desktop.ensure() # self-register icon + .desktop so updates show it without re-installing
|
||||||
app = QApplication(argv if argv is not None else sys.argv)
|
app = QApplication(argv if argv is not None else sys.argv)
|
||||||
app.setApplicationName("RigDoctor")
|
app.setApplicationName("RigDoctor")
|
||||||
app.setApplicationDisplayName("RigDoctor")
|
app.setApplicationDisplayName("RigDoctor")
|
||||||
|
# Match the installed rigdoctor.desktop so the dock/launcher shows our icon (Wayland app-id).
|
||||||
|
app.setDesktopFileName("rigdoctor")
|
||||||
|
if ICON.exists():
|
||||||
|
app.setWindowIcon(QIcon(str(ICON)))
|
||||||
app.setStyle("Fusion")
|
app.setStyle("Fusion")
|
||||||
app.setStyleSheet(STYLESHEET)
|
app.setStyleSheet(STYLESHEET)
|
||||||
|
|
||||||
|
|||||||
@@ -0,0 +1,8 @@
|
|||||||
|
<svg xmlns="http://www.w3.org/2000/svg" width="256" height="256" viewBox="0 0 256 256">
|
||||||
|
<rect x="8" y="8" width="240" height="240" rx="52" fill="#15181e"/>
|
||||||
|
<circle cx="128" cy="128" r="84" fill="none" stroke="#2a2f39" stroke-width="14"/>
|
||||||
|
<path d="M128 44 a84 84 0 1 1 -59.4 24.6" fill="none" stroke="#38bdf8"
|
||||||
|
stroke-width="14" stroke-linecap="round"/>
|
||||||
|
<path d="M60 132 H100 L116 96 L140 168 L156 132 H196" fill="none" stroke="#e6e8eb"
|
||||||
|
stroke-width="14" stroke-linecap="round" stroke-linejoin="round"/>
|
||||||
|
</svg>
|
||||||
|
After Width: | Height: | Size: 533 B |
@@ -0,0 +1,51 @@
|
|||||||
|
"""Best-effort desktop integration: install our icon + .desktop so the dock shows it.
|
||||||
|
|
||||||
|
Runs at GUI launch (idempotent), so a self-update + relaunch refreshes the icon without
|
||||||
|
re-running install.sh. No-op for non-installed (dev) runs where the launcher is absent.
|
||||||
|
"""
|
||||||
|
|
||||||
|
from __future__ import annotations
|
||||||
|
|
||||||
|
import shutil
|
||||||
|
import sys
|
||||||
|
from pathlib import Path
|
||||||
|
|
||||||
|
from .. import config
|
||||||
|
|
||||||
|
_ICON_SRC = Path(__file__).parent / "assets" / "rigdoctor.svg"
|
||||||
|
|
||||||
|
_DESKTOP = """[Desktop Entry]
|
||||||
|
Type=Application
|
||||||
|
Name=RigDoctor
|
||||||
|
Comment=Hardware monitoring & crash diagnostics for Linux gamers
|
||||||
|
Exec={exec}
|
||||||
|
Icon=rigdoctor
|
||||||
|
Terminal=false
|
||||||
|
Categories=System;Monitor;Utility;
|
||||||
|
StartupWMClass=rigdoctor
|
||||||
|
"""
|
||||||
|
|
||||||
|
|
||||||
|
def ensure() -> None:
|
||||||
|
share = config.DATA_DIR.parent # ~/.local/share
|
||||||
|
|
||||||
|
try:
|
||||||
|
if _ICON_SRC.exists():
|
||||||
|
icon_dst = share / "icons" / "hicolor" / "scalable" / "apps" / "rigdoctor.svg"
|
||||||
|
icon_dst.parent.mkdir(parents=True, exist_ok=True)
|
||||||
|
if not icon_dst.exists() or icon_dst.read_bytes() != _ICON_SRC.read_bytes():
|
||||||
|
shutil.copyfile(_ICON_SRC, icon_dst)
|
||||||
|
except OSError:
|
||||||
|
pass
|
||||||
|
|
||||||
|
gui_exec = Path(sys.executable).with_name("rigdoctor-gui")
|
||||||
|
if not gui_exec.exists(): # dev / not a normal install — don't fabricate a .desktop
|
||||||
|
return
|
||||||
|
try:
|
||||||
|
desktop = share / "applications" / "rigdoctor.desktop"
|
||||||
|
content = _DESKTOP.format(exec=gui_exec)
|
||||||
|
desktop.parent.mkdir(parents=True, exist_ok=True)
|
||||||
|
if not desktop.exists() or desktop.read_text() != content:
|
||||||
|
desktop.write_text(content)
|
||||||
|
except OSError:
|
||||||
|
pass
|
||||||
@@ -2,11 +2,6 @@
|
|||||||
|
|
||||||
from __future__ import annotations
|
from __future__ import annotations
|
||||||
|
|
||||||
import json
|
|
||||||
import os
|
|
||||||
import shutil
|
|
||||||
import subprocess
|
|
||||||
import sys
|
|
||||||
import threading
|
import threading
|
||||||
import time
|
import time
|
||||||
|
|
||||||
@@ -77,11 +72,6 @@ class HealthPage(QWidget):
|
|||||||
self._status = QLabel("")
|
self._status = QLabel("")
|
||||||
self._status.setObjectName("Muted")
|
self._status.setObjectName("Muted")
|
||||||
header.addWidget(self._status)
|
header.addWidget(self._status)
|
||||||
self._admin_btn = QPushButton("Run with admin")
|
|
||||||
self._admin_btn.setToolTip("Run all checks with root (SMART needs it) — prompts for your password")
|
|
||||||
self._admin_btn.clicked.connect(self._run_admin)
|
|
||||||
self._admin_btn.setEnabled(shutil.which("pkexec") is not None)
|
|
||||||
header.addWidget(self._admin_btn)
|
|
||||||
self._run_btn = QPushButton("Run health report")
|
self._run_btn = QPushButton("Run health report")
|
||||||
self._run_btn.setObjectName("PrimaryButton")
|
self._run_btn.setObjectName("PrimaryButton")
|
||||||
self._run_btn.clicked.connect(self._run)
|
self._run_btn.clicked.connect(self._run)
|
||||||
@@ -116,32 +106,10 @@ class HealthPage(QWidget):
|
|||||||
findings = []
|
findings = []
|
||||||
self._result.emit(findings)
|
self._result.emit(findings)
|
||||||
|
|
||||||
def _run_admin(self) -> None:
|
|
||||||
self._run_btn.setEnabled(False)
|
|
||||||
self._admin_btn.setEnabled(False)
|
|
||||||
self._status.setText("Running all checks with admin (you'll be prompted)…")
|
|
||||||
threading.Thread(target=self._work_admin, daemon=True).start()
|
|
||||||
|
|
||||||
def _work_admin(self) -> None:
|
|
||||||
from ..core.health import Finding
|
|
||||||
|
|
||||||
cli = os.path.join(os.path.dirname(sys.executable), "rigdoctor")
|
|
||||||
if os.path.exists(cli):
|
|
||||||
cmd = ["pkexec", cli, "report", "--json"]
|
|
||||||
else: # dev / not on PATH next to python
|
|
||||||
cmd = ["pkexec", sys.executable, "-m", "rigdoctor", "report", "--json"]
|
|
||||||
try:
|
|
||||||
proc = subprocess.run(cmd, capture_output=True, text=True, timeout=180)
|
|
||||||
findings = [Finding(**d) for d in json.loads(proc.stdout)] if proc.returncode == 0 else None
|
|
||||||
except Exception:
|
|
||||||
findings = None # pkexec cancelled / failed / unparsable
|
|
||||||
self._result.emit(findings)
|
|
||||||
|
|
||||||
def _render_findings(self, findings) -> None:
|
def _render_findings(self, findings) -> None:
|
||||||
self._run_btn.setEnabled(True)
|
self._run_btn.setEnabled(True)
|
||||||
self._admin_btn.setEnabled(shutil.which("pkexec") is not None)
|
if findings is None: # collection failed — keep current results
|
||||||
if findings is None: # elevated run cancelled/failed — keep current results
|
self._status.setText("check failed")
|
||||||
self._status.setText("admin run cancelled")
|
|
||||||
return
|
return
|
||||||
|
|
||||||
while self._list.count():
|
while self._list.count():
|
||||||
|
|||||||
@@ -2,11 +2,7 @@
|
|||||||
|
|
||||||
from __future__ import annotations
|
from __future__ import annotations
|
||||||
|
|
||||||
import json
|
|
||||||
import os
|
import os
|
||||||
import shutil
|
|
||||||
import subprocess
|
|
||||||
import sys
|
|
||||||
import threading
|
import threading
|
||||||
|
|
||||||
from PySide6.QtCore import Qt, QTimer, Signal
|
from PySide6.QtCore import Qt, QTimer, Signal
|
||||||
@@ -73,11 +69,6 @@ class InventoryPage(QWidget):
|
|||||||
self._status = QLabel("")
|
self._status = QLabel("")
|
||||||
self._status.setObjectName("Muted")
|
self._status.setObjectName("Muted")
|
||||||
header.addWidget(self._status)
|
header.addWidget(self._status)
|
||||||
self._admin_btn = QPushButton("Run with admin")
|
|
||||||
self._admin_btn.setToolTip("Re-collect with root for motherboard/BIOS/RAM details (dmidecode)")
|
|
||||||
self._admin_btn.setEnabled(shutil.which("pkexec") is not None)
|
|
||||||
self._admin_btn.clicked.connect(self._run_admin)
|
|
||||||
header.addWidget(self._admin_btn)
|
|
||||||
self._copy_btn = QPushButton("Copy Markdown")
|
self._copy_btn = QPushButton("Copy Markdown")
|
||||||
self._copy_btn.clicked.connect(self._copy)
|
self._copy_btn.clicked.connect(self._copy)
|
||||||
header.addWidget(self._copy_btn)
|
header.addWidget(self._copy_btn)
|
||||||
@@ -115,32 +106,17 @@ class InventoryPage(QWidget):
|
|||||||
sections = []
|
sections = []
|
||||||
self._result.emit(sections)
|
self._result.emit(sections)
|
||||||
|
|
||||||
def _run_admin(self) -> None:
|
|
||||||
self._busy("Collecting with admin (you'll be prompted)…")
|
|
||||||
threading.Thread(target=self._work_admin, daemon=True).start()
|
|
||||||
|
|
||||||
def _work_admin(self) -> None:
|
|
||||||
cli = os.path.join(os.path.dirname(sys.executable), "rigdoctor")
|
|
||||||
cmd = [cli, "inventory", "--json"] if os.path.exists(cli) else [sys.executable, "-m", "rigdoctor", "inventory", "--json"]
|
|
||||||
try:
|
|
||||||
proc = subprocess.run(["pkexec", *cmd], capture_output=True, text=True, timeout=120)
|
|
||||||
sections = inventory.from_dict(json.loads(proc.stdout)) if proc.returncode == 0 else None
|
|
||||||
except Exception:
|
|
||||||
sections = None
|
|
||||||
self._result.emit(sections)
|
|
||||||
|
|
||||||
def _busy(self, text: str) -> None:
|
def _busy(self, text: str) -> None:
|
||||||
self._status.setText(text)
|
self._status.setText(text)
|
||||||
for b in (self._refresh_btn, self._admin_btn, self._copy_btn, self._save_btn):
|
for b in (self._refresh_btn, self._copy_btn, self._save_btn):
|
||||||
b.setEnabled(False)
|
b.setEnabled(False)
|
||||||
|
|
||||||
def _render(self, sections) -> None:
|
def _render(self, sections) -> None:
|
||||||
self._refresh_btn.setEnabled(True)
|
self._refresh_btn.setEnabled(True)
|
||||||
self._admin_btn.setEnabled(shutil.which("pkexec") is not None)
|
|
||||||
self._copy_btn.setEnabled(True)
|
self._copy_btn.setEnabled(True)
|
||||||
self._save_btn.setEnabled(True)
|
self._save_btn.setEnabled(True)
|
||||||
if sections is None: # admin run cancelled/failed — keep current
|
if sections is None: # collection failed — keep current
|
||||||
self._status.setText("admin run cancelled")
|
self._status.setText("collection failed")
|
||||||
return
|
return
|
||||||
|
|
||||||
self._sections = sections
|
self._sections = sections
|
||||||
|
|||||||
@@ -7,6 +7,7 @@ import sys
|
|||||||
import threading
|
import threading
|
||||||
|
|
||||||
from PySide6.QtCore import Qt, QProcess, QTimer, Signal
|
from PySide6.QtCore import Qt, QProcess, QTimer, Signal
|
||||||
|
from PySide6.QtGui import QTextDocument
|
||||||
from PySide6.QtWidgets import (
|
from PySide6.QtWidgets import (
|
||||||
QApplication,
|
QApplication,
|
||||||
QButtonGroup,
|
QButtonGroup,
|
||||||
@@ -25,27 +26,30 @@ from PySide6.QtWidgets import (
|
|||||||
|
|
||||||
from .. import __version__
|
from .. import __version__
|
||||||
from ..config import load_config
|
from ..config import load_config
|
||||||
from ..core import updates
|
from ..core import alerts, elevation, updates
|
||||||
from .dashboard import Dashboard
|
from .dashboard import Dashboard
|
||||||
from .health_page import HealthPage
|
from .health_page import HealthPage
|
||||||
from .inventory_page import InventoryPage
|
from .inventory_page import InventoryPage
|
||||||
|
from .notifications_page import NotificationsPage
|
||||||
from .recorder_page import RecorderPage
|
from .recorder_page import RecorderPage
|
||||||
from .setup_page import SetupPage
|
from .setup_page import SetupPage
|
||||||
from .theme import ACCENT, GOOD, MUTED
|
from .theme import ACCENT, GOOD, MUTED
|
||||||
from .worker import SamplerWorker
|
from .worker import SamplerWorker
|
||||||
|
|
||||||
_NAV_ITEMS = ["Dashboard", "Logs", "Health", "Setup", "Inventory"]
|
_NAV_ITEMS = ["Dashboard", "Logs", "Health", "Setup", "Inventory", "Notifications"]
|
||||||
|
|
||||||
|
|
||||||
class MainWindow(QMainWindow):
|
class MainWindow(QMainWindow):
|
||||||
_update_checked = Signal(object) # (state, tag, notes)
|
_update_checked = Signal(object) # (state, tag, notes)
|
||||||
_update_applied = Signal(int) # pip exit code
|
_update_applied = Signal(int) # pip exit code
|
||||||
_changelog_ready = Signal(object) # ([(tag, date, notes)], error)
|
_changelog_ready = Signal(object) # ([(tag, date, notes)], error)
|
||||||
|
_elevated = Signal() # privileged data collected at launch
|
||||||
|
|
||||||
def __init__(self, interval: float = 1.0) -> None:
|
def __init__(self, interval: float = 1.0) -> None:
|
||||||
super().__init__()
|
super().__init__()
|
||||||
self.setWindowTitle("RigDoctor")
|
self.setWindowTitle("RigDoctor")
|
||||||
self.resize(1000, 680)
|
self.resize(1000, 680)
|
||||||
|
cfg = load_config()
|
||||||
|
|
||||||
central = QWidget()
|
central = QWidget()
|
||||||
self.setCentralWidget(central)
|
self.setCentralWidget(central)
|
||||||
@@ -64,11 +68,14 @@ class MainWindow(QMainWindow):
|
|||||||
self.health_page = HealthPage()
|
self.health_page = HealthPage()
|
||||||
self.setup_page = SetupPage()
|
self.setup_page = SetupPage()
|
||||||
self.inventory_page = InventoryPage()
|
self.inventory_page = InventoryPage()
|
||||||
|
self.notifications_page = NotificationsPage()
|
||||||
|
self.notifications_page.changed.connect(self._apply_alert_settings)
|
||||||
self._stack.addWidget(self.dashboard) # 0 Dashboard
|
self._stack.addWidget(self.dashboard) # 0 Dashboard
|
||||||
self._stack.addWidget(self.recorder_page) # 1 Logs
|
self._stack.addWidget(self.recorder_page) # 1 Logs
|
||||||
self._stack.addWidget(self.health_page) # 2 Health
|
self._stack.addWidget(self.health_page) # 2 Health
|
||||||
self._stack.addWidget(self.setup_page) # 3 Setup
|
self._stack.addWidget(self.setup_page) # 3 Setup
|
||||||
self._stack.addWidget(self.inventory_page) # 4 Inventory
|
self._stack.addWidget(self.inventory_page) # 4 Inventory
|
||||||
|
self._stack.addWidget(self.notifications_page) # 5 Notifications
|
||||||
content_layout.addWidget(self._stack)
|
content_layout.addWidget(self._stack)
|
||||||
|
|
||||||
layout.addWidget(self._build_sidebar())
|
layout.addWidget(self._build_sidebar())
|
||||||
@@ -76,8 +83,24 @@ class MainWindow(QMainWindow):
|
|||||||
|
|
||||||
self._worker = SamplerWorker(interval=interval)
|
self._worker = SamplerWorker(interval=interval)
|
||||||
self._worker.sampled.connect(self.dashboard.update_sample)
|
self._worker.sampled.connect(self.dashboard.update_sample)
|
||||||
|
# Desktop alerts (M8): overheat / GPU-lost from the sample stream, new-version below.
|
||||||
|
# Configurable on the Notifications page; gated by AlertMonitor.enabled.
|
||||||
|
self._notified_update_tag = None
|
||||||
|
self._alert_monitor = alerts.AlertMonitor(
|
||||||
|
gpu_temp=float(cfg.get("gpu_temp_alert", 90.0)),
|
||||||
|
cpu_temp=float(cfg.get("cpu_temp_alert", 95.0)),
|
||||||
|
)
|
||||||
|
self._alert_monitor.enabled = bool(cfg.get("alerts_enabled", True))
|
||||||
|
self._worker.sampled.connect(self._alert_monitor.check)
|
||||||
self._worker.start()
|
self._worker.start()
|
||||||
|
|
||||||
|
# Ask for the password once at launch and collect root-only data (SMART +
|
||||||
|
# dmidecode); Health/Inventory then always show the full picture (config:
|
||||||
|
# elevate_on_launch). Falls back silently to non-root if cancelled/unavailable.
|
||||||
|
if cfg.get("elevate_on_launch", True) and elevation.available():
|
||||||
|
self._elevated.connect(self._on_elevated)
|
||||||
|
threading.Thread(target=self._collect_privileged, daemon=True).start()
|
||||||
|
|
||||||
# Update check (M13): once at launch, then periodically so a newly published
|
# Update check (M13): once at launch, then periodically so a newly published
|
||||||
# release is detected without restarting (interval from config; 0 disables).
|
# release is detected without restarting (interval from config; 0 disables).
|
||||||
self._latest_tag = None
|
self._latest_tag = None
|
||||||
@@ -87,7 +110,7 @@ class MainWindow(QMainWindow):
|
|||||||
self._update_applied.connect(self._on_update_applied)
|
self._update_applied.connect(self._on_update_applied)
|
||||||
self._changelog_ready.connect(self._on_changelog)
|
self._changelog_ready.connect(self._on_changelog)
|
||||||
self._start_update_check()
|
self._start_update_check()
|
||||||
minutes = float(load_config().get("update_check_minutes", 30) or 0)
|
minutes = float(cfg.get("update_check_minutes", 30) or 0)
|
||||||
if minutes > 0:
|
if minutes > 0:
|
||||||
self._update_timer = QTimer(self)
|
self._update_timer = QTimer(self)
|
||||||
self._update_timer.setInterval(int(minutes * 60_000))
|
self._update_timer.setInterval(int(minutes * 60_000))
|
||||||
@@ -171,7 +194,9 @@ class MainWindow(QMainWindow):
|
|||||||
box = QMessageBox(self)
|
box = QMessageBox(self)
|
||||||
box.setWindowTitle(f"Update to {self._latest_tag}")
|
box.setWindowTitle(f"Update to {self._latest_tag}")
|
||||||
box.setText(f"Update RigDoctor to {self._latest_tag}?")
|
box.setText(f"Update RigDoctor to {self._latest_tag}?")
|
||||||
box.setInformativeText(self._latest_notes or "(no release notes)")
|
notes_doc = QTextDocument()
|
||||||
|
notes_doc.setMarkdown(self._latest_notes or "_(no release notes)_")
|
||||||
|
box.setInformativeText(notes_doc.toHtml()) # render Markdown as rich text (#1)
|
||||||
box.setStandardButtons(QMessageBox.StandardButton.Ok | QMessageBox.StandardButton.Cancel)
|
box.setStandardButtons(QMessageBox.StandardButton.Ok | QMessageBox.StandardButton.Cancel)
|
||||||
box.button(QMessageBox.StandardButton.Ok).setText("Update")
|
box.button(QMessageBox.StandardButton.Ok).setText("Update")
|
||||||
if box.exec() != QMessageBox.StandardButton.Ok:
|
if box.exec() != QMessageBox.StandardButton.Ok:
|
||||||
@@ -193,6 +218,23 @@ class MainWindow(QMainWindow):
|
|||||||
self._update_label.setText("update failed")
|
self._update_label.setText("update failed")
|
||||||
self._update_btn.setEnabled(True)
|
self._update_btn.setEnabled(True)
|
||||||
|
|
||||||
|
def _collect_privileged(self) -> None:
|
||||||
|
data = elevation.collect_via_pkexec()
|
||||||
|
if data is not None:
|
||||||
|
elevation.set_privileged(data)
|
||||||
|
self._elevated.emit()
|
||||||
|
|
||||||
|
def _on_elevated(self) -> None:
|
||||||
|
# Re-run Health and Inventory now that root-only data (SMART/dmidecode) is available.
|
||||||
|
self.health_page._run()
|
||||||
|
self.inventory_page._run()
|
||||||
|
|
||||||
|
def _apply_alert_settings(self) -> None:
|
||||||
|
cfg = load_config()
|
||||||
|
self._alert_monitor.enabled = bool(cfg.get("alerts_enabled", True))
|
||||||
|
self._alert_monitor.gpu_temp = float(cfg.get("gpu_temp_alert", 90.0))
|
||||||
|
self._alert_monitor.cpu_temp = float(cfg.get("cpu_temp_alert", 95.0))
|
||||||
|
|
||||||
def _manual_check(self) -> None:
|
def _manual_check(self) -> None:
|
||||||
if self._applied:
|
if self._applied:
|
||||||
return
|
return
|
||||||
@@ -232,9 +274,9 @@ class MainWindow(QMainWindow):
|
|||||||
return
|
return
|
||||||
blocks = []
|
blocks = []
|
||||||
for tag, date, notes in releases:
|
for tag, date, notes in releases:
|
||||||
head = tag + (f" ({date})" if date else "")
|
title = f"## {tag}" + (f" — {date}" if date else "")
|
||||||
blocks.append(f"{head}\n{'─' * len(head)}\n{notes or '(no notes)'}")
|
blocks.append(f"{title}\n\n{notes or '_(no notes)_'}")
|
||||||
view.setPlainText("\n\n".join(blocks))
|
view.setMarkdown("\n\n".join(blocks)) # render Markdown instead of raw text (#1)
|
||||||
|
|
||||||
def _check_updates(self) -> None:
|
def _check_updates(self) -> None:
|
||||||
self._update_checked.emit(updates.update_state())
|
self._update_checked.emit(updates.update_state())
|
||||||
@@ -256,6 +298,9 @@ class MainWindow(QMainWindow):
|
|||||||
self._update_label.setText(f'<span style="color:{GOOD};">{tag} available</span>')
|
self._update_label.setText(f'<span style="color:{GOOD};">{tag} available</span>')
|
||||||
self._update_btn.setText(f"Update to {tag}")
|
self._update_btn.setText(f"Update to {tag}")
|
||||||
self._update_btn.setVisible(True)
|
self._update_btn.setVisible(True)
|
||||||
|
if self._alert_monitor.enabled and tag != self._notified_update_tag:
|
||||||
|
self._notified_update_tag = tag # once per version, not every poll
|
||||||
|
alerts.notify("Update available", f"RigDoctor {tag} is available — open RigDoctor to update.")
|
||||||
else: # UP_TO_DATE
|
else: # UP_TO_DATE
|
||||||
self._update_label.setText("up-to-date")
|
self._update_label.setText("up-to-date")
|
||||||
|
|
||||||
|
|||||||
@@ -0,0 +1,108 @@
|
|||||||
|
"""Notifications page (M8 config): user-configurable alert settings."""
|
||||||
|
|
||||||
|
from __future__ import annotations
|
||||||
|
|
||||||
|
from PySide6.QtCore import Qt, Signal
|
||||||
|
from PySide6.QtWidgets import (
|
||||||
|
QCheckBox,
|
||||||
|
QDoubleSpinBox,
|
||||||
|
QFrame,
|
||||||
|
QGridLayout,
|
||||||
|
QHBoxLayout,
|
||||||
|
QLabel,
|
||||||
|
QPushButton,
|
||||||
|
QVBoxLayout,
|
||||||
|
QWidget,
|
||||||
|
)
|
||||||
|
|
||||||
|
from ..config import load_config, update_config
|
||||||
|
from ..core import alerts
|
||||||
|
|
||||||
|
|
||||||
|
class NotificationsPage(QWidget):
|
||||||
|
changed = Signal() # settings saved — main window re-applies them live
|
||||||
|
|
||||||
|
def __init__(self) -> None:
|
||||||
|
super().__init__()
|
||||||
|
self.setObjectName("Page")
|
||||||
|
root = QVBoxLayout(self)
|
||||||
|
root.setContentsMargins(20, 18, 20, 18)
|
||||||
|
root.setSpacing(16)
|
||||||
|
|
||||||
|
title = QLabel("Notifications")
|
||||||
|
title.setObjectName("PageTitle")
|
||||||
|
root.addWidget(title)
|
||||||
|
|
||||||
|
card = QFrame()
|
||||||
|
card.setObjectName("Card")
|
||||||
|
v = QVBoxLayout(card)
|
||||||
|
v.setContentsMargins(16, 14, 16, 14)
|
||||||
|
v.setSpacing(10)
|
||||||
|
head = QLabel("Alerts")
|
||||||
|
head.setStyleSheet("font-weight: 700; background: transparent;")
|
||||||
|
v.addWidget(head)
|
||||||
|
|
||||||
|
self._enabled = QCheckBox("Enable desktop notifications")
|
||||||
|
v.addWidget(self._enabled)
|
||||||
|
|
||||||
|
grid = QGridLayout()
|
||||||
|
grid.setHorizontalSpacing(12)
|
||||||
|
grid.setColumnStretch(2, 1)
|
||||||
|
self._gpu = self._spin()
|
||||||
|
self._cpu = self._spin()
|
||||||
|
grid.addWidget(QLabel("GPU temperature alert"), 0, 0)
|
||||||
|
grid.addWidget(self._gpu, 0, 1)
|
||||||
|
grid.addWidget(QLabel("CPU temperature alert"), 1, 0)
|
||||||
|
grid.addWidget(self._cpu, 1, 1)
|
||||||
|
v.addLayout(grid)
|
||||||
|
|
||||||
|
note = QLabel("GPU-lost and new-version alerts are included whenever notifications are enabled.")
|
||||||
|
note.setObjectName("Muted")
|
||||||
|
note.setWordWrap(True)
|
||||||
|
v.addWidget(note)
|
||||||
|
|
||||||
|
buttons = QHBoxLayout()
|
||||||
|
save = QPushButton("Save")
|
||||||
|
save.setObjectName("PrimaryButton")
|
||||||
|
save.clicked.connect(self._save)
|
||||||
|
test = QPushButton("Send test")
|
||||||
|
test.clicked.connect(self._test)
|
||||||
|
buttons.addWidget(save)
|
||||||
|
buttons.addWidget(test)
|
||||||
|
buttons.addStretch(1)
|
||||||
|
v.addLayout(buttons)
|
||||||
|
self._status = QLabel("")
|
||||||
|
self._status.setObjectName("Muted")
|
||||||
|
v.addWidget(self._status)
|
||||||
|
|
||||||
|
root.addWidget(card)
|
||||||
|
root.addStretch(1)
|
||||||
|
self._load()
|
||||||
|
|
||||||
|
@staticmethod
|
||||||
|
def _spin() -> QDoubleSpinBox:
|
||||||
|
spin = QDoubleSpinBox()
|
||||||
|
spin.setRange(40, 110)
|
||||||
|
spin.setDecimals(0)
|
||||||
|
spin.setSingleStep(1)
|
||||||
|
spin.setSuffix(" °C")
|
||||||
|
return spin
|
||||||
|
|
||||||
|
def _load(self) -> None:
|
||||||
|
cfg = load_config()
|
||||||
|
self._enabled.setChecked(bool(cfg.get("alerts_enabled", True)))
|
||||||
|
self._gpu.setValue(float(cfg.get("gpu_temp_alert", 90.0)))
|
||||||
|
self._cpu.setValue(float(cfg.get("cpu_temp_alert", 95.0)))
|
||||||
|
|
||||||
|
def _save(self) -> None:
|
||||||
|
update_config(
|
||||||
|
alerts_enabled=self._enabled.isChecked(),
|
||||||
|
gpu_temp_alert=self._gpu.value(),
|
||||||
|
cpu_temp_alert=self._cpu.value(),
|
||||||
|
)
|
||||||
|
self.changed.emit()
|
||||||
|
self._status.setText("Saved.")
|
||||||
|
|
||||||
|
def _test(self) -> None:
|
||||||
|
ok = alerts.notify("RigDoctor", "Test notification — alerts are working.")
|
||||||
|
self._status.setText("Test notification sent." if ok else "notify-send not found — install libnotify-bin (Setup).")
|
||||||
@@ -0,0 +1,38 @@
|
|||||||
|
"""Tests for the M8 alert monitor (edge-triggered; notify mocked)."""
|
||||||
|
|
||||||
|
import unittest
|
||||||
|
from unittest import mock
|
||||||
|
|
||||||
|
from rigdoctor.core import alerts
|
||||||
|
from rigdoctor.core.sample import Reading, Sample
|
||||||
|
|
||||||
|
|
||||||
|
def _gpu(temp):
|
||||||
|
return Sample(readings=[Reading("gpu", "temp", temp, "°C")])
|
||||||
|
|
||||||
|
|
||||||
|
class AlertTests(unittest.TestCase):
|
||||||
|
@mock.patch.object(alerts, "notify")
|
||||||
|
def test_edge_triggered_no_repeat(self, m):
|
||||||
|
mon = alerts.AlertMonitor(gpu_temp=90.0, cooldown=0.0)
|
||||||
|
mon.check(_gpu(95)) # fires
|
||||||
|
mon.check(_gpu(96)) # still hot — no repeat while active
|
||||||
|
self.assertEqual(m.call_count, 1)
|
||||||
|
mon.check(_gpu(50)) # clears
|
||||||
|
mon.check(_gpu(95)) # hot again — fires
|
||||||
|
self.assertEqual(m.call_count, 2)
|
||||||
|
|
||||||
|
@mock.patch.object(alerts, "notify")
|
||||||
|
def test_no_alert_below_threshold(self, m):
|
||||||
|
alerts.AlertMonitor(gpu_temp=90.0).check(_gpu(70))
|
||||||
|
m.assert_not_called()
|
||||||
|
|
||||||
|
@mock.patch.object(alerts, "notify")
|
||||||
|
def test_gpu_lost(self, m):
|
||||||
|
mon = alerts.AlertMonitor()
|
||||||
|
mon.check(Sample(readings=[Reading("gpu", "status", None, "", "query-timeout")]))
|
||||||
|
m.assert_called_once()
|
||||||
|
|
||||||
|
|
||||||
|
if __name__ == "__main__":
|
||||||
|
unittest.main()
|
||||||
@@ -0,0 +1,32 @@
|
|||||||
|
"""Tests for config save/load (flat TOML writer)."""
|
||||||
|
|
||||||
|
import tempfile
|
||||||
|
import unittest
|
||||||
|
from pathlib import Path
|
||||||
|
from unittest import mock
|
||||||
|
|
||||||
|
from rigdoctor import config
|
||||||
|
|
||||||
|
|
||||||
|
class ConfigTests(unittest.TestCase):
|
||||||
|
def test_save_load_round_trip(self):
|
||||||
|
with tempfile.TemporaryDirectory() as d:
|
||||||
|
cf = Path(d) / "config.toml"
|
||||||
|
with mock.patch.object(config, "CONFIG_FILE", cf), mock.patch.object(config, "CONFIG_DIR", Path(d)):
|
||||||
|
config.save_config({"alerts_enabled": False, "gpu_temp_alert": 88.0, "update_check_minutes": 5})
|
||||||
|
loaded = config.load_config()
|
||||||
|
self.assertIs(loaded["alerts_enabled"], False)
|
||||||
|
self.assertEqual(loaded["gpu_temp_alert"], 88.0)
|
||||||
|
self.assertEqual(loaded["update_check_minutes"], 5)
|
||||||
|
|
||||||
|
def test_update_config_merges_and_keeps_defaults(self):
|
||||||
|
with tempfile.TemporaryDirectory() as d:
|
||||||
|
cf = Path(d) / "config.toml"
|
||||||
|
with mock.patch.object(config, "CONFIG_FILE", cf), mock.patch.object(config, "CONFIG_DIR", Path(d)):
|
||||||
|
config.update_config(cpu_temp_alert=70.0)
|
||||||
|
self.assertEqual(config.load_config()["cpu_temp_alert"], 70.0)
|
||||||
|
self.assertEqual(config.load_config()["gpu_temp_alert"], 90.0) # default preserved
|
||||||
|
|
||||||
|
|
||||||
|
if __name__ == "__main__":
|
||||||
|
unittest.main()
|
||||||
Reference in New Issue
Block a user