Add Phase 5 viewer modes and Phase 6 region review with dept export.

T1/T2/Overlay view tabs, Confirm/False Positive per region, CSV export, and configurable DEPT_API submit on dev Space.

Co-authored-by: Cursor <cursoragent@cursor.com>

.env.example

@@ -28,3 +28,7 @@
 
 # Hugging Face model cache directory
 # HF_HOME=./.hf_cache
+
+# Department export API (optional — FR-08)
+# DEPT_API_URL=https://dept.example.gov/api/changes
+# DEPT_API_KEY=your-api-key

DEV_SETUP.md

@@ -157,6 +157,7 @@ Expected: `"appMode": "dda"`, `"status": "ok"`.
 2. **Change Detection** — pick Base (T1) and Comparison (T2), run detection (async jobs on HF; sync locally).
 3. **Reports** — history, PDF download, browser report at `/dda/reports/{id}`.
 4. **Bell icon** — in-app notifications for completed jobs.
+5. **Review (FR-08)** — Confirm / False Positive per region, export confirmed CSV, submit to dept API (`DEPT_API_URL`).
 
 ---
 

Dockerfile

@@ -21,7 +21,7 @@ WORKDIR /app
 
 # Build-time info + cache-bust:
 # Changing APP_BUILD forces Docker to re-run subsequent layers (including pip install).
-ARG APP_BUILD=34
+ARG APP_BUILD=35
 ENV MAX_GEOTIFF_MB=5120
 ENV APP_BUILD=${APP_BUILD}
 ENV GDAL_CONFIG=/usr/bin/gdal-config

app/dda/bootstrap.py

@@ -9,6 +9,7 @@ from .jobs_routes import router as jobs_router
 from .library_routes import router as library_router
 from .local_routes import router as local_router
 from .reports_routes import router as reports_router
+from .review_routes import router as review_router
 from .seed import seed_delhi_hierarchy
 
 logger = logging.getLogger(__name__)
@@ -57,5 +58,6 @@ def setup_dda(app: FastAPI) -> None:
     app.include_router(library_router, prefix="/api/dda", tags=["dda"])
     app.include_router(jobs_router, prefix="/api/dda", tags=["dda-jobs"])
     app.include_router(reports_router, prefix="/api/dda", tags=["dda-reports"])
+    app.include_router(review_router, prefix="/api/dda", tags=["dda-review"])
     app.include_router(local_router, prefix="/api/dda", tags=["dda-local"])
-    logger.info("APP_MODE=dda — DDA routes enabled (library, jobs, reports, local folder)")
+    logger.info("APP_MODE=dda — DDA routes enabled (library, jobs, reports, review, local folder)")

app/dda/dept_export.py

@@ -0,0 +1,152 @@
+"""Department export adapter — API or file fallback (FR-08)."""
+from __future__ import annotations
+
+import csv
+import io
+import json
+import logging
+import os
+from dataclasses import dataclass
+from typing import Any, Dict, List, Optional
+
+import requests
+
+from ..models import DetectionRun
+
+logger = logging.getLogger(__name__)
+
+DEPT_API_URL = os.environ.get("DEPT_API_URL", "").strip()
+DEPT_API_KEY = os.environ.get("DEPT_API_KEY", "").strip()
+
+
+@dataclass
+class ExportResult:
+    ok: bool
+    mode: str  # api | file
+    message: str
+    submitted_count: int = 0
+    download_path: Optional[str] = None
+    detail: Optional[Dict[str, Any]] = None
+
+
+def regions_to_csv_rows(run: DetectionRun, regions: List[dict]) -> str:
+    """Build CSV string for export."""
+    buf = io.StringIO()
+    writer = csv.writer(buf)
+    writer.writerow([
+        "run_id", "run_title", "zone", "village", "region_id",
+        "dda_change_type", "internal_type", "confidence", "area_px",
+        "latitude", "longitude", "area_sq_m", "review_status", "notes",
+    ])
+    for r in regions:
+        lat = r.get("latitude") or (r.get("latLng") or {}).get("lat")
+        lng = r.get("longitude") or (r.get("latLng") or {}).get("lng")
+        writer.writerow([
+            run.id,
+            run.title,
+            run.zone or "",
+            run.village or "",
+            r.get("id", ""),
+            r.get("ddaChangeType") or "",
+            r.get("internalObjectType") or r.get("objectType") or "",
+            f'{(r.get("confidence") or 0):.4f}',
+            r.get("area", ""),
+            lat if lat is not None else "",
+            lng if lng is not None else "",
+            r.get("areaSqM") if r.get("areaSqM") is not None else "",
+            r.get("reviewStatus") or "pending",
+            r.get("reviewNotes") or "",
+        ])
+    return buf.getvalue()
+
+
+class FileExporter:
+    """Fallback — client downloads CSV via API."""
+
+    def submit(self, run: DetectionRun, regions: List[dict]) -> ExportResult:
+        if not regions:
+            return ExportResult(
+                ok=False,
+                mode="file",
+                message="No confirmed regions to export.",
+                submitted_count=0,
+            )
+        return ExportResult(
+            ok=True,
+            mode="file",
+            message=f"{len(regions)} confirmed region(s) ready for CSV download.",
+            submitted_count=len(regions),
+            download_path=f"/api/dda/reports/{run.id}/export.csv?confirmed=1",
+        )
+
+
+class ApiExporter:
+    """POST confirmed changes to departmental API."""
+
+    def __init__(self, url: str, api_key: str = ""):
+        self.url = url.rstrip("/")
+        self.api_key = api_key
+
+    def submit(self, run: DetectionRun, regions: List[dict]) -> ExportResult:
+        if not regions:
+            return ExportResult(
+                ok=False,
+                mode="api",
+                message="No confirmed regions to submit.",
+            )
+        payload = {
+            "runId": run.id,
+            "title": run.title,
+            "zone": run.zone or "",
+            "village": run.village or "",
+            "changePercentage": run.change_percentage,
+            "regions": [
+                {
+                    "id": r.get("id"),
+                    "ddaChangeType": r.get("ddaChangeType"),
+                    "objectType": r.get("objectType"),
+                    "confidence": r.get("confidence"),
+                    "area": r.get("area"),
+                    "latitude": r.get("latitude") or (r.get("latLng") or {}).get("lat"),
+                    "longitude": r.get("longitude") or (r.get("latLng") or {}).get("lng"),
+                    "areaSqM": r.get("areaSqM"),
+                }
+                for r in regions
+            ],
+        }
+        headers = {"Content-Type": "application/json", "Accept": "application/json"}
+        if self.api_key:
+            headers["Authorization"] = f"Bearer {self.api_key}"
+        try:
+            resp = requests.post(self.url, json=payload, headers=headers, timeout=60)
+            if 200 <= resp.status_code < 300:
+                detail = None
+                try:
+                    detail = resp.json()
+                except Exception:
+                    detail = {"body": resp.text[:500]}
+                return ExportResult(
+                    ok=True,
+                    mode="api",
+                    message=f"Submitted {len(regions)} confirmed region(s) to departmental system.",
+                    submitted_count=len(regions),
+                    detail=detail,
+                )
+            return ExportResult(
+                ok=False,
+                mode="api",
+                message=f"Department API returned {resp.status_code}: {resp.text[:200]}",
+            )
+        except Exception as exc:
+            logger.exception("Department API submit failed")
+            return ExportResult(
+                ok=False,
+                mode="api",
+                message=f"Department API error: {exc}",
+            )
+
+
+def get_exporter() -> FileExporter | ApiExporter:
+    if DEPT_API_URL:
+        return ApiExporter(DEPT_API_URL, DEPT_API_KEY)
+    return FileExporter()

app/dda/jobs_routes.py

@@ -132,6 +132,8 @@ def _run_detail(db: Session, run: DetectionRun, user_id: int) -> dict:
         raise HTTPException(status_code=403, detail="Not allowed")
 
     regions = json.loads(run.regions_json or "[]")
+    from .review_service import merge_reviews
+    regions = merge_reviews(db, run.id, regions)
     overlay_b64 = ""
     if run.overlay_path:
         overlay_file = DATA_DIR / run.overlay_path

app/dda/models.py

@@ -88,3 +88,18 @@ class DetectionJob(Base):
     started_at = Column(DateTime, nullable=True)
     completed_at = Column(DateTime, nullable=True)
     created_at = Column(DateTime, default=_utcnow)
+
+
+class RegionReview(Base):
+    """Per-region human review state (FR-08)."""
+    __tablename__ = "dda_region_reviews"
+
+    id = Column(Integer, primary_key=True, index=True)
+    run_id = Column(Integer, ForeignKey("detection_runs.id"), nullable=False, index=True)
+    region_id = Column(Integer, nullable=False, index=True)
+    status = Column(String(32), default="pending")  # pending|confirmed|false_positive|submitted
+    reviewer_id = Column(Integer, ForeignKey("users.id"), nullable=True)
+    notes = Column(Text, default="")
+    reviewed_at = Column(DateTime, nullable=True)
+    submitted_at = Column(DateTime, nullable=True)
+    created_at = Column(DateTime, default=_utcnow)

app/dda/report_pdf.py

@@ -17,8 +17,9 @@ def _safe_filename(title: str, run_id: int) -> str:
     return f"DDA_Report_{run_id}_{base}.pdf"
 
 
-def build_report_dict(run: DetectionRun, *, include_overlay_b64: bool = False) -> Dict[str, Any]:
-    regions: List[dict] = json.loads(run.regions_json or "[]")
+def build_report_dict(run: DetectionRun, *, include_overlay_b64: bool = False, regions: Optional[List[dict]] = None) -> Dict[str, Any]:
+    if regions is None:
+        regions = json.loads(run.regions_json or "[]")
     payload: Dict[str, Any] = {
         "id": run.id,
         "title": run.title,

app/dda/reports_routes.py

@@ -15,6 +15,7 @@ from ..models import DetectionRun
 from ..notifier import send_notification
 from .config import get_public_base_url
 from .report_pdf import build_report_dict, generate_report_pdf
+from .review_service import load_regions, merge_reviews
 
 logger = logging.getLogger(__name__)
 router = APIRouter()
@@ -42,7 +43,8 @@ def get_report(run_id: int, db: Session = Depends(get_db)):
     _require_dda()
     user = get_or_create_guest_user(db)
     run = _get_user_run(db, run_id, user.id)
-    data = build_report_dict(run, include_overlay_b64=True)
+    regions = merge_reviews(db, run.id, load_regions(run))
+    data = build_report_dict(run, include_overlay_b64=True, regions=regions)
     data["reportUrl"] = f"{get_public_base_url()}/dda/reports/{run.id}"
     return data
 

app/dda/review_routes.py

@@ -0,0 +1,139 @@
+"""Region review and departmental export API (FR-08)."""
+from __future__ import annotations
+
+import logging
+from typing import Optional
+
+from fastapi import APIRouter, Depends, HTTPException, Query
+from fastapi.responses import Response
+from pydantic import BaseModel
+from sqlalchemy.orm import Session
+
+from ..auth import get_or_create_guest_user
+from ..database import get_db
+from ..models import DetectionRun
+from .dept_export import ExportResult, get_exporter, regions_to_csv_rows
+from .review_service import (
+    filter_regions_by_review,
+    load_regions,
+    mark_confirmed_submitted,
+    merge_reviews,
+    review_summary,
+    set_region_review,
+)
+
+logger = logging.getLogger(__name__)
+router = APIRouter()
+
+
+def _require_dda():
+    from .config import IS_DDA_MODE
+    if not IS_DDA_MODE:
+        raise HTTPException(status_code=404, detail="DDA mode is not enabled")
+
+
+def _get_user_run(db: Session, run_id: int, user_id: int) -> DetectionRun:
+    run = db.query(DetectionRun).filter(
+        DetectionRun.id == run_id,
+        DetectionRun.user_id == user_id,
+    ).first()
+    if not run:
+        raise HTTPException(status_code=404, detail="Report not found")
+    return run
+
+
+class RegionReviewBody(BaseModel):
+    reviewStatus: str
+    notes: Optional[str] = ""
+
+
+@router.patch("/reports/{run_id}/regions/{region_id}")
+def patch_region_review(
+    run_id: int,
+    region_id: int,
+    body: RegionReviewBody,
+    db: Session = Depends(get_db),
+):
+    """Mark a region as confirmed or false positive."""
+    _require_dda()
+    user = get_or_create_guest_user(db)
+    run = _get_user_run(db, run_id, user.id)
+    if body.reviewStatus not in ("confirmed", "false_positive", "pending"):
+        raise HTTPException(status_code=400, detail="reviewStatus must be confirmed, false_positive, or pending")
+    try:
+        updated = set_region_review(
+            db, run, region_id, body.reviewStatus, user.id, body.notes or ""
+        )
+    except LookupError:
+        raise HTTPException(status_code=404, detail="Region not found")
+    except PermissionError as exc:
+        raise HTTPException(status_code=409, detail=str(exc))
+    except ValueError as exc:
+        raise HTTPException(status_code=400, detail=str(exc))
+    return {"ok": True, "region": updated, "summary": review_summary(merge_reviews(db, run.id, load_regions(run)))}
+
+
+@router.get("/reports/{run_id}/review-summary")
+def get_review_summary(run_id: int, db: Session = Depends(get_db)):
+    _require_dda()
+    user = get_or_create_guest_user(db)
+    run = _get_user_run(db, run_id, user.id)
+    regions = merge_reviews(db, run.id, load_regions(run))
+    return {"runId": run.id, "summary": review_summary(regions)}
+
+
+@router.get("/reports/{run_id}/export.csv")
+def export_csv(
+    run_id: int,
+    confirmed: int = Query(0, ge=0, le=1),
+    db: Session = Depends(get_db),
+):
+    """Export regions as CSV. Use ?confirmed=1 for confirmed-only."""
+    _require_dda()
+    user = get_or_create_guest_user(db)
+    run = _get_user_run(db, run_id, user.id)
+    regions = merge_reviews(db, run.id, load_regions(run))
+    if confirmed:
+        regions = filter_regions_by_review(regions, "confirmed")
+    if not regions:
+        raise HTTPException(status_code=404, detail="No regions to export")
+    csv_text = regions_to_csv_rows(run, regions)
+    suffix = "confirmed" if confirmed else "all"
+    filename = f"dda_run_{run_id}_{suffix}.csv"
+    return Response(
+        content=csv_text,
+        media_type="text/csv",
+        headers={"Content-Disposition": f'attachment; filename="{filename}"'},
+    )
+
+
+@router.post("/reports/{run_id}/submit")
+def submit_confirmed(run_id: int, db: Session = Depends(get_db)):
+    """Submit confirmed regions to departmental API (or file fallback)."""
+    _require_dda()
+    user = get_or_create_guest_user(db)
+    run = _get_user_run(db, run_id, user.id)
+    regions = merge_reviews(db, run.id, load_regions(run))
+    confirmed = filter_regions_by_review(regions, "confirmed")
+    if not confirmed:
+        raise HTTPException(status_code=400, detail="No confirmed regions to submit. Review regions first.")
+
+    exporter = get_exporter()
+    result: ExportResult = exporter.submit(run, confirmed)
+    if not result.ok:
+        raise HTTPException(status_code=502, detail=result.message)
+
+    submitted_ids = [int(r["id"]) for r in confirmed if r.get("id") is not None]
+    if result.mode == "api":
+        mark_confirmed_submitted(db, run.id, submitted_ids)
+        for r in confirmed:
+            r["reviewStatus"] = "submitted"
+
+    return {
+        "ok": True,
+        "mode": result.mode,
+        "message": result.message,
+        "submittedCount": result.submitted_count,
+        "downloadUrl": result.download_path,
+        "detail": result.detail,
+    }

app/dda/review_service.py

@@ -0,0 +1,127 @@
+"""Merge and persist region review state (FR-08)."""
+from __future__ import annotations
+
+import json
+from datetime import datetime, timezone
+from typing import Any, Dict, List, Optional
+
+from sqlalchemy.orm import Session
+
+from ..models import DetectionRun
+from .models import RegionReview
+
+VALID_REVIEW_STATUSES = {"pending", "confirmed", "false_positive", "submitted"}
+
+
+def _utcnow() -> datetime:
+    return datetime.now(timezone.utc)
+
+
+def load_regions(run: DetectionRun) -> List[dict]:
+    try:
+        return json.loads(run.regions_json or "[]")
+    except json.JSONDecodeError:
+        return []
+
+
+def save_regions(db: Session, run: DetectionRun, regions: List[dict]) -> None:
+    run.regions_json = json.dumps(regions)
+    run.regions_count = len(regions)
+    db.commit()
+
+
+def merge_reviews(db: Session, run_id: int, regions: List[dict]) -> List[dict]:
+    """Attach reviewStatus / notes from RegionReview rows onto region dicts."""
+    reviews = {
+        r.region_id: r
+        for r in db.query(RegionReview).filter(RegionReview.run_id == run_id).all()
+    }
+    out = []
+    for region in regions:
+        rid = int(region.get("id", 0))
+        rev = reviews.get(rid)
+        merged = dict(region)
+        if rev:
+            merged["reviewStatus"] = rev.status
+            merged["reviewNotes"] = rev.notes or ""
+            merged["reviewedAt"] = rev.reviewed_at.isoformat() if rev.reviewed_at else None
+        else:
+            merged.setdefault("reviewStatus", region.get("reviewStatus", "pending"))
+        out.append(merged)
+    return out
+
+
+def get_or_create_review(db: Session, run_id: int, region_id: int) -> RegionReview:
+    row = (
+        db.query(RegionReview)
+        .filter(RegionReview.run_id == run_id, RegionReview.region_id == region_id)
+        .first()
+    )
+    if row:
+        return row
+    row = RegionReview(run_id=run_id, region_id=region_id, status="pending")
+    db.add(row)
+    db.commit()
+    db.refresh(row)
+    return row
+
+
+def set_region_review(
+    db: Session,
+    run: DetectionRun,
+    region_id: int,
+    review_status: str,
+    reviewer_id: int,
+    notes: str = "",
+) -> dict:
+    if review_status not in ("confirmed", "false_positive", "pending"):
+        raise ValueError(f"Invalid reviewStatus: {review_status}")
+
+    regions = load_regions(run)
+    region = next((r for r in regions if int(r.get("id", -1)) == region_id), None)
+    if not region:
+        raise LookupError("Region not found in run")
+
+    row = get_or_create_review(db, run.id, region_id)
+    if row.status == "submitted":
+        raise PermissionError("Region already submitted — cannot change review")
+
+    row.status = review_status
+    row.reviewer_id = reviewer_id
+    row.notes = (notes or "").strip()
+    row.reviewed_at = _utcnow()
+    db.commit()
+
+    region["reviewStatus"] = review_status
+    if notes:
+        region["reviewNotes"] = notes
+    save_regions(db, run, regions)
+    return merge_reviews(db, run.id, [region])[0]
+
+
+def filter_regions_by_review(regions: List[dict], status: str) -> List[dict]:
+    return [r for r in regions if (r.get("reviewStatus") or "pending") == status]
+
+
+def mark_confirmed_submitted(db: Session, run_id: int, region_ids: List[int]) -> int:
+    q = db.query(RegionReview).filter(
+        RegionReview.run_id == run_id,
+        RegionReview.region_id.in_(region_ids),
+        RegionReview.status == "confirmed",
+    )
+    now = _utcnow()
+    count = 0
+    for row in q.all():
+        row.status = "submitted"
+        row.submitted_at = now
+        count += 1
+    db.commit()
+    return count
+
+
+def review_summary(regions: List[dict]) -> Dict[str, int]:
+    summary = {"pending": 0, "confirmed": 0, "false_positive": 0, "submitted": 0}
+    for r in regions:
+        st = r.get("reviewStatus") or "pending"
+        summary[st] = summary.get(st, 0) + 1
+    return summary

app/main.py

@@ -28,7 +28,7 @@ from .auth import (
 from .database import Base, engine, get_db, DATA_DIR
 from .models import User, DetectionRun
 from . import dda as _dda_pkg  # noqa: F401 — register DDA tables
-from .dda.models import DdaZone, DdaVillage, ImageAsset, DetectionJob  # noqa: F401
+from .dda.models import DdaZone, DdaVillage, ImageAsset, DetectionJob, RegionReview  # noqa: F401
 from .dda.config import IS_DDA_MODE
 from .dda.bootstrap import init_dda_database, setup_dda
 from .notifier import send_notification, send_test_email
@@ -483,6 +483,9 @@ def get_run(
     if not run:
         raise HTTPException(status_code=404, detail="Run not found")
     regions = _load_regions_json(run.regions_json)
+    if IS_DDA_MODE:
+        from .dda.review_service import merge_reviews
+        regions = merge_reviews(db, run.id, regions)
     return {
         "id": run.id,
         "title": run.title,

static/css/dda.css

@@ -419,3 +419,62 @@
 .dda-report-overlay { max-width: 100%; height: auto; border-radius: 8px; border: 1px solid var(--border); }
 .dda-report-email-row { display: flex; flex-wrap: wrap; gap: 0.5rem; margin-top: 0.5rem; }
 .dda-report-email-row input { flex: 1; min-width: 200px; }
+
+.dda-view-toolbar {
+  display: flex;
+  flex-wrap: wrap;
+  align-items: center;
+  gap: 0.35rem;
+  margin: 0.5rem 0 0.75rem;
+}
+.dda-view-label { font-size: 0.8rem; color: var(--text-muted, #888); margin-right: 0.25rem; }
+.dda-view-btn {
+  font-size: 0.78rem;
+  padding: 0.25rem 0.55rem;
+  border: 1px solid var(--border);
+  border-radius: 6px;
+  background: transparent;
+  cursor: pointer;
+  color: inherit;
+}
+.dda-view-btn.active {
+  background: var(--grad-start, #2e33c5);
+  color: #fff;
+  border-color: transparent;
+}
+.dda-mode-slider .compare-handle { display: block; }
+.dda-mode-t1 .compare-handle,
+.dda-mode-t2 .compare-handle,
+.dda-mode-overlay .compare-handle { pointer-events: none; }
+
+.dda-review-bar {
+  display: flex;
+  flex-wrap: wrap;
+  align-items: center;
+  justify-content: space-between;
+  gap: 0.5rem;
+  margin: 0.75rem 0;
+  padding: 0.5rem 0;
+  border-top: 1px solid var(--border);
+}
+.dda-review-actions { display: flex; flex-wrap: wrap; gap: 0.35rem; }
+.dda-review-btns { white-space: nowrap; }
+.dda-review-btns .btn { padding: 0.15rem 0.4rem; margin-right: 0.15rem; }
+
+.review-badge {
+  font-size: 0.72rem;
+  font-weight: 600;
+  text-transform: capitalize;
+  padding: 0.1rem 0.35rem;
+  border-radius: 4px;
+}
+.review-pending { background: rgba(0,0,0,0.06); color: #666; }
+.review-confirmed { background: rgba(22, 163, 74, 0.15); color: #15803d; }
+.review-false-positive { background: rgba(220, 38, 38, 0.12); color: #b91c1c; }
+.review-submitted { background: rgba(46, 51, 197, 0.12); color: #2e33c5; }
+
+.highlight-pulse { animation: dda-pulse 1.2s ease-in-out 2; }
+@keyframes dda-pulse {
+  0%, 100% { box-shadow: 0 0 0 0 rgba(16, 185, 129, 0.5); }
+  50% { box-shadow: 0 0 0 6px rgba(16, 185, 129, 0); }
+}

static/js/dda/result.js

@@ -1,9 +1,12 @@
-/** DDA result modal — before/after compare slider (same as production). */
+/** DDA result modal — compare slider, view modes, region review (FR-07, FR-08). */
 
 let ddaCurrentResult = null;
 let ddaRegionRows = [];
 let ddaRegionList = [];
 let ddaRegionPage = 0;
+let ddaSelectedRegionId = null;
+let ddaViewMode = 'slider';
+let ddaViewUrls = { before: '', after: '', overlay: '' };
 const DDA_REGIONS_PER_PAGE = 10;
 let ddaZoom = 1;
 const DDA_ZOOM_MIN = 0.5;
@@ -16,6 +19,95 @@ function formatCompact(n) {
   return n.toLocaleString();
 }
 
+function reviewBadgeClass(status) {
+  const s = (status || 'pending').toLowerCase();
+  return `review-badge review-${s.replace('_', '-')}`;
+}
+
+function setDdaViewMode(mode) {
+  ddaViewMode = mode;
+  document.querySelectorAll('.dda-view-btn').forEach((btn) => {
+    btn.classList.toggle('active', btn.dataset.view === mode);
+  });
+  const slider = document.getElementById('compare-slider');
+  const handle = document.getElementById('compare-handle');
+  const beforeImg = document.getElementById('compare-before-img');
+  const afterImg = document.getElementById('compare-after-img');
+  const afterClip = document.getElementById('compare-after-clip');
+  if (!slider || !beforeImg || !afterImg) return;
+
+  slider.classList.remove('dda-mode-t1', 'dda-mode-t2', 'dda-mode-overlay', 'dda-mode-slider');
+  slider.classList.add(`dda-mode-${mode}`);
+
+  const overlaySrc = ddaViewUrls.overlay;
+  const beforeSrc = ddaViewUrls.before;
+  const afterSrc = ddaViewUrls.after || overlaySrc;
+
+  if (mode === 'slider') {
+    beforeImg.src = beforeSrc;
+    afterImg.src = overlaySrc;
+    if (afterClip) afterClip.style.clipPath = 'inset(0 0 0 50%)';
+    if (handle) { handle.style.display = ''; handle.style.left = '50%'; }
+  } else if (mode === 't1') {
+    beforeImg.src = beforeSrc;
+    afterImg.src = overlaySrc;
+    if (afterClip) afterClip.style.clipPath = 'inset(0 0 0 100%)';
+    if (handle) handle.style.display = 'none';
+  } else if (mode === 't2') {
+    beforeImg.src = beforeSrc;
+    afterImg.src = afterSrc;
+    if (afterClip) afterClip.style.clipPath = 'inset(0 0 0 0%)';
+    if (handle) handle.style.display = 'none';
+  } else if (mode === 'overlay') {
+    beforeImg.src = beforeSrc;
+    afterImg.src = overlaySrc;
+    if (afterClip) afterClip.style.clipPath = 'inset(0 0 0 0%)';
+    if (handle) handle.style.display = 'none';
+  }
+}
+
+function updateDdaReviewSummary(regions) {
+  const el = document.getElementById('dda-review-summary');
+  if (!el) return;
+  const counts = { pending: 0, confirmed: 0, false_positive: 0, submitted: 0 };
+  (regions || []).forEach((r) => {
+    const s = r.reviewStatus || 'pending';
+    counts[s] = (counts[s] || 0) + 1;
+  });
+  el.textContent = `Review: ${counts.confirmed} confirmed · ${counts.false_positive} false positive · ${counts.pending} pending`;
+}
+
+function setupDdaReviewBar(runId, regions) {
+  const exportAll = document.getElementById('dda-export-csv');
+  const exportConfirmed = document.getElementById('dda-export-confirmed');
+  const submitBtn = document.getElementById('dda-submit-dept');
+  if (exportAll) exportAll.href = `/api/dda/reports/${runId}/export.csv`;
+  if (exportConfirmed) exportConfirmed.href = `/api/dda/reports/${runId}/export.csv?confirmed=1`;
+  updateDdaReviewSummary(regions);
+
+  submitBtn?.replaceWith(submitBtn.cloneNode(true));
+  document.getElementById('dda-submit-dept')?.addEventListener('click', async () => {
+    try {
+      const res = await ddaApi('POST', `/api/dda/reports/${runId}/submit`);
+      if (typeof showDdaSuccess === 'function') showDdaSuccess(res.message || 'Submitted.');
+      if (res.downloadUrl && res.mode === 'file') {
+        window.open(res.downloadUrl, '_blank');
+      }
+      const data = await ddaApi('GET', `/api/history/${runId}`);
+      showDdaResult(data);
+    } catch (err) {
+      if (typeof showDdaError === 'function') showDdaError(err.message || 'Submit failed');
+    }
+  });
+}
+
+async function patchRegionReview(runId, regionId, reviewStatus) {
+  const res = await ddaApi('PATCH', `/api/dda/reports/${runId}/regions/${regionId}`, {
+    body: JSON.stringify({ reviewStatus }),
+  });
+  return res.region;
+}
+
 function showDdaResult(data) {
   const modal = document.getElementById('result-modal');
   const statsEl = document.getElementById('result-stats');
@@ -23,6 +115,9 @@ function showDdaResult(data) {
   if (!modal || !statsEl) return;
 
   ddaCurrentResult = data;
+  ddaSelectedRegionId = null;
+  document.getElementById('dda-locate-btn')?.setAttribute('disabled', 'disabled');
+
   if (titleEl) titleEl.textContent = data.title || 'Detection Result';
 
   const stats = data.statistics || {};
@@ -57,17 +152,19 @@ function showDdaResult(data) {
     <div class="stat-box"><div class="value value-sm">${regOk === true ? 'OK' : regOk === false ? 'Weak' : '—'}</div><div class="label">Alignment</div></div>
   `;
 
+  const overlaySrc = data.overlayBase64Png
+    ? 'data:image/png;base64,' + data.overlayBase64Png
+    : (data.overlayUrl || '');
+  const beforeSrc = data.beforeFullUrl || data.beforeThumbUrl || '';
+  const afterSrc = data.afterThumbUrl || data.afterFullUrl || beforeSrc;
+
+  ddaViewUrls = { before: beforeSrc, after: afterSrc, overlay: overlaySrc };
+  setDdaViewMode('slider');
+
   const beforeImg = document.getElementById('compare-before-img');
   const afterImg = document.getElementById('compare-after-img');
   if (!beforeImg || !afterImg) return;
 
-  if (data.overlayBase64Png) {
-    afterImg.src = 'data:image/png;base64,' + data.overlayBase64Png;
-  } else {
-    afterImg.src = data.overlayUrl || '';
-  }
-  beforeImg.src = data.beforeFullUrl || data.beforeThumbUrl || '';
-
   let loaded = 0;
   const onReady = () => {
     if (++loaded >= 2) {
@@ -91,6 +188,8 @@ function showDdaResult(data) {
     const stories = r.estimatedStories != null ? r.estimatedStories : '—';
     const height = r.estimatedHeightM != null ? r.estimatedHeightM + ' m' : '—';
     const stage = r.constructionStage && r.constructionStage !== 'Unknown' ? r.constructionStage : '—';
+    const reviewStatus = r.reviewStatus || 'pending';
+    const locked = reviewStatus === 'submitted';
     tr.innerHTML = `
       <td>${r.id}</td>
       <td>${r.objectType}</td>
@@ -104,12 +203,19 @@ function showDdaResult(data) {
       <td>${stories}</td>
       <td>${height}</td>
       <td>${stage}</td>
+      <td><span class="${reviewBadgeClass(reviewStatus)}">${reviewStatus.replace('_', ' ')}</span></td>
+      <td class="dda-review-btns">
+        <button type="button" class="btn btn-secondary btn-sm btn-review-ok" data-action="confirmed" ${locked ? 'disabled' : ''} title="Confirm">✓</button>
+        <button type="button" class="btn btn-secondary btn-sm btn-review-fp" data-action="false_positive" ${locked ? 'disabled' : ''} title="False positive">✗</button>
+        <button type="button" class="btn btn-secondary btn-sm btn-review-locate" title="Locate">◎</button>
+      </td>
     `;
     return tr;
   });
 
   ddaRegionPage = 0;
   renderDdaRegionPage();
+  if (data.id) setupDdaReviewBar(data.id, regions);
   openDdaResultModal();
 }
 
@@ -127,6 +233,7 @@ function renderDdaRegionPage() {
   tbody.innerHTML = '';
   pageRows.forEach((tr) => tbody.appendChild(tr));
   setupDdaRegionHover(tbody, pageData);
+  setupDdaReviewButtons(tbody, pageData);
 
   if (!pag) return;
   pag.innerHTML = '';
@@ -153,48 +260,123 @@ function renderDdaRegionPage() {
   pag.appendChild(next);
 }
 
+function setupDdaReviewButtons(tbody, regions) {
+  const runId = ddaCurrentResult?.id;
+  if (!runId) return;
+
+  tbody.querySelectorAll('tr[data-region-id]').forEach((tr) => {
+    tr.querySelector('.btn-review-ok')?.addEventListener('click', async (e) => {
+      e.stopPropagation();
+      const id = parseInt(tr.dataset.regionId, 10);
+      try {
+        const updated = await patchRegionReview(runId, id, 'confirmed');
+        const idx = ddaRegionList.findIndex((x) => x.id === id);
+        if (idx >= 0) ddaRegionList[idx] = { ...ddaRegionList[idx], ...updated };
+        renderDdaRegionPage();
+        updateDdaReviewSummary(ddaRegionList);
+      } catch (err) {
+        if (typeof showDdaError === 'function') showDdaError(err.message);
+      }
+    });
+    tr.querySelector('.btn-review-fp')?.addEventListener('click', async (e) => {
+      e.stopPropagation();
+      const id = parseInt(tr.dataset.regionId, 10);
+      try {
+        const updated = await patchRegionReview(runId, id, 'false_positive');
+        const idx = ddaRegionList.findIndex((x) => x.id === id);
+        if (idx >= 0) ddaRegionList[idx] = { ...ddaRegionList[idx], ...updated };
+        renderDdaRegionPage();
+        updateDdaReviewSummary(ddaRegionList);
+      } catch (err) {
+        if (typeof showDdaError === 'function') showDdaError(err.message);
+      }
+    });
+    tr.querySelector('.btn-review-locate')?.addEventListener('click', (e) => {
+      e.stopPropagation();
+      selectAndLocateRegion(tr, regions);
+    });
+  });
+}
+
+function selectAndLocateRegion(tr, regions) {
+  const id = parseInt(tr.dataset.regionId, 10);
+  const r = regions.find((x) => x.id === id);
+  ddaSelectedRegionId = id;
+  document.getElementById('dda-locate-btn')?.removeAttribute('disabled');
+  tr.closest('tbody')?.querySelectorAll('tr').forEach((row) => row.classList.remove('region-selected'));
+  tr.classList.add('region-selected');
+  locateRegionOnViewer(r);
+}
+
+function locateRegionOnViewer(r) {
+  const overlay = document.getElementById('region-highlight-overlay');
+  if (!r || !r.bbox || !overlay) return;
+  overlay.innerHTML = '';
+  const box = document.createElement('div');
+  box.className = 'highlight-box highlight-pulse';
+  const imgEl = document.getElementById('compare-after-img');
+  const slider = document.getElementById('compare-slider');
+  const wrapper = document.getElementById('zoom-wrapper');
+  if (!imgEl || !slider || !imgEl.naturalWidth) return;
+  const rw = slider.offsetWidth;
+  const rh = slider.offsetHeight;
+  const imgW = imgEl.naturalWidth || 1;
+  const imgH = imgEl.naturalHeight || 1;
+  const scale = Math.min(rw / imgW, rh / imgH);
+  const drawW = imgW * scale;
+  const drawH = imgH * scale;
+  const offsetX = (rw - drawW) / 2;
+  const offsetY = (rh - drawH) / 2;
+  box.style.left = (offsetX + r.bbox.x * scale) + 'px';
+  box.style.top = (offsetY + r.bbox.y * scale) + 'px';
+  box.style.width = (r.bbox.w * scale) + 'px';
+  box.style.height = (r.bbox.h * scale) + 'px';
+  overlay.appendChild(box);
+
+  if (wrapper && r.bbox.w > 0 && r.bbox.h > 0) {
+    const cx = r.bbox.x + r.bbox.w / 2;
+    const cy = r.bbox.y + r.bbox.h / 2;
+    ddaZoom = Math.min(DDA_ZOOM_MAX, Math.max(1.5, Math.min(drawW / (r.bbox.w * scale * 2.5), drawH / (r.bbox.h * scale * 2.5))));
+    applyDdaZoom();
+    wrapper.scrollLeft = Math.max(0, (offsetX + cx * scale) * ddaZoom - wrapper.clientWidth / 2);
+    wrapper.scrollTop = Math.max(0, (offsetY + cy * scale) * ddaZoom - wrapper.clientHeight / 2);
+  }
+}
+
 function setupDdaRegionHover(tbody, regions) {
   const overlay = document.getElementById('region-highlight-overlay');
   if (!overlay) return;
 
   function showRegionHighlight(r, zoomTo) {
     if (!r || !r.bbox) return;
-    overlay.innerHTML = '';
-    const box = document.createElement('div');
-    box.className = 'highlight-box';
-    const imgEl = document.getElementById('compare-after-img');
-    const slider = document.getElementById('compare-slider');
-    const wrapper = document.getElementById('zoom-wrapper');
-    if (!imgEl || !slider || !imgEl.naturalWidth) return;
-    const rw = slider.offsetWidth;
-    const rh = slider.offsetHeight;
-    const imgW = imgEl.naturalWidth || 1;
-    const imgH = imgEl.naturalHeight || 1;
-    const scale = Math.min(rw / imgW, rh / imgH);
-    const drawW = imgW * scale;
-    const drawH = imgH * scale;
-    const offsetX = (rw - drawW) / 2;
-    const offsetY = (rh - drawH) / 2;
-    box.style.left = (offsetX + r.bbox.x * scale) + 'px';
-    box.style.top = (offsetY + r.bbox.y * scale) + 'px';
-    box.style.width = (r.bbox.w * scale) + 'px';
-    box.style.height = (r.bbox.h * scale) + 'px';
-    overlay.appendChild(box);
-
-    if (zoomTo && wrapper && r.bbox.w > 0 && r.bbox.h > 0) {
-      const cx = r.bbox.x + r.bbox.w / 2;
-      const cy = r.bbox.y + r.bbox.h / 2;
-      const targetZoom = Math.min(DDA_ZOOM_MAX, Math.max(1.5, Math.min(drawW / (r.bbox.w * scale * 2.5), drawH / (r.bbox.h * scale * 2.5))));
-      ddaZoom = targetZoom;
-      applyDdaZoom();
-      wrapper.scrollLeft = Math.max(0, (offsetX + cx * scale) * ddaZoom - wrapper.clientWidth / 2);
-      wrapper.scrollTop = Math.max(0, (offsetY + cy * scale) * ddaZoom - wrapper.clientHeight / 2);
+    if (zoomTo) locateRegionOnViewer(r);
+    else {
+      overlay.innerHTML = '';
+      const box = document.createElement('div');
+      box.className = 'highlight-box';
+      const imgEl = document.getElementById('compare-after-img');
+      const slider = document.getElementById('compare-slider');
+      if (!imgEl || !slider || !imgEl.naturalWidth) return;
+      const rw = slider.offsetWidth;
+      const rh = slider.offsetHeight;
+      const imgW = imgEl.naturalWidth || 1;
+      const imgH = imgEl.naturalHeight || 1;
+      const scale = Math.min(rw / imgW, rh / imgH);
+      const drawW = imgW * scale;
+      const drawH = imgH * scale;
+      const offsetX = (rw - drawW) / 2;
+      const offsetY = (rh - drawH) / 2;
+      box.style.left = (offsetX + r.bbox.x * scale) + 'px';
+      box.style.top = (offsetY + r.bbox.y * scale) + 'px';
+      box.style.width = (r.bbox.w * scale) + 'px';
+      box.style.height = (r.bbox.h * scale) + 'px';
+      overlay.appendChild(box);
     }
   }
 
   tbody.querySelectorAll('tr[data-region-id]').forEach((tr) => {
     tr.style.cursor = 'pointer';
-    tr.title = 'Click to locate on map';
+    tr.title = 'Click to select and locate';
     tr.addEventListener('mouseenter', () => {
       const id = parseInt(tr.dataset.regionId, 10);
       const r = regions.find((x) => x.id === id);
@@ -204,15 +386,9 @@ function setupDdaRegionHover(tbody, regions) {
     });
     tr.addEventListener('mouseleave', () => {
       tr.classList.remove('region-hover');
-      overlay.innerHTML = '';
-    });
-    tr.addEventListener('click', () => {
-      const id = parseInt(tr.dataset.regionId, 10);
-      const r = regions.find((x) => x.id === id);
-      tbody.querySelectorAll('tr').forEach((row) => row.classList.remove('region-selected'));
-      tr.classList.add('region-selected');
-      showRegionHighlight(r, true);
+      if (!tr.classList.contains('region-selected')) overlay.innerHTML = '';
     });
+    tr.addEventListener('click', () => selectAndLocateRegion(tr, regions));
   });
 }
 
@@ -234,6 +410,7 @@ function closeDdaResultModal() {
 }
 
 function resetDdaCompareSlider() {
+  if (ddaViewMode !== 'slider') return;
   const ac = document.getElementById('compare-after-clip');
   const h = document.getElementById('compare-handle');
   if (ac) ac.style.clipPath = 'inset(0 0 0 50%)';
@@ -260,20 +437,31 @@ function initDdaCompareSlider() {
   let isDragging = false;
 
   function updatePosition(clientX) {
+    if (ddaViewMode !== 'slider') return;
     const rect = slider.getBoundingClientRect();
     const pct = Math.max(0, Math.min(100, ((clientX - rect.left) / rect.width) * 100));
     document.getElementById('compare-after-clip').style.clipPath = `inset(0 0 0 ${pct}%)`;
     document.getElementById('compare-handle').style.left = pct + '%';
   }
 
-  slider.addEventListener('mousedown', (e) => { e.preventDefault(); isDragging = true; updatePosition(e.clientX); });
+  slider.addEventListener('mousedown', (e) => { if (ddaViewMode !== 'slider') return; e.preventDefault(); isDragging = true; updatePosition(e.clientX); });
   document.addEventListener('mousemove', (e) => { if (isDragging) updatePosition(e.clientX); });
   document.addEventListener('mouseup', () => { isDragging = false; });
-  slider.addEventListener('touchstart', (e) => { isDragging = true; updatePosition(e.touches[0].clientX); }, { passive: true });
+  slider.addEventListener('touchstart', (e) => { if (ddaViewMode !== 'slider') return; isDragging = true; updatePosition(e.touches[0].clientX); }, { passive: true });
   document.addEventListener('touchmove', (e) => { if (isDragging) updatePosition(e.touches[0].clientX); }, { passive: true });
   document.addEventListener('touchend', () => { isDragging = false; });
 }
 
+function initDdaViewToolbar() {
+  document.querySelectorAll('.dda-view-btn').forEach((btn) => {
+    btn.addEventListener('click', () => setDdaViewMode(btn.dataset.view));
+  });
+  document.getElementById('dda-locate-btn')?.addEventListener('click', () => {
+    const r = ddaRegionList.find((x) => x.id === ddaSelectedRegionId);
+    if (r) locateRegionOnViewer(r);
+  });
+}
+
 function initDdaZoom() {
   document.getElementById('zoom-in')?.addEventListener('click', () => {
     ddaZoom = Math.min(DDA_ZOOM_MAX, ddaZoom + DDA_ZOOM_STEP);
@@ -305,3 +493,4 @@ document.addEventListener('keydown', (e) => {
 
 initDdaCompareSlider();
 initDdaZoom();
+initDdaViewToolbar();

templates/index_dda.html

@@ -5,7 +5,7 @@
   <meta name="viewport" content="width=device-width, initial-scale=1.0" />
   <title>DDA Change Detection</title>
   <link rel="stylesheet" href="/static/css/style.css?v=30" />
-  <link rel="stylesheet" href="/static/css/dda.css?v=9" />
+  <link rel="stylesheet" href="/static/css/dda.css?v=10" />
 </head>
 <body>
   <div class="app dda-app">
@@ -189,6 +189,14 @@
         </button>
       </div>
       <div class="result-stats" id="result-stats"></div>
+      <div class="dda-view-toolbar">
+        <span class="dda-view-label">View:</span>
+        <button type="button" class="dda-view-btn active" data-view="slider">Slider</button>
+        <button type="button" class="dda-view-btn" data-view="t1">Base (T1)</button>
+        <button type="button" class="dda-view-btn" data-view="t2">Comparison (T2)</button>
+        <button type="button" class="dda-view-btn" data-view="overlay">Overlay</button>
+        <button type="button" class="btn btn-secondary btn-sm" id="dda-locate-btn" title="Locate selected region" disabled>Locate</button>
+      </div>
       <div class="zoom-slider-section">
         <div class="zoom-controls">
           <button type="button" class="btn btn-zoom" id="zoom-out" title="Zoom out" aria-label="Zoom out">−</button>
@@ -216,6 +224,14 @@
           </div>
         </div>
       </div>
+      <div class="dda-review-bar" id="dda-review-bar">
+        <span class="dda-review-summary dim" id="dda-review-summary"></span>
+        <div class="dda-review-actions">
+          <a class="btn btn-secondary btn-sm" id="dda-export-csv" href="#" download>Export CSV</a>
+          <a class="btn btn-secondary btn-sm" id="dda-export-confirmed" href="#" download>Export confirmed</a>
+          <button type="button" class="btn btn-primary btn-sm" id="dda-submit-dept">Submit confirmed</button>
+        </div>
+      </div>
       <div class="regions-table-wrap">
         <table class="regions-table" id="regions-table">
           <thead>
@@ -232,6 +248,8 @@
               <th>Stories</th>
               <th>Height</th>
               <th>Stage</th>
+              <th>Review</th>
+              <th>Actions</th>
             </tr>
           </thead>
           <tbody id="regions-tbody"></tbody>
@@ -243,7 +261,7 @@
 
   <script src="/static/js/dda/app.js?v=11"></script>
   <script src="/static/js/dda/library.js?v=6"></script>
-  <script src="/static/js/dda/result.js?v=4"></script>
+  <script src="/static/js/dda/result.js?v=5"></script>
   <script src="/static/js/dda/compare.js?v=8"></script>
   <script src="/static/js/dda/reports.js?v=3"></script>
   <script src="/static/js/dda/notifications.js?v=1"></script>