📦 feat(inspect): Add docker build #3136

ashwinvaidya17 · 2025-11-21T09:30:10Z

📝 Description

Add CPU build

TODO

Add GPU build
Add XPU build

✨ Changes

Select what type of change your PR is:

✅ Checklist

Before you submit your pull request, please make sure you have completed the following steps:

📚 I have made the necessary updates to the documentation (if applicable).
🧪 I have written tests that support my changes and prove that my fix is effective or my feature works (if applicable).
🏷️ My PR title follows conventional commit format.

For more information about code review checklists, see the Code Review Checklist.

Mainly based on the UI files and architecture from [Geti Tune](https://github.com/open-edge-platform/training_extensions/tree/develop/ui)

* initial backend commit Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * app -> src Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * Remove empty file Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * move code Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * fix style backend Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * rename media endpoint Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> --------- Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com>

…#2942) add unit tests for endpoints Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com>

These won't be needed yet

…tform#2948) * Use src folder inside run.sh * Set openapi_url path * Update to react 19 * Specify bash language in readme

…edge-platform#2945) * add training + inference endpoint Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * remove model api Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * cleanup code Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * update async execution Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * improve training worker loop and predict endpoint Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * fix style Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * fix style to use python3.10 generics Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * add tests for services Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * style fix Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * style fix Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * style fix Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * style fix Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * style fix Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> --------- Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com>

…edge-platform#2961) * Add github actions for ui and server of geti inspect * Exclude UI from pre-commit prettier configuration The UI uses a different prettier configuration that does not seem to be picked up by pre-commit. * Add newline to .prettierignore * Apply prettier to `geti-inspect.yaml` * Generate OpenAPI spec before running UI checks * Checkout with lfs * Fix lint issues by removing wip components * Try installing git lfs in the playwright docker image * Fix unused noqa

…edge-platform#2963) * Add OpenAPI route * Remove MSW browser worker * Rename infernece to inspect * Update navbar title * Rename infernece to inspect

* rename models/ to pydantic_models/ Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * switch to use async session context manager Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * fix unit tests Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * add pipeline endpoints Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * fix frame aquisition worker and rename pipiline endpoints Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * add sources and sinks endpoints Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * stream loading working Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * add webrtc endpoints Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * fix workers: stream loading + inference + dispatcher Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * fix sinks Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * style Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * add unit tests and address comments Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * add tests Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * fix example schema Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * add todo Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> --------- Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com>

…2970) * chore: Add path alias to icons * feat: Add sidebar with dataset, models and stats

* feat: Display placeholders for images * refactor: Fix scrollbar * chore: Remove learn more

* chore(inspect): Update UI scripts * chore: Update github actions * chore: Update port to 8000

…en-edge-platform#2978) chore(inspect): Configure project(s) mocks for playwright

* chore: Add project route * feat: Add project management * revert ui lock change * chore: Remove not needed code for project management

…ts from the UI (open-edge-platform#2980) chore(inspect): Update allowed origins

… training progress (open-edge-platform#2984) * feat: Allow user to upload images * feat: List uploaded images * chore: Extract components to separate files and add ready to train and training progress * chore: comment thumbnail url generation

chore(inspect): Update openapi page title

…en-edge-platform#2989) * chore(inspect): Renamed app to application * chore(inspect): Rename app to application in github actions

…of email (open-edge-platform#2990) refactor: Update photo placeholder to use indicator instead of email

…dge-platform#2991) Improve error and suspense handling in router By moving all of the routes into a single root route we can make sure that all routes are rendered inside of an layout that has a suspense and error boundary.

Signed-off-by: Ashwin Vaidya <ashwin.vaidya@intel.com>

…open-edge-platform#2992) refactor: Improvements to the jobs management and training

…platform#2994) chore(inspect): Add more models

🐛 fix(inspect): Fix unable to start train job

chore(inspect): Update uv.lock

* add trainable models endpoint * fix Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * add test Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * add copyright Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> --------- Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com>

…rm#3004) * add thumbnails endpoint Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * add tests Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * generate thumbnails as background task Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> * update docstring Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com> --------- Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com>

Copilot

Pull Request Overview

This PR adds Docker build support for the Geti Inspect application, enabling containerized deployment of the CPU version. The changes prepare the application for containerization by making paths relative, adding a main entry point, and configuring the build system for proper packaging.

Key changes:

Docker build infrastructure with multi-stage builds for UI and backend
Application refactoring to support containerized execution
Build system configuration using Hatchling

Reviewed Changes

Copilot reviewed 7 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
application/backend/src/settings.py	Updates Alembic paths to be relative to module directory for container compatibility
application/backend/src/main.py	Extracts main logic into a callable function for package entry point
application/backend/pyproject.toml	Adds Hatchling build system, project scripts, and SSE dependency
application/.packaging/docker/nginx.conf	Adds nginx configuration for serving UI and proxying API requests
application/.packaging/docker/docker-compose.yml	Defines service configuration with volume mounts and device access
application/.packaging/docker/Dockerfile	Multi-stage build for UI compilation and Python application packaging
.dockerignore	Excludes unnecessary files from Docker build context

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

application/backend/pyproject.toml

application/.packaging/docker/nginx.conf

application/.packaging/docker/Dockerfile

Signed-off-by: Ashwin Vaidya <ashwin.vaidya@intel.com>

* feat(inspect): enhance ModelsView Signed-off-by: Dmitry Kalinin <dmitry.kalinin@intel.com> * Adjusted styling Signed-off-by: Dmitry Kalinin <dmitry.kalinin@intel.com> * Fixed comments Signed-off-by: Dmitry Kalinin <dmitry.kalinin@intel.com> * Fixed format Signed-off-by: Dmitry Kalinin <dmitry.kalinin@intel.com> --------- Signed-off-by: Dmitry Kalinin <dmitry.kalinin@intel.com>

…to allow all origins. Update package-lock.json with new dependencies and version upgrades for various packages. Signed-off-by: Ashwin Vaidya <ashwinnitinvaidya@gmail.com>

Signed-off-by: Ashwin Vaidya <ashwinnitinvaidya@gmail.com>

Copilot

Pull request overview

Copilot reviewed 7 out of 11 changed files in this pull request and generated 3 comments.

Files not reviewed (1)

application/ui/package-lock.json: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2025-11-24T12:03:08Z

application/backend/src/main.py

-        "http://localhost:9000",
-        "http://127.0.0.1:9000",
-    ],
+    allow_origins=["*"],


Allowing all origins with allow_origins=['*'] creates a security vulnerability by permitting any domain to make requests to the API. This exposes the application to CSRF attacks. Configure specific allowed origins or use environment variables to control CORS settings in production environments.

Matching https://github.com/open-edge-platform/training_extensions/blob/4d9a2b88f82680f869fe79fc3a85b5b0ec0f38eb/application/backend/app/main.py#L42

application/.packaging/docker/nginx.conf

application/.packaging/docker/docker-compose.yml

Signed-off-by: Ashwin Vaidya <ashwin.vaidya@intel.com>

Signed-off-by: Ashwin Vaidya <ashwinnitinvaidya@gmail.com>

Copilot

Pull request overview

Copilot reviewed 246 out of 359 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: Ashwin Vaidya <ashwin.vaidya@intel.com>

Copilot

Pull request overview

Copilot reviewed 26 out of 29 changed files in this pull request and generated 4 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

application/backend/src/main.py

application/.packaging/docker/README.md

application/.packaging/docker/docker-compose.yml

Copilot · 2025-11-27T15:56:21Z

application/backend/pyproject.toml

    "uvloop==0.21.0",
    "loguru==0.7.3",
    "trackio~=0.6.0",
+    "sse-starlette",


The dependency sse-starlette is added without a version constraint. This could lead to compatibility issues if breaking changes are introduced in future versions. Consider pinning to a specific version or version range.

Suggested change

"sse-starlette",

"sse-starlette~=1.6.1",

Signed-off-by: Ashwin Vaidya <ashwin.vaidya@intel.com>

Copilot

Pull request overview

Copilot reviewed 26 out of 29 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2025-12-01T13:50:03Z

application/backend/src/main.py

-        "http://localhost:9000",
-        "http://127.0.0.1:9000",
-    ],
+    allow_origins=["*"],


Changing CORS from specific origins to allow all origins (*) introduces a security risk. This allows any website to make requests to your API. Consider using environment variables to configure allowed origins for different deployment environments, or maintain the original restrictive list for production.

Copilot · 2025-12-01T13:50:03Z

application/backend/.pre-commit-config.yaml

+    rev: v2.12.0.2
+    hooks:
+      - id: hadolint
+        files: ^application/\.packaging/docker/ # TODO: this will break when the Dockerfile is moved


The TODO comment indicates technical debt. Consider using a more flexible file pattern (e.g., Dockerfile$ or **/*Dockerfile*) that won't break if the Dockerfile is moved, or create a tracking issue for this concern rather than leaving it as an inline TODO.

Suggested change

files: ^application/\.packaging/docker/ # TODO: this will break when the Dockerfile is moved

files: Dockerfile$|Dockerfile\..*$

application/.packaging/docker/docker-compose.yml

+      # Persist logs
+      - backend-logs:/app/logs
+    ports:
+      - "80:80"


application/backend/src/main.py

-        "http://localhost:9000",
-        "http://127.0.0.1:9000",
-    ],
+    allow_origins=["*"],


maxxgx · 2025-12-01T16:09:34Z

application/.packaging/docker/Dockerfile

+WORKDIR /app
+
+RUN --mount=type=cache,target=/home/non-root/.cache/uv,uid=10001,gid=10001 \
+    uv sync --frozen --no-dev --extra cpu --extra application


Why run uv sync from the root anomalib folder instead of application?

Installing inspect as an extra might be useful during development but I don't see the value in production environments

MarkRedeman and others added 30 commits October 8, 2025 13:52

Initial UI setup for Geti Inspect

5bdd704

Mainly based on the UI files and architecture from [Geti Tune](https://github.com/open-edge-platform/training_extensions/tree/develop/ui)

🧪 test(get-inspect): add unit tests for endpoints (open-edge-platform…

82dd6b0

…#2942) add unit tests for endpoints Signed-off-by: Ma, Xiangxiang <xiangxiang.ma@intel.com>

Add openapi spec to gitignore

1588d72

Use error page

22f8f8f

Delete nginx and Dockerfile

978a895

These won't be needed yet

🔧 chore(geti-inspect): Update Geti Inspect to react 19 (open-edge-pla…

cbb0a46

…tform#2948) * Use src folder inside run.sh * Set openapi_url path * Update to react 19 * Specify bash language in readme

🔧 chore(inspect): Update route structure and add openapi route (open-…

0614756

…edge-platform#2963) * Add OpenAPI route * Remove MSW browser worker * Rename infernece to inspect * Update navbar title * Rename infernece to inspect

chore: Update package lock (open-edge-platform#2967)

e07a955

feat: Add sidebar with dataset, models and stats (open-edge-platform#…

e04ef66

…2970) * chore: Add path alias to icons * feat: Add sidebar with dataset, models and stats

🚀 feat(inspect): Add dataset list (open-edge-platform#2971)

3489422

* feat: Display placeholders for images * refactor: Fix scrollbar * chore: Remove learn more

🔧 chore(inspect): Update UI scripts (open-edge-platform#2974)

4e160a1

* chore(inspect): Update UI scripts * chore: Update github actions * chore: Update port to 8000

🔧 chore(inspect): Configure project(s) mocks for playwright tests (op…

7437855

…en-edge-platform#2978) chore(inspect): Configure project(s) mocks for playwright

🚀 feat(inspect): Add project management (open-edge-platform#2977)

590ece3

* chore: Add project route * feat: Add project management * revert ui lock change * chore: Remove not needed code for project management

🔧 chore(inspect): Update allowed origins so server can receive reques…

d9a613d

…ts from the UI (open-edge-platform#2980) chore(inspect): Update allowed origins

🔧 chore(inspect): Update openapi page (open-edge-platform#2985)

1baa2eb

chore(inspect): Update openapi page title

🔧 chore(inspect): Rename folder structure from app to application (op…

0a2af70

…en-edge-platform#2989) * chore(inspect): Renamed app to application * chore(inspect): Rename app to application in github actions

refactor(inspect): Update photo placeholder to use indicator instead …

8f1ef19

…of email (open-edge-platform#2990) refactor: Update photo placeholder to use indicator instead of email

Fix path (open-edge-platform#2993)

237731d

Signed-off-by: Ashwin Vaidya <ashwin.vaidya@intel.com>

🔄 refactor(inspect): Improvements to the jobs management and training (…

2a4f257

…open-edge-platform#2992) refactor: Improvements to the jobs management and training

🔧 chore(inspect): Add more models to be used for training (open-edge-…

7653915

…platform#2994) chore(inspect): Add more models

🐛 fix(inspect): Use null pool for async engine (open-edge-platform#2998)

f0a7643

🐛 fix(inspect): Fix unable to start train job

🔧 chore(inspect): Update uv.lock (open-edge-platform#2995)

df8eb42

chore(inspect): Update uv.lock

Copilot AI review requested due to automatic review settings November 21, 2025 09:30

Copilot AI reviewed Nov 21, 2025

View reviewed changes

application/backend/pyproject.toml Show resolved Hide resolved

application/.packaging/docker/nginx.conf Show resolved Hide resolved

application/.packaging/docker/Dockerfile Outdated Show resolved Hide resolved

ashwinvaidya17 and others added 5 commits November 21, 2025 15:54

Fix non-root user mounting issues

a9c5f70

Signed-off-by: Ashwin Vaidya <ashwin.vaidya@intel.com>

Merge branch 'feature/geti-inspect' into ashwin/feature/dockerfile_2

40c5e5a

Update Dockerfile to include g++ and modify CORS settings in main.py …

4d09a21

…to allow all origins. Update package-lock.json with new dependencies and version upgrades for various packages. Signed-off-by: Ashwin Vaidya <ashwinnitinvaidya@gmail.com>

fix proxy issue

0a232b8

Signed-off-by: Ashwin Vaidya <ashwinnitinvaidya@gmail.com>

Copilot AI review requested due to automatic review settings November 24, 2025 12:02

Copilot AI reviewed Nov 24, 2025

View reviewed changes

cuda build

c7ebce5

Signed-off-by: Ashwin Vaidya <ashwin.vaidya@intel.com>

maxxgx force-pushed the feature/geti-inspect branch from 4567ce7 to ea4d226 Compare November 26, 2025 09:55

Add XPU build

ef63ed2

Signed-off-by: Ashwin Vaidya <ashwinnitinvaidya@gmail.com>

Copilot AI review requested due to automatic review settings November 27, 2025 15:10

Copilot AI reviewed Nov 27, 2025

View reviewed changes

ashwinvaidya17 added 2 commits November 27, 2025 16:48

Merge branch 'feature/geti-inspect' into ashwin/feature/dockerfile_2

f0ef85b

Fix conflicts

c0d3c78

Signed-off-by: Ashwin Vaidya <ashwin.vaidya@intel.com>

Copilot AI review requested due to automatic review settings November 27, 2025 15:55

Copilot AI reviewed Nov 27, 2025

View reviewed changes

Comment XPU settings

438a499

Signed-off-by: Ashwin Vaidya <ashwin.vaidya@intel.com>

ashwinvaidya17 marked this pull request as ready for review November 28, 2025 08:52

ashwinvaidya17 requested review from MarkRedeman, maxxgx, rajeshgangireddy and samet-akcay as code owners November 28, 2025 08:52

ashwinvaidya17 added 2 commits December 1, 2025 14:45

Merge branch 'feature/geti-inspect' into ashwin/feature/dockerfile_2

70f05fa

Merge branch 'feature/geti-inspect' into ashwin/feature/dockerfile_2

39209fc

Copilot AI review requested due to automatic review settings December 1, 2025 13:48

Copilot AI reviewed Dec 1, 2025

View reviewed changes

github-advanced-security bot found potential problems Dec 1, 2025

View reviewed changes

maxxgx reviewed Dec 1, 2025

View reviewed changes

	files: ^application/\.packaging/docker/ # TODO: this will break when the Dockerfile is moved
	files: Dockerfile$\|Dockerfile\..*$

📦 feat(inspect): Add docker build #3136

Are you sure you want to change the base?

📦 feat(inspect): Add docker build #3136

Uh oh!

Conversation

ashwinvaidya17 commented Nov 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📝 Description

TODO

✨ Changes

✅ Checklist

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Nov 24, 2025

Choose a reason for hiding this comment

Uh oh!

ashwinvaidya17 Nov 27, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI Nov 27, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Dec 1, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 1, 2025

Choose a reason for hiding this comment

Uh oh!

Check warning

Check warning

maxxgx Dec 1, 2025

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

ashwinvaidya17 commented Nov 21, 2025 •

edited

Loading