Merge pull request #219 from hotosm/develop

Production Release

src/backend/app/projects/project_schemas.py

@@ -283,18 +283,38 @@ async def all(
         limit: int = 100,
     ):
         """
-        Get all projects. Optionally filter by the project creator (user).
+        Get all projects, count total tasks and task states (ongoing, completed, etc.).
+        Optionally filter by the project creator (user).
         """
         async with db.cursor(row_factory=dict_row) as cur:
             await cur.execute(
                 """
                 SELECT
-                    id, slug, name, description, per_task_instructions,
-                    ST_AsGeoJSON(outline)::jsonb AS outline,
-                    requires_approval_from_manager_for_locking
-                FROM projects
-                WHERE (author_id = COALESCE(%(user_id)s, author_id))
-                ORDER BY created_at DESC
+                    p.id, p.slug, p.name, p.description, p.per_task_instructions,
+                    ST_AsGeoJSON(p.outline)::jsonb AS outline,
+                    p.requires_approval_from_manager_for_locking,
+
+                    -- Count total tasks for each project
+                    COUNT(t.id) AS total_task_count,
+
+                    -- Count based on the latest state of tasks
+                    COUNT(CASE WHEN te.state = 'LOCKED_FOR_MAPPING' THEN 1 END) AS ongoing_task_count
+
+                FROM projects p
+                LEFT JOIN tasks t ON t.project_id = p.id
+                LEFT JOIN (
+                    -- Get the latest event per task
+                    SELECT DISTINCT ON (te.task_id)
+                        te.task_id,
+                        te.state,
+                        te.created_at
+                    FROM task_events te
+                    ORDER BY te.task_id, te.created_at DESC
+                ) AS te ON te.task_id = t.id
+
+                WHERE (p.author_id = COALESCE(%(user_id)s, p.author_id))
+                GROUP BY p.id
+                ORDER BY p.created_at DESC
                 OFFSET %(skip)s
                 LIMIT %(limit)s
                 """,
@@ -417,9 +437,10 @@ class ProjectOut(BaseModel):
     outline: Optional[Polygon | Feature | FeatureCollection]
     no_fly_zones: Optional[Polygon | Feature | FeatureCollection | MultiPolygon] = None
     requires_approval_from_manager_for_locking: bool
-    task_count: int = 0
+    total_task_count: int = 0
     tasks: Optional[list[TaskOut]] = []
     image_url: Optional[str] = None
+    ongoing_task_count: Optional[int] = 0
 
     @model_validator(mode="after")
     def set_image_url(cls, values):

src/frontend/src/components/CreateProject/CreateprojectLayout/index.tsx

@@ -25,6 +25,7 @@ import {
 import { convertGeojsonToFile } from '@Utils/convertLayerUtils';
 import prepareFormData from '@Utils/prepareFormData';
 import hasErrorBoundary from '@Utils/hasErrorBoundary';
+import { getFrontOverlap, getSideOverlap, gsdToAltitude } from '@Utils/index';
 
 /**
  * This function looks up the provided map of components to find and return
@@ -86,6 +87,9 @@ const CreateprojectLayout = () => {
   const capturedProjectMap = useTypedSelector(
     state => state.createproject.capturedProjectMap,
   );
+  const imageMergeType = useTypedSelector(
+    state => state.createproject.imageMergeType,
+  );
 
   const initialState: FieldValues = {
     name: '',
@@ -214,13 +218,29 @@ const CreateprojectLayout = () => {
       return;
     }
 
+    // get altitude
+    const agl =
+      measurementType === 'gsd'
+        ? gsdToAltitude(data?.gsd_cm_px)
+        : data?.altitude_from_ground;
+
     const refactoredData = {
       ...data,
       is_terrain_follow: isTerrainFollow,
       requires_approval_from_manager_for_locking:
         requireApprovalFromManagerForLocking === 'required',
       deadline_at: data?.deadline_at ? data?.deadline_at : null,
+      front_overlap:
+        imageMergeType === 'spacing'
+          ? getFrontOverlap(agl, data?.forward_spacing)
+          : data?.front_overlap,
+      side_overlap:
+        imageMergeType === 'spacing'
+          ? getSideOverlap(agl, data?.side_spacing)
+          : data?.side_overlap,
     };
+    delete refactoredData?.forward_spacing;
+    delete refactoredData?.side_spacing;
 
     // remove key
     if (isNoflyzonePresent === 'no')

src/frontend/src/components/CreateProject/DescriptionContents/GenerateTask/index.tsx

@@ -1,3 +1,5 @@
+import { taskGenerationGuidelines } from '@Constants/createProject';
+
 export default function GenerateTask() {
   return (
     <div className="naxatw-px-10 naxatw-py-5">
@@ -6,6 +8,19 @@ export default function GenerateTask() {
         Split the task into smaller chunks based on the given dimensions to
         ensure more efficient and precise data collection and analysis.
       </p>
+      <br />
+      <div>
+        <p className="naxatw-mt-2 naxatw-text-body-md">
+          {taskGenerationGuidelines?.title}
+        </p>
+        <ol className="naxatw-flex naxatw-list-decimal naxatw-flex-col naxatw-gap-1 naxatw-px-2 naxatw-py-2">
+          {taskGenerationGuidelines?.guidelines?.map(item => (
+            <li key={item} className="naxatw-text-left naxatw-text-body-md">
+              {item}
+            </li>
+          ))}
+        </ol>
+      </div>
     </div>
   );
 }

src/frontend/src/components/CreateProject/FormContents/GenerateTask/index.tsx

@@ -1,4 +1,6 @@
 import { useMutation } from '@tanstack/react-query';
+import { useState } from 'react';
+import ErrorMessage from '@Components/common/ErrorMessage';
 import { useTypedDispatch, useTypedSelector } from '@Store/hooks';
 import { FormControl, Label, Input } from '@Components/common/FormUI';
 import { Button } from '@Components/RadixComponents/Button';
@@ -11,6 +13,7 @@ import MapSection from './MapSection';
 
 export default function GenerateTask({ formProps }: { formProps: any }) {
   const dispatch = useTypedDispatch();
+  const [error, setError] = useState('');
 
   const { register, watch } = formProps;
   const dimension = watch('task_split_dimension');
@@ -53,11 +56,15 @@ export default function GenerateTask({ formProps }: { formProps: any }) {
               type="number"
               className="naxatw-mt-1"
               value={dimension}
+              min={50}
+              max={700}
               {...register('task_split_dimension', {
                 required: 'Required',
                 valueAsNumber: true,
               })}
+              onFocus={() => setError('')}
             />
+            {error && <ErrorMessage message={error} />}
           </FormControl>
           <Button
             withLoader
@@ -66,14 +73,16 @@ export default function GenerateTask({ formProps }: { formProps: any }) {
             disabled={!dimension}
             className="naxatw-mt-4 naxatw-bg-red"
             onClick={() => {
-              if (!projectArea) return;
+              if (!projectArea) return () => {};
+              if (dimension < 50 || dimension > 700)
+                return setError('Dimension must in between 50-700');
               dispatch(
                 setCreateProjectState({
                   splitGeojson: null,
                   capturedProjectMap: false,
                 }),
               );
-              mutate(payload);
+              return mutate(payload);
             }}
           >
             Generate Task