Correct pothole deletion function

MapView/config.py

@@ -2,5 +2,3 @@ import os
 
 STORE_HOST = os.environ.get("STORE_HOST") or "localhost"
 STORE_PORT = os.environ.get("STORE_PORT") or 8000
-
-TRACK_ID = int(os.environ.get("TID") or '1')

MapView/main.py

@@ -4,7 +4,6 @@ from kivy_garden.mapview import MapMarker, MapView
 from kivy.clock import Clock
 from lineMapLayer import LineMapLayer
 from datasource import Datasource
-import config
 
 line_layer_colors = [
     [1, 0, 0, 1],
@@ -15,18 +14,31 @@ line_layer_colors = [
     [1, 0, 1, 1],
 ]
 
+
+def get_lat_lon(point: dict[str, float] | list[float] | tuple[float, float]) -> tuple[float, float] | None:
+    if isinstance(point, dict):
+        lat = point.get("lat")
+        lon = point.get("lon")
+    else:
+        lat, lon = point
+
+    if lat is None or lon is None:
+        return None
+    return lat, lon
+
+
 class MapViewApp(App):
     def __init__(self, **kwargs):
         super().__init__(**kwargs)
 
-        self.mapview = None
+        self.mapview: MapView | None = None
         self.datasource = Datasource(user_id=1)
         self.line_layers = dict()
         self.car_markers = dict()
 
         # додати необхідні змінні
-        self.bump_markers = []
+        self.bump_markers: list[MapMarker] = []
-        self.pothole_markers = []
+        self.pothole_markers: list[MapMarker] = []
 
     def on_start(self):
         """
@@ -88,16 +100,17 @@ class MapViewApp(App):
             self.car_markers[user_id].lat = lat
             self.car_markers[user_id].lon = lon
 
-        if user_id == config.TRACK_ID:
+        if user_id == 1:
             self.mapview.center_on(lat, lon)
 
-    def set_pothole_marker(self, point):
+    def map_lat_lon_to_pothole(self, lat: float, lon: float) -> MapMarker | None:
-        if isinstance(point, dict):
+        click_tolerance = self.mapview.zoom * 10
-            lat = point.get("lat")
+        flt = filter(lambda marker: abs(lat - marker.lat) + abs(lon - marker.lon) < click_tolerance,
-            lon = point.get("lon")
+                     self.pothole_markers)
-        else:
+        return next(flt)
-            lat, lon = point
 
+    def set_pothole_marker(self, point):
+        lat, lon = get_lat_lon(point)
         if lat is None or lon is None:
             return
 
@@ -111,12 +124,7 @@ class MapViewApp(App):
         self.pothole_markers.append(marker)
 
     def set_bump_marker(self, point):
-        if isinstance(point, dict):
+        lat, lon = get_lat_lon(point)
-            lat = point.get("lat")
-            lon = point.get("lon")
-        else:
-            lat, lon = point
-
         if lat is None or lon is None:
             return
 
@@ -129,6 +137,16 @@ class MapViewApp(App):
         self.mapview.add_marker(marker)
         self.bump_markers.append(marker)
 
+    def delete_pothole_marker(self, point):
+        lat, lon = get_lat_lon(point)
+        if lat is None or lon is None:
+            return
+
+        pothole = self.map_lat_lon_to_pothole(lat, lon)
+        if pothole:
+            self.mapview.remove_marker(pothole)
+            self.pothole_markers.pop(self.pothole_markers.index(pothole))
+
 
     def build(self):
         """

agent/src/main.py

@@ -16,8 +16,6 @@ def connect_mqtt(broker, port):
             print("Failed to connect {broker}:{port}, return code %d\n", rc)
             exit(rc)  # Stop execution
 
-    logging.info(f"Acting as USER_ID = {config.USER_ID}")
-
     client = mqtt_client.Client()
     client.on_connect = on_connect
     client.connect(broker, port)
@@ -31,14 +29,13 @@ def publish(client, topic, datasource):
         data = datasource.read()
         msg = AggregatedDataSchema().dumps(data)
         result = client.publish(topic, msg)
-        logging.debug(f"Published to {topic}: {msg[:50]}...")
+        logging.info(f"Published to {topic}: {msg[:50]}...")
         status = result[0]
         if status != 0:
-            logging.error(f"Failed to send message to topic {topic}")
+            print(f"Failed to send message to topic {topic}")
 
 
 def run():
-    logging.basicConfig(level = logging.INFO)
     # Prepare mqtt client
     client = connect_mqtt(config.MQTT_BROKER_HOST, config.MQTT_BROKER_PORT)
     # Prepare datasource

store/main.py

@@ -45,7 +45,7 @@ async def websocket_endpoint(websocket: WebSocket, user_id: int):
 
     try:
         # send already available data
-        r = processed_agent_data.select().order_by(processed_agent_data.c.timestamp)
+        r = processed_agent_data.select()
         stored_data = SessionLocal().execute(r).fetchall()
 
         jsonable_data = [{c.name: getattr(i, c.name) for c in processed_agent_data.columns} for i in stored_data]
@@ -96,7 +96,7 @@ async def create_processed_agent_data(data: List[ProcessedAgentData], user_id: i
         created_records = [dict(row._mapping) for row in result.fetchall()]
         session.commit()
 
-        for record in sorted(created_records, key = lambda x: x['timestamp']):
+        for record in created_records:
             await send_data_to_subscribers(jsonable_encoder(record))
         return created_records
     except Exception as err:

utils/check-up.py

@@ -1,5 +1,4 @@
 import sys
-import os
 
 print("Checking for dead containers...")
 
@@ -15,9 +14,6 @@ for i in statuses:
     if not i[status_index:].startswith("Up "):
         service_name = i[name_index:]
         print(f"Crash detected in {service_name}")
-        print(f"docker logs for the container:\n")
-        os.system(f"docker logs {i.split(' ')[0]}")
-        print()
         exit_code = 1
 
 sys.exit(exit_code)