Merge pull request #109 from badgerloop-software/strategy-integration

Strategy integration - I updated the dashboard API’s `get-processed-data` route to support an optional output format parameter, defaulting to an Excel file but now allowing JSON responses, and also fixed a Docker issue by upgrading Poetry to resolve build problems and ensure the GitHub Actions test workflow runs smoothly.

Author: sbasu107

.gitignore

@@ -21,6 +21,7 @@
 /**/.idea
 /**/.vscode
 /recordedData
+venv/
 
 npm-debug.log*
 yarn-debug.log*

.gitmodules

@@ -5,3 +5,6 @@
 [submodule "Backend/file_sync"]
 	path = Backend/file_sync
 	url = git@github.com:badgerloop-software/file_sync
+[submodule "race-profile"]
+	path = race-profile
+	url = git@github.com:badgerloop-software/race-profile.git

Backend/.gitignore

@@ -1,10 +1,11 @@
 node_modules
 dist
 __pycache__/
+venv/
 
 # Data recording
 recordedData/sessions/*.bin
 recordedData/processedData/*.csv
 
 # Python
-*.egg-info
+*.egg-info

Backend/components/record_data.py

@@ -6,27 +6,39 @@
 router = APIRouter()
 
 @router.get("/get-processed-data")
-async def get_processed_data(start_time, end_time):
-    all_keys = list(config.FORMAT.keys())
-    all_keys.remove('tstamp_ms')
-    all_keys.remove('tstamp_sc')
-    all_keys.remove('tstamp_mn')
-    all_keys.remove('tstamp_hr')
-    all_keys.remove('tstamp_unix')
+async def get_processed_data(start_time, end_time, output_format: str = 'xlsx'):
+    try:
+        all_keys = list(config.FORMAT.keys())
+        all_keys.remove('tstamp_ms')
+        all_keys.remove('tstamp_sc')
+        all_keys.remove('tstamp_mn')
+        all_keys.remove('tstamp_hr')
+        all_keys.remove('tstamp_unix')
 
-    result = await db.query_without_aggregation(all_keys, start_time, end_time)
+        result = await db.query_without_aggregation(all_keys, start_time, end_time)
 
+        if output_format == 'json':
+            json_data = result.to_json(orient='records', lines=True, date_format='iso', double_precision=3)
+            return Response(
+                content=json_data,
+                media_type="application/json",
+                # Only needed if you want to force download
+                # headers={"Content-Disposition": "attachment; filename=\"download.json\""}
+            )
+        else:
+            # see https://stackoverflow.com/a/63989481
+            output = io.BytesIO()
+            writer = pd.ExcelWriter(output, engine='xlsxwriter', datetime_format='MM-DD HH:MM:SS')
+            result.to_excel(writer, sheet_name='Sheet1')
 
-    # see https://stackoverflow.com/a/63989481
-    output = io.BytesIO()
-    writer = pd.ExcelWriter(output, engine='xlsxwriter', datetime_format='MM-DD HH:MM:SS')
-    result.to_excel(writer, sheet_name='Sheet1')
-
-    writer.close()
-    xlsx_data = output.getvalue()
-
-
-    return Response(
-        content=xlsx_data,
-        media_type="application/vnd.openxmlformats-officedocument.spreadsheetml.sheet",
-        headers={"Content-Disposition": "attachment; filename=\"download.xlsx\""})
+            writer.close()
+            xlsx_data = output.getvalue()
+            return Response(
+                content=xlsx_data,
+                media_type="application/vnd.openxmlformats-officedocument.spreadsheetml.sheet",
+                headers={"Content-Disposition": "attachment; filename=\"download.xlsx\""})
+    except Exception as e:
+        raise Exception(
+            status_code=500,
+            detail="Unknown error occurred: " + str(e)
+        )

Backend/pyproject.toml

@@ -2,6 +2,8 @@
 name = "chase-car-dashboard-backend"
 version = "5.2.0"
 package-mode = false
+description = ""
+authors = []
 
 [tool.poetry.dependencies]
 python = "^3.10"

Dockerfile

@@ -1,4 +1,5 @@
 FROM nikolaik/python-nodejs:python3.10-nodejs16
+RUN pip install --upgrade pip && pip install poetry
 WORKDIR /chase-car-dashboard
 COPY . .
 RUN npm install