-
Notifications
You must be signed in to change notification settings - Fork 2
Description
The application was successfully tested on a local machine. The following points of feedback were identified:
-
Layout: The current font sizes and/or component layout may require zooming in within the browser. Adjusting the layout and typography would improve usability and accessibility across screen sizes.
-
Timeout with Large Datasets: When handling large datasets, the application encounters a timeout issue. Specifically, the following error was observed:
2025/04/16 22:34:20 [error] 36#36: *18 upstream timed out (110: Operation timed out) while reading response header from upstream, client: 172.21.0.1, server: , request: "GET /api/raw-data-overview HTTP/1.1", upstream: "http://172.21.0.10:8000/api/raw-data-overview", host: "localhost:3000", referrer: "http://localhost:3000/"
This suggests a need for performance optimization or an increased timeout threshold in the backend configuration.
-
Flexible Data and Mask Source Configuration: Currently, the application assumes that both image data and masks are hosted on the same Tiled server. To increase flexibility and support independent sources, it would be beneficial to support separate environment variables:
TILED_URI_MASK= TILED_API_KEY_MASK= TILED_URI_IMAGES= TILED_API_KEY_IMAGES=This would allow the application to connect to different endpoints for images and masks as needed.
-
The repo is missing documentation, e.g. how to start the application, what does it do, etc.