Files
maziggy 28781ea558 Keep the printer's name on statistics after it is deleted (issue #2873)
Every per-printer breakdown resolved the name against the printers that
exist now, so deleting a printer and choosing to keep its prints turned
"Ultron" into "Printer 1" in Prints by Printer, the success-rate and
time-accuracy lists, and Failures by Printer. Archives lose their printer
on that delete as well, so nothing was left to read a name from.

The runs themselves recorded the name they printed on. /archives/stats now
reports the last name each id was known by - taken from the newest run that
has one, so a later name-less row cannot blank it - and failure analysis
falls back to the same thing for ids with no printer left. The client keeps
preferring a live printer's own record, so a rename still shows up straight
away rather than after the next print.
2026-08-18 14:04:18 +02:00

267 lines
12 KiB
Python

from collections import defaultdict
from datetime import date, datetime, time, timedelta, timezone
from sqlalchemy import and_, func, select
from sqlalchemy.ext.asyncio import AsyncSession
from backend.app.models.print_log import PrintLogEntry
from backend.app.models.printer import Printer
class FailureAnalysisService:
"""Service for analyzing print failure patterns.
Reads from print_log_entries (per-event data) rather than print_archives
so reprints contribute each run and orphan events (archive deleted, log
row survived via ON DELETE SET NULL) still count consistently with
Quick Stats. The archive-based predecessor diverged from Quick Stats
after #1378 moved the rest of the page to per-event aggregation.
"""
def __init__(self, db: AsyncSession):
self.db = db
async def analyze_failures(
self,
days: int | None = None,
date_from: date | None = None,
date_to: date | None = None,
printer_id: int | None = None,
project_id: int | None = None,
created_by_id: int | None = None,
) -> dict:
"""Analyze failure patterns across logged print events."""
# Build base query — separate date vs non-date filters for trend reuse
base_filter = []
non_date_filter = []
if date_from or date_to:
if date_from:
dt_from = datetime.combine(date_from, time.min, tzinfo=timezone.utc)
base_filter.append(PrintLogEntry.created_at >= dt_from)
if date_to:
dt_to = datetime.combine(date_to, time.max, tzinfo=timezone.utc)
base_filter.append(PrintLogEntry.created_at <= dt_to)
range_start = dt_from if date_from else datetime.now(timezone.utc) - timedelta(days=365)
range_end = dt_to if date_to else datetime.now(timezone.utc)
effective_days = max((range_end - range_start).days, 1)
else:
effective_days = days if days is not None else 30
cutoff_date = datetime.now(timezone.utc) - timedelta(days=effective_days)
base_filter.append(PrintLogEntry.created_at >= cutoff_date)
if printer_id:
non_date_filter.append(PrintLogEntry.printer_id == printer_id)
# project_id is an archive-level concept; PrintLogEntry has no project
# link, so we resolve it by archive_id where present.
if project_id:
from backend.app.models.archive import PrintArchive
# Soft-deleted archives (#1343) keep their project_id, so without
# this the failure rate for a project still counts prints the user
# deleted from it — and disagrees with the project's own numbers,
# which now exclude them (#2731).
project_archive_ids = await self.db.execute(
select(PrintArchive.id).where(
PrintArchive.project_id == project_id,
PrintArchive.deleted_at.is_(None),
)
)
archive_ids = [row[0] for row in project_archive_ids.fetchall()]
if archive_ids:
non_date_filter.append(PrintLogEntry.archive_id.in_(archive_ids))
else:
# No archives in this project → nothing to count
non_date_filter.append(PrintLogEntry.id.is_(None))
if created_by_id is not None:
if created_by_id == -1:
non_date_filter.append(PrintLogEntry.created_by_id.is_(None))
else:
non_date_filter.append(PrintLogEntry.created_by_id == created_by_id)
base_filter.extend(non_date_filter)
# Total counts
total_result = await self.db.execute(select(func.count(PrintLogEntry.id)).where(and_(*base_filter)))
total_prints = total_result.scalar() or 0
successful_result = await self.db.execute(
select(func.count(PrintLogEntry.id)).where(and_(*base_filter, PrintLogEntry.status == "completed"))
)
successful_prints = successful_result.scalar() or 0
failed_result = await self.db.execute(
select(func.count(PrintLogEntry.id)).where(
and_(*base_filter, PrintLogEntry.status.in_(["failed", "aborted"]))
)
)
failed_prints = failed_result.scalar() or 0
# Failure rate divides by quality-outcome prints only — a cancelled or
# skipped print is neither a success nor a failure of the printer, so
# including it in the denominator silently lowered the displayed rate
# whenever the user stopped jobs (#1390). Total Prints (the absolute
# count incl. cancelled) is still returned separately for the "X / Y
# prints failed" caption.
outcome_prints = successful_prints + failed_prints
failure_rate = (failed_prints / outcome_prints * 100) if outcome_prints > 0 else 0
# Failures by reason
reason_result = await self.db.execute(
select(
PrintLogEntry.failure_reason,
func.count(PrintLogEntry.id).label("count"),
)
.where(and_(*base_filter, PrintLogEntry.status.in_(["failed", "aborted"])))
.group_by(PrintLogEntry.failure_reason)
.order_by(func.count(PrintLogEntry.id).desc())
)
failures_by_reason = {(row[0] or "Unknown"): row[1] for row in reason_result.fetchall()}
# Failures by filament type
filament_result = await self.db.execute(
select(
PrintLogEntry.filament_type,
func.count(PrintLogEntry.id).label("count"),
)
.where(and_(*base_filter, PrintLogEntry.status.in_(["failed", "aborted"])))
.group_by(PrintLogEntry.filament_type)
.order_by(func.count(PrintLogEntry.id).desc())
)
failures_by_filament = {(row[0] or "Unknown"): row[1] for row in filament_result.fetchall()}
# Failures by printer
printer_result = await self.db.execute(
select(
PrintLogEntry.printer_id,
func.count(PrintLogEntry.id).label("count"),
)
.where(
and_(
*base_filter,
PrintLogEntry.status.in_(["failed", "aborted"]),
PrintLogEntry.printer_id.isnot(None),
)
)
.group_by(PrintLogEntry.printer_id)
.order_by(func.count(PrintLogEntry.id).desc())
)
failures_by_printer_id = {row[0]: row[1] for row in printer_result.fetchall()}
# Get printer names
if failures_by_printer_id:
printers_result = await self.db.execute(
select(Printer.id, Printer.name).where(Printer.id.in_(failures_by_printer_id.keys()))
)
printer_names = {row[0]: row[1] for row in printers_result.fetchall()}
# A printer deleted with its history kept has no row left to read a
# name from, and "Printer 3" tells nobody which machine kept failing
# (#2873). Each run recorded the name it printed on, so fall back to
# the last one that id was known by.
missing = [pid for pid in failures_by_printer_id if pid not in printer_names]
if missing:
last_named_run = (
select(func.max(PrintLogEntry.id).label("entry_id"))
.where(PrintLogEntry.printer_id.in_(missing), PrintLogEntry.printer_name.isnot(None))
.group_by(PrintLogEntry.printer_id)
.subquery()
)
historic_result = await self.db.execute(
select(PrintLogEntry.printer_id, PrintLogEntry.printer_name).join(
last_named_run, PrintLogEntry.id == last_named_run.c.entry_id
)
)
for pid, name in historic_result.fetchall():
printer_names[pid] = name
failures_by_printer = {
printer_names.get(pid, f"Printer {pid}"): count for pid, count in failures_by_printer_id.items()
}
else:
failures_by_printer = {}
# Failures by hour of day
failed_events_result = await self.db.execute(
select(PrintLogEntry.started_at).where(
and_(
*base_filter,
PrintLogEntry.status.in_(["failed", "aborted"]),
PrintLogEntry.started_at.isnot(None),
)
)
)
failures_by_hour = defaultdict(int)
for (started_at,) in failed_events_result.fetchall():
if started_at:
hour = started_at.hour
failures_by_hour[hour] += 1
failures_by_hour_complete = {h: failures_by_hour.get(h, 0) for h in range(24)}
# Recent failures
recent_result = await self.db.execute(
select(PrintLogEntry)
.where(and_(*base_filter, PrintLogEntry.status.in_(["failed", "aborted"])))
.order_by(PrintLogEntry.created_at.desc())
.limit(10)
)
recent_failures = [
{
"id": e.archive_id,
"print_name": e.print_name,
"failure_reason": e.failure_reason,
"filament_type": e.filament_type,
"printer_id": e.printer_id,
"created_at": e.created_at.isoformat() if e.created_at else None,
}
for e in recent_result.scalars().all()
]
# Failure rate trend (by week)
trend_data = []
num_weeks = max(effective_days // 7, 1)
for i in range(num_weeks):
week_end = datetime.now(timezone.utc) - timedelta(weeks=i)
week_start = week_end - timedelta(weeks=1)
week_filter = [
PrintLogEntry.created_at >= week_start,
PrintLogEntry.created_at < week_end,
*non_date_filter,
]
week_total = await self.db.execute(select(func.count(PrintLogEntry.id)).where(and_(*week_filter)))
week_successful = await self.db.execute(
select(func.count(PrintLogEntry.id)).where(and_(*week_filter, PrintLogEntry.status == "completed"))
)
week_failed = await self.db.execute(
select(func.count(PrintLogEntry.id)).where(
and_(*week_filter, PrintLogEntry.status.in_(["failed", "aborted"]))
)
)
total = week_total.scalar() or 0
successful = week_successful.scalar() or 0
failed = week_failed.scalar() or 0
week_outcome = successful + failed
rate = (failed / week_outcome * 100) if week_outcome > 0 else 0
trend_data.append(
{
"week_start": week_start.date().isoformat(),
"total_prints": total,
"failed_prints": failed,
"failure_rate": round(rate, 1),
}
)
trend_data.reverse() # Oldest first
return {
"period_days": effective_days,
"total_prints": total_prints,
"failed_prints": failed_prints,
"failure_rate": round(failure_rate, 1),
"failures_by_reason": failures_by_reason,
"failures_by_filament": failures_by_filament,
"failures_by_printer": failures_by_printer,
"failures_by_hour": failures_by_hour_complete,
"recent_failures": recent_failures,
"trend": trend_data,
}