Measurement Report: save PDF next to the original file, not the temp folder + tidy header (Knut)

- Imported i1Profiler measurements convert into a temp folder; the PDF and Reveal
  defaulted there ('/private/var/folders/.../T/chromiq_report_…'). Each source now
  remembers its ORIGINAL path, and the PDF/Reveal anchor on that folder.
- The per-page report header listed every profile name; with many imported
  measurements that's unwieldy, so past a handful it collapses to the run count +
  date range.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: itsab1989

tests/test_i1profiler_convert_report_fixes.py

@@ -98,6 +98,30 @@ def fake_convert(src, argyll, out_dir):
     assert seen["src"] == txt
 
 
+def test_report_pdf_anchors_on_origin_not_temp(qapp, tmp_path, monkeypatch):
+    """Imported measurements convert into a temp folder, but the PDF / Reveal must
+    default to the user's ORIGINAL folder, never the temp one (Knut)."""
+    from ui.dialogs.measurement_report_dialog import MeasurementReportDialog as M
+
+    monkeypatch.setattr(
+        "workflow.measurement_report.list_project_reports", lambda d: [])
+    origin = tmp_path / "myfolder" / "Epson_2026-01-06.txt"
+    origin.parent.mkdir(parents=True)
+    origin.write_text("x")
+    temp_ti3 = tmp_path / "chromiq_report_xyz" / "Epson_2026-01-06.ti3"
+    temp_ti3.parent.mkdir()
+    temp_ti3.write_text("y")
+
+    host = types.SimpleNamespace(_sources=[], _ti3=None)
+    host._source_key = types.MethodType(M._source_key, host)
+    host._gather_runs = lambda t: (t.stem, [{"created": "2026-01-01"}])
+    types.MethodType(M._append_source, host)(temp_ti3, origin=origin)
+
+    anchor = types.MethodType(M._anchor_dir, host)()
+    assert anchor == origin.parent                 # user's folder…
+    assert anchor != temp_ti3.parent               # …not the temp conversion folder
+
+
 def test_report_as_ti3_raises_on_bad_file(qapp, settings, tmp_path, monkeypatch):
     """_as_ti3 raises on a bad file so the batch adder can list what failed."""
     from ui.dialogs.measurement_report_dialog import MeasurementReportDialog

ui/dialogs/measurement_report_dialog.py

@@ -689,27 +689,32 @@ def _source_key(self, ti3: Path) -> tuple:
             return ("dir", str(ti3.parent))
         return ("file", str(ti3))
 
-    def _append_source(self, ti3: Path) -> bool:
+    def _append_source(self, ti3: Path, origin: "Path | None" = None) -> bool:
         """Add one measurement to the source list (no repaint). Returns False if it
         is already present or has no runs. Raises on a gather error, so a batch add
-        can report which files failed."""
+        can report which files failed.
+
+        *origin* is the file the user actually picked (the same as *ti3* for a
+        ChromIQ .ti3, but the original .mxf/.txt/.cxf when *ti3* is a temp
+        conversion). The report is saved next to the origin, never the temp folder
+        (Knut)."""
         key = self._source_key(ti3)
         if any(s.get("key") == key for s in self._sources):
             return False
         name, runs = self._gather_runs(ti3)
         if not runs:
             return False
         self._sources.append({"key": key, "name": name, "dir": ti3.parent,
-                              "ti3": ti3, "runs": runs})
+                              "ti3": ti3, "origin": Path(origin or ti3), "runs": runs})
         if self._ti3 is None:
             self._ti3 = ti3
         return True
 
-    def _add_source(self, ti3: Path) -> None:
+    def _add_source(self, ti3: Path, origin: "Path | None" = None) -> None:
         """Add a single measurement and repaint (used when opening the report on
         one file)."""
         try:
-            added = self._append_source(ti3)
+            added = self._append_source(ti3, origin)
         except Exception as exc:  # noqa: BLE001
             self._view.setHtml(self._error_html(str(exc)))
             return
@@ -755,7 +760,7 @@ def _on_add_project(self) -> None:
         added, failed = 0, []
         for path in paths:
             try:
-                if self._append_source(self._as_ti3(Path(path))):
+                if self._append_source(self._as_ti3(Path(path)), origin=Path(path)):
                     added += 1
             except Exception as exc:  # noqa: BLE001
                 failed.append(f"{Path(path).name} — {exc}")
@@ -848,21 +853,31 @@ def _trend_configs(self) -> list:
              corner_metrics, None, 1, False),
         ]
 
+    def _anchor_dir(self) -> Path:
+        """The folder the PDF and Reveal default to: the folder of the FIRST
+        source's ORIGINAL file — a ChromIQ run folder, or the user's own folder for
+        an imported measurement — never the temp folder an i1Profiler file is
+        converted into (Knut)."""
+        if self._sources:
+            return self._sources[0]["origin"].parent
+        return self._ti3.parent if self._ti3 else Path.cwd()
+
     def _profile_root(self) -> Path:
         """The profile's project folder (``<project>/runs/<id>`` → ``<project>``),
-        or the run folder itself for a browsed external ``.ti3`` that isn't in a
+        or the folder itself for a browsed/imported measurement that isn't in a
         ChromIQ project layout."""
-        run_dir = self._ti3.parent if self._ti3 else Path.cwd()
+        run_dir = self._anchor_dir()
         if run_dir.parent.name == "runs":
             return run_dir.parents[1]
         return run_dir
 
     def _report_dir(self) -> Path:
         """Where a PDF is saved (Knut): an all-runs report belongs to the whole
         profile, so it goes in a ``reports`` folder next to ``runs/``; a single-run
-        report goes in that run's own ``reports`` folder."""
+        report goes in that run's own ``reports`` folder. For an imported
+        measurement it goes in a ``reports`` folder next to the file itself."""
         from core.file_manager import reports_subdir
-        run_dir = self._ti3.parent
+        run_dir = self._anchor_dir()
         if self._all_runs_check.isChecked() and run_dir.parent.name == "runs":
             return reports_subdir(self._profile_root())
         return reports_subdir(run_dir)
@@ -872,7 +887,7 @@ def _on_reveal(self) -> None:
         the reports folder and open saved PDFs (Knut)."""
         from PyQt6.QtCore import QUrl
         from PyQt6.QtGui import QDesktopServices
-        if self._ti3:
+        if self._sources or self._ti3:
             QDesktopServices.openUrl(QUrl.fromLocalFile(str(self._profile_root())))
 
     def _export_pdf(self) -> None:
@@ -1045,12 +1060,18 @@ def _scope_header_units(self, runs: list) -> list:
         from workflow.measurement_report import report_scope
         sc = report_scope(runs)
         names = list(dict.fromkeys(p["name"] for p in sc["profiles"]))  # de-dup, ordered
-        units = [n + ("," if i < len(names) - 1 else "")
-                 for i, n in enumerate(names)]
         d0, d1 = sc["date_range"]
         n = sc["total"]
-        units.append("  " + tr("{n} measurement run").format(n=n) if n == 1
-                     else "  " + tr("{n} measurement runs").format(n=n))
+        # Listing every name gets unwieldy once there are many (e.g. a folder of
+        # imported measurements); past a handful, drop the names and let the run
+        # count + date range speak for the scope (Knut).
+        if len(names) <= 4:
+            units = [nm + ("," if i < len(names) - 1 else "")
+                     for i, nm in enumerate(names)]
+            units.append("  " + (tr("{n} measurement run").format(n=n) if n == 1
+                                  else tr("{n} measurement runs").format(n=n)))
+        else:
+            units = [tr("{n} measurement runs").format(n=n)]
         units[-1] += f" ({d0} – {d1})"
         # Trailing space on the name units so they don't run together.
         return [(u + " ") if u.endswith(",") else u for u in units]