Skip to content

feat: add temporal model API service#40

Merged
MateoLostanlen merged 8 commits into
mainfrom
feat/temporal-model-api
Jun 10, 2026
Merged

feat: add temporal model API service#40
MateoLostanlen merged 8 commits into
mainfrom
feat/temporal-model-api

Conversation

@MateoLostanlen

@MateoLostanlen MateoLostanlen commented Jun 9, 2026

Copy link
Copy Markdown
Member
  • Add temporal_model_api (pyronear/temporal-model-api:latest) under a new temporal profile, wired to the existing MinIO so it can read alert-API images from S3. Uses latest so a locally built image is picked up.
  • Add make run-temporal / stop-temporal targets and include the profile in make stop.
  • Bucket is set to frames via TEMPORAL_API_S3_BUCKET; change that var if the alert API uses a different bucket.

@MateoLostanlen MateoLostanlen force-pushed the feat/temporal-model-api branch from 3f63735 to 47e2d4b Compare June 9, 2026 16:12
@MateoLostanlen MateoLostanlen force-pushed the feat/temporal-model-api branch from 47e2d4b to 5f7b486 Compare June 9, 2026 16:15

@Chouffe Chouffe left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@MateoLostanlen MateoLostanlen merged commit 822dd0b into main Jun 10, 2026
1 check 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.

2 participants