[P] Fix several crashes triggered in one call sequence

MapView/datasource.py

@@ -9,7 +9,6 @@ from config import STORE_HOST, STORE_PORT
 
 # Pydantic models
 class ProcessedAgentData(BaseModel):
-    id: int
     road_state: str
     user_id: int
     x: float
@@ -33,9 +32,9 @@ class ProcessedAgentData(BaseModel):
 
 
 class Datasource:
-    def __init__(self):
+    def __init__(self, user_id: int):
-        self.websocket: Connection | None = None
         self.index = 0
+        self.user_id = user_id
         self.connection_status = None
         self._new_points = []
         self._active_markers = []
@@ -48,12 +47,11 @@ class Datasource:
         return points
 
     async def connect_to_server(self):
-        uri = f"ws://{STORE_HOST}:{STORE_PORT}/ws"
+        uri = f"ws://{STORE_HOST}:{STORE_PORT}/ws/{self.user_id}"
         while True:
             Logger.debug("CONNECT TO SERVER")
             async with websockets.connect(uri) as websocket:
                 self.connection_status = "Connected"
-                self.websocket = websocket
                 try:
                     while True:
                         data = await websocket.recv()
@@ -63,14 +61,8 @@ class Datasource:
                     self.connection_status = "Disconnected"
                     Logger.debug("SERVER DISCONNECT")
 
-    def update_db_record_visibility(self, record_id):
+    def map_lat_lon_to_ProcessedAgentData(self, lat: float, lon: float) -> ProcessedAgentData | None:
-        if self.websocket:
+        distances = tuple((abs(lon - marker.latitude) ** 2 + abs(lat - marker.longitude) ** 2) ** 0.5 for marker in self._active_markers)
-            data = json.dumps({"id": record_id})
-            asyncio.ensure_future(self.websocket.send(data))
-
-    def map_lat_lon_to_processed_agent_data(self, lat: float, lon: float) -> ProcessedAgentData | None:
-        distances = tuple((abs(lon - marker.latitude) ** 2 + abs(lat - marker.longitude) ** 2) ** 0.5 for marker in
-                          self._active_markers)
 
         if len(distances) == 0:
             return None

MapView/main.py

@@ -33,7 +33,7 @@ class MapViewApp(App):
         super().__init__(**kwargs)
 
         self.mapview: MapView | None = None
-        self.datasource = Datasource()
+        self.datasource = Datasource(user_id=1)
         self.line_layers = dict()
         self.car_markers = dict()
 
@@ -109,7 +109,7 @@ class MapViewApp(App):
 
         try:
             return next(flt)
-        except StopIteration:
+        except StopIteration as e:
             return None
 
     def set_pothole_marker(self, point):
@@ -145,14 +145,14 @@ class MapViewApp(App):
         if lat is None or lon is None:
             return
 
-        clicked_marker_data = self.datasource.map_lat_lon_to_processed_agent_data(lat, lon)
+        clicked_marker_data = self.datasource.map_lat_lon_to_ProcessedAgentData(lat, lon)
 
         if not clicked_marker_data:
             return
 
         clicked_marker = self.map_lat_lon_to_marker(clicked_marker_data.latitude, clicked_marker_data.longitude)
 
-        if clicked_marker is None:
+        if clicked_marker == None:
             return
 
         self.mapview.remove_marker(clicked_marker)
@@ -162,14 +162,13 @@ class MapViewApp(App):
         elif clicked_marker in self.bump_markers:
             self.bump_markers.pop(self.bump_markers.index(clicked_marker))
 
-        self.datasource.update_db_record_visibility(clicked_marker_data.id)
-
     def on_touch_down(self, widget, touch):
         if touch.button == "right":
             coordinate = self.mapview.get_latlon_at(touch.x, touch.y, self.mapview.zoom)
             self.delete_pothole_marker(coordinate)
             return True
 
+
     def build(self):
         """
         Ініціалізує мапу MapView(zoom, lat, lon)

store/main.py

@@ -39,18 +39,15 @@ subscriptions: Set[WebSocket] = set()
 
 
 # FastAPI WebSocket endpoint
-@app.websocket("/ws")
+@app.websocket("/ws/{user_id}")
-async def websocket_endpoint(websocket: WebSocket):
+async def websocket_endpoint(websocket: WebSocket, user_id: int):
     await websocket.accept()
 
     subscriptions.add(websocket)
 
     try:
         # send already available data
-        r = processed_agent_data.select() \
+        r = processed_agent_data.select().order_by(processed_agent_data.c.timestamp)
-                                .where(processed_agent_data.c.visible == True) \
-                                .order_by(processed_agent_data.c.timestamp)
-
         session = SessionLocal()
         stored_data = session.execute(r).fetchall()
         session.close() 

store/test_db.py

@@ -1,45 +0,0 @@
-from fastapi.testclient import TestClient
-from main import app, processed_agent_data
-from database import SessionLocal
-from sqlalchemy import select
-from datetime import datetime
-
-client = TestClient(app)
-
-
-def test_create_processed_agent_data():
-    db = SessionLocal()
-    before = db.execute(select(processed_agent_data)).fetchall()
-    before_count = len(before)
-    payload = {
-        "user_id": 123,
-        "data": [
-            {
-                "road_state": "normal",
-                "agent_data": {
-                    "user_id": 999, 
-                    "accelerometer": {
-                        "x": 1.1,
-                        "y": 2.2,
-                        "z": 3.3
-                    },
-                    "gps": {
-                        "latitude": 50.45,
-                        "longitude": 30.52
-                    },
-                    "timestamp": datetime.now().isoformat()
-                }
-            }
-        ]
-    }
-    
-    response = client.post("/processed_agent_data/", json=payload)
-
-    assert response.status_code == 200
-
-    after = db.execute(select(processed_agent_data)).fetchall()
-    after_count = len(after)
-
-    assert after_count == before_count + 1
-
-    db.close()