feat(frontend,backed): adds review maps shows all images; review map show rejection_reason

Author: konishon

src/backend/app/images/image_classification.py

@@ -1027,7 +1027,7 @@ async def get_batch_map_data(
     ) -> dict:
         """Get map data for batch review visualization.
 
-        Returns task geometries and image point locations as GeoJSON.
+        Returns task geometries and all images as GeoJSON (with/without GPS coordinates).
         """
         # Get all task IDs that have images in this batch
         task_ids_query = """
@@ -1075,47 +1075,68 @@ async def get_batch_map_data(
                     }
                 )
 
-        # Get image locations as GeoJSON points
-        images_query = """
+        # Get all images with or without GPS data
+        all_images_query = """
             SELECT
                 id,
                 filename,
                 status,
+                rejection_reason,
                 task_id,
                 ST_X(location::geometry) as longitude,
                 ST_Y(location::geometry) as latitude
             FROM project_images
             WHERE batch_id = %(batch_id)s
             AND project_id = %(project_id)s
-            AND location IS NOT NULL
+            ORDER BY uploaded_at DESC
         """
 
         async with db.cursor(row_factory=dict_row) as cur:
             await cur.execute(
-                images_query,
+                all_images_query,
                 {"batch_id": str(batch_id), "project_id": str(project_id)},
             )
-            images = await cur.fetchall()
+            all_images = await cur.fetchall()
+
+        # Build GeoJSON features for each image
+        images_features = []
+        located_count = 0
+        unlocated_count = 0
+
+        for img in all_images:
+            properties = {
+                "id": str(img["id"]),
+                "filename": img["filename"],
+                "status": img["status"],
+                "task_id": str(img["task_id"]) if img["task_id"] else None,
+                "rejection_reason": img["rejection_reason"],
+            }
 
-        images_geojson = {
-            "type": "FeatureCollection",
-            "features": [
-                {
+            # Add Point geometry if GPS data exists
+            if img["longitude"] is not None and img["latitude"] is not None:
+                feature = {
                     "type": "Feature",
                     "geometry": {
                         "type": "Point",
                         "coordinates": [img["longitude"], img["latitude"]],
                     },
-                    "properties": {
-                        "id": str(img["id"]),
-                        "filename": img["filename"],
-                        "status": img["status"],
-                        "task_id": str(img["task_id"]) if img["task_id"] else None,
-                    },
+                    "properties": properties,
                 }
-                for img in images
-                if img["longitude"] is not None and img["latitude"] is not None
-            ],
+                located_count += 1
+            else:
+                # Add feature with null geometry for images without GPS
+                feature = {
+                    "type": "Feature",
+                    "geometry": None,
+                    "properties": properties,
+                }
+                unlocated_count += 1
+
+            images_features.append(feature)
+
+        images_geojson = {
+            "type": "FeatureCollection",
+            "features": images_features,
         }
 
         return {
@@ -1124,6 +1145,8 @@ async def get_batch_map_data(
             "images": images_geojson,
             "total_tasks": len(tasks_geojson["features"]),
             "total_images": len(images_geojson["features"]),
+            "total_images_with_gps": located_count,
+            "total_images_without_gps": unlocated_count,
         }
 
     @staticmethod

src/backend/app/migrations/versions/add_image_classification.py

@@ -0,0 +1,60 @@
+"""add image classification fields
+
+Revision ID: add_image_classification
+Revises: 001_project_images, 7389d0d528c3
+Create Date: 2025-01-06
+
+"""
+
+from alembic import op
+import sqlalchemy as sa
+from sqlalchemy.dialects import postgresql
+
+revision = "add_image_classification"
+down_revision = ("001_project_images", "7389d0d528c3")
+branch_labels = None
+depends_on = None
+
+
+def upgrade():
+    # Add batch_id column
+    try:
+        op.add_column(
+            "project_images",
+            sa.Column("batch_id", postgresql.UUID(as_uuid=True), nullable=True),
+        )
+    except Exception:
+        # Column might already exist
+        pass
+
+    # Add rejection_reason column
+    try:
+        op.add_column(
+            "project_images", sa.Column("rejection_reason", sa.Text(), nullable=True)
+        )
+    except Exception:
+        pass
+
+    # Add sharpness_score column
+    try:
+        op.add_column(
+            "project_images", sa.Column("sharpness_score", sa.Float(), nullable=True)
+        )
+    except Exception:
+        pass
+
+    # Create indexes
+    op.execute(
+        "CREATE INDEX IF NOT EXISTS idx_project_images_batch_id ON project_images (batch_id)"
+    )
+    op.execute(
+        "CREATE INDEX IF NOT EXISTS idx_project_images_batch_status ON project_images (batch_id, status)"
+    )
+
+
+def downgrade():
+    op.drop_index("idx_project_images_batch_status", table_name="project_images")
+    op.drop_index("idx_project_images_batch_id", table_name="project_images")
+    op.drop_column("project_images", "sharpness_score")
+    op.drop_column("project_images", "rejection_reason")
+    op.drop_column("project_images", "batch_id")