From 5967e9a4e788eed606b06c05886ec79bbbb3f0c7 Mon Sep 17 00:00:00 2001
From: "David P. Federl" <david-peter.federl@stud.uni-due.de>
Date: Mon, 1 Jul 2024 22:18:39 +0200
Subject: [PATCH 1/2] fix(hw-test-script): update expected URL schema

---
 .idea/misc.xml                       |  4 +-
 bitfile_scripts/HW-Test_Webserver.py | 89 ++++++++++++----------------
 2 files changed, 38 insertions(+), 55 deletions(-)

diff --git a/.idea/misc.xml b/.idea/misc.xml
index bbbb6073..9d859815 100644
--- a/.idea/misc.xml
+++ b/.idea/misc.xml
@@ -1,10 +1,8 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <project version="4">
   <component name="Black">
-    <option name="enabledOnReformat" value="true" />
     <option name="executionMode" value="BINARY" />
-    <option name="pathToExecutable" value="/usr/bin/black" />
-    <option name="sdkName" value="Python 3.8 (elastic-ai.runtime.enV5-acrh)" />
+    <option name="sdkName" value="Python 3.12 (elastic-ai.runtime.enV5)" />
   </component>
   <component name="CMakePythonSetting">
     <option name="pythonIntegrationState" value="NO" />
diff --git a/bitfile_scripts/HW-Test_Webserver.py b/bitfile_scripts/HW-Test_Webserver.py
index da43989d..50b53758 100644
--- a/bitfile_scripts/HW-Test_Webserver.py
+++ b/bitfile_scripts/HW-Test_Webserver.py
@@ -1,54 +1,37 @@
-import os
+from pathlib import Path
 from io import BytesIO
-from flask import Flask, send_file
+from flask import Flask, send_file, request
 
-bytes_per_request = 512  # should be matching with the page size of the flash
-
-app = Flask(__name__)
+app: Flask = Flask(__name__)
+bin_file_source_dir: Path = Path(__file__).resolve().parent
 
 
 def print_bytearray(arr: bytes):
     print("0x" + " 0x".join("%02X" % b for b in arr))
 
 
-def read_slice(position: int, filename: str) -> bytes:
+def read_slice(position: int, filename: Path, max_length: int = 512) -> bytes:
     with open(filename, "rb") as file:
-        file.seek(position * bytes_per_request, 0)
-        chunk: bytes = file.read(bytes_per_request)
-        # print(f"Chunk {position}:")
-        # print_bytearray(chunk)
+        file.seek(position * max_length, 0)
+        chunk: bytes = file.read(max_length)
     return chunk
 
 
-@app.route("/getfast/<position>")
-def get_file_fast(position: str):
-    """
-    Using positional arguments as parameter did not work!
-    """
-    buffer = BytesIO()
-    buffer.write(
-        read_slice(
-            int(position), "bitfiles/env5_bitfiles/blink/blink_fast/led_test.bin"
-        )
-    )
-    buffer.seek(0)
-    return send_file(
-        buffer,
-        as_attachment=True,
-        download_name="bitfile.bin",
-        mimetype="application/octet-stream",
-    )
+@app.route("/getfast")
+def get_file_fast():
+    chunk_size = request.args.get("chunkMaxSize", type=int)
+    chunk_number = request.args.get("chunkNumber", type=int)
 
+    print(f"chunk_size: {chunk_size}, chunk_number: {chunk_number}")
 
-@app.route("/getslow/<position>")
-def get_file_slow(position: str):
-    """
-    Using positional arguments as parameter did not work!
-    """
     buffer = BytesIO()
     buffer.write(
         read_slice(
-            int(position), "bitfiles/env5_bitfiles/blink/blink_slow/led_test.bin"
+            chunk_number,
+            bin_file_source_dir.joinpath(
+                "bitfiles/env5_bitfiles/blink/blink_fast/led_test.bin"
+            ),
+            chunk_size,
         )
     )
     buffer.seek(0)
@@ -60,15 +43,19 @@ def get_file_slow(position: str):
     )
 
 
-@app.route("/getecho/<position>")
-def get_file_echo(position: str):
-    """
-    Using positional arguments as parameter did not work!
-    """
+@app.route("/getslow")
+def get_file_slow():
+    chunk_size = request.args.get("chunkMaxSize", type=int)
+    chunk_number = request.args.get("chunkNumber", type=int)
+
     buffer = BytesIO()
     buffer.write(
         read_slice(
-            int(position), "bitfiles/env5_bitfiles/echo_server/env5_top_reconfig.bin"
+            chunk_number,
+            bin_file_source_dir.joinpath(
+                "bitfiles/env5_bitfiles/blink/blink_slow/led_test.bin"
+            ),
+            chunk_size,
         )
     )
     buffer.seek(0)
@@ -80,15 +67,19 @@ def get_file_echo(position: str):
     )
 
 
-@app.route("/getconfig/<position>")
-def get_bin_file(position: str):
-    """
-    Using positional arguments as parameter did not work!
-    """
+@app.route("/getecho")
+def get_file_echo():
+    chunk_size = request.args.get("chunkMaxSize", type=int)
+    chunk_number = request.args.get("chunkNumber", type=int)
+
     buffer = BytesIO()
     buffer.write(
         read_slice(
-            int(position), "./../config.bin"
+            chunk_number,
+            bin_file_source_dir.joinpath(
+                "bitfiles/env5_bitfiles/echo_server/env5_top_reconfig.bin"
+            ),
+            chunk_size,
         )
     )
     buffer.seek(0)
@@ -100,12 +91,6 @@ def get_bin_file(position: str):
     )
 
 
-@app.route("/length")
-def get_bin_file_length():
-    size = os.path.getsize("./../config.bin")
-    return str(size)
-
-
 @app.route("/check")
 def check_server_available():
     return "AVAILABLE\0"

From fd234ef7fc4d99a29830764708cf44df884362bc Mon Sep 17 00:00:00 2001
From: "David P. Federl" <david-peter.federl@stud.uni-due.de>
Date: Mon, 1 Jul 2024 22:19:04 +0200
Subject: [PATCH 2/2] build(cmake): disable cp to out directory

---
 helpers.cmake | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/helpers.cmake b/helpers.cmake
index 6fe859bf..9e3e5996 100644
--- a/helpers.cmake
+++ b/helpers.cmake
@@ -72,12 +72,12 @@ function(create_enV5_executable target)
     pico_enable_stdio_uart(${target} 0)
     # create map/bin/hex/uf2 file etc.
     pico_add_uf2_output(${target})
-    # copy u2f files after build to out directory
-    file(RELATIVE_PATH relative_path ${CMAKE_SOURCE_DIR} ${CMAKE_CURRENT_LIST_DIR})
-    add_custom_command(TARGET ${target} POST_BUILD
-            COMMAND ${CMAKE_COMMAND} -E copy
-            ${CMAKE_BINARY_DIR}/${relative_path}/${target}.uf2
-            ${CMAKE_SOURCE_DIR}/out/${CMAKE_BUILD_TYPE}/${relative_path}/${target}.uf2)
+    #    # copy u2f files after build to out directory
+    #    file(RELATIVE_PATH relative_path ${CMAKE_SOURCE_DIR} ${CMAKE_CURRENT_LIST_DIR})
+    #    add_custom_command(TARGET ${target} POST_BUILD
+    #            COMMAND ${CMAKE_COMMAND} -E copy
+    #            ${CMAKE_BINARY_DIR}/${relative_path}/${target}.uf2
+    #            ${CMAKE_SOURCE_DIR}/out/${CMAKE_BUILD_TYPE}-Rev${REVISION}/${relative_path}/${target}.uf2)
 endfunction()
 
 function(add_dependency_graph)