Skip to content

fix(inference): isolate overlapping image requests and avoid shared result files - #13

Merged
FuTseYi merged 4 commits into
mainfrom
fix/image-request-isolation
Oct 8, 2026
Merged

FuTseYi merged 4 commits into
mainfrom
fix/image-request-isolation

Conversation

@FuTseYi

@FuTseYi FuTseYi commented Oct 8, 2026

Copy link
Copy Markdown
Owner

Changes

  • Remove shared self.data mutations from the Flask image endpoint; keep the original JSON response shape for the Spring Boot client.
  • Generate one unpredictable image output filename per request in the local runs/ directory; clean it up in finally.
  • Do not attempt to upload a result when YOLO returned no detections or no result file, and do not report success when the result upload fails.
  • Remove the previous image-route cleanup code which could remove an unrelated local file named like the URL basename.
  • Add 4 dependency-free regression tests including a synchronized overlapping-request test that verifies outputs cannot overwrite one another.
  • Update testing documentation with the image-only scope.

Boundaries

  • This PR does not solve shared self.data / temporary path state in video and camera routes; those remain separate work.
  • Does not claim real GPU/CPU inference or full Flask/Spring Boot integration testing.
  • CI: Python 3.10/3.12 regressions, syntax check, Vue and Spring Boot builds.

@FuTseYi
FuTseYi merged commit 71b15b4 into main Oct 8, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant