From f13f964c1ab07f78907fb105e68b8f80d47cd464 Mon Sep 17 00:00:00 2001
From: Arturo Manzoli <arturomanzoli@gmail.com>
Date: Tue, 22 Oct 2024 16:23:09 -0300
Subject: [PATCH 1/2] Mission-planner: Fix loaction of mission's upload and
 download linear progress bar

Signed-off-by: Arturo Manzoli <arturomanzoli@gmail.com>
---
 src/components/widgets/Map.vue    | 12 ++++++++++++
 src/views/MissionPlanningView.vue | 14 ++++++++++++--
 2 files changed, 24 insertions(+), 2 deletions(-)

diff --git a/src/components/widgets/Map.vue b/src/components/widgets/Map.vue
index 82ee8af7c..5d1f023c6 100644
--- a/src/components/widgets/Map.vue
+++ b/src/components/widgets/Map.vue
@@ -81,7 +81,15 @@
     absolute
     bottom
     color="white"
+    :style="`top: ${topProgressBarDisplacement}`"
   />
+  <p
+    v-if="fetchingMission"
+    :style="{ top: topProgressBarDisplacement }"
+    class="absolute left-[7px] mt-4 flex text-md font-bold text-white z-30 drop-shadow-md"
+  >
+    Loading mission...
+  </p>
 </template>
 
 <script setup lang="ts">
@@ -557,6 +565,10 @@ const widgetStore = useWidgetManagerStore()
 const bottomButtonsDisplacement = computed(() => {
   return `${Math.max(-widgetStore.widgetClearanceForVisibleArea(widget.value).bottom, 0)}px`
 })
+
+const topProgressBarDisplacement = computed(() => {
+  return `${Math.max(-widgetStore.widgetClearanceForVisibleArea(widget.value).top, 0)}px`
+})
 </script>
 
 <style>
diff --git a/src/views/MissionPlanningView.vue b/src/views/MissionPlanningView.vue
index 62fb4f76c..c2c1d57fa 100644
--- a/src/views/MissionPlanningView.vue
+++ b/src/views/MissionPlanningView.vue
@@ -227,8 +227,15 @@
       absolute
       bottom
       height="10"
-      color="rgba(0, 110, 255, 0.8)"
+      color="white"
+      :style="`top: ${widgetStore.currentTopBarHeightPixels}px`"
     />
+    <p
+      v-if="uploadingMission"
+      class="fixed top-[58px] left-[7px] flex text-md font-bold text-white z-30 drop-shadow-md"
+    >
+      Uploading mission to vehicle...
+    </p>
   </div>
 </template>
 
@@ -283,6 +290,9 @@ const uploadMissionToVehicle = async (): Promise<void> => {
     missionUploadProgress.value = loadingPerc
   }
   try {
+    if (!vehicleStore.isVehicleOnline) {
+      throw 'Vehicle is not online.'
+    }
     await vehicleStore.uploadMission(missionStore.currentPlanningWaypoints, loadingCallback)
     const message = `Mission upload succeed! Open the Map widget in Flight Mode and click the "play" button to start the mission.`
     showDialog({ variant: 'success', message, timer: 6000 })
@@ -729,7 +739,7 @@ const generateWaypointsFromSurvey = (): void => {
   clearSurveyPath()
   isCreatingSurvey.value = false
 
-  showDialog({ variant: 'success', message: 'Waypoints generated from survey path.', timer: 3000 })
+  showDialog({ variant: 'success', message: 'Waypoints generated from survey path.', timer: 1000 })
 }
 
 // Helper function to connect two waypoints with a polyline

From 26198d3c2c4465ae17762b9159f82d84032559be Mon Sep 17 00:00:00 2001
From: Arturo Manzoli <arturomanzoli@gmail.com>
Date: Tue, 22 Oct 2024 16:24:26 -0300
Subject: [PATCH 2/2] Mission-planner: Fix upload and download timeout for long
 missions and surveys

Signed-off-by: Arturo Manzoli <arturomanzoli@gmail.com>
---
 src/composables/settingsSyncer.ts       | 4 ++--
 src/libs/vehicle/ardupilot/ardupilot.ts | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/composables/settingsSyncer.ts b/src/composables/settingsSyncer.ts
index da705c591..08c4f19a6 100644
--- a/src/composables/settingsSyncer.ts
+++ b/src/composables/settingsSyncer.ts
@@ -136,11 +136,11 @@ export function useBlueOsStorage<T>(key: string, defaultValue: MaybeRef<T>): Rem
       try {
         await setKeyDataOnCockpitVehicleStorage(vehicleAddress, `settings/${username}/${key}`, newValue)
         const message = `Success updating '${key}' on BlueOS.`
-        openSnackbar({ message, duration: 3000, variant: 'success' })
+        openSnackbar({ message, duration: 3000, variant: 'success', closeButton: true })
         console.info(message)
       } catch (fetchError) {
         const message = `Failed updating '${key}' on BlueOS. Will keep trying.`
-        openSnackbar({ message, duration: 3000, variant: 'error' })
+        openSnackbar({ message, duration: 3000, variant: 'error', closeButton: true })
         console.error(message)
         console.error(fetchError)
 
diff --git a/src/libs/vehicle/ardupilot/ardupilot.ts b/src/libs/vehicle/ardupilot/ardupilot.ts
index 2653148c8..2e8269425 100644
--- a/src/libs/vehicle/ardupilot/ardupilot.ts
+++ b/src/libs/vehicle/ardupilot/ardupilot.ts
@@ -1149,7 +1149,7 @@ export abstract class ArduPilotVehicle<Modes> extends Vehicle.AbstractVehicle<Mo
     let lastSeqRequested = -1
     while (missionAck === undefined && !timeoutReachedUpload) {
       await sleep(10)
-      timeoutReachedUpload = new Date().getTime() - initTimeUpload > 10000
+      timeoutReachedUpload = new Date().getTime() - initTimeUpload > 50000
       const lastMissionItemRequestMessage =
         this._messages.get(MAVLinkType.MISSION_REQUEST) || this._messages.get(MAVLinkType.MISSION_REQUEST_INT)
       if (lastMissionItemRequestMessage === undefined) continue