Skip to content

HOTFIX/worker fix#34

Closed
wailbentafat wants to merge 10 commits intodevelopfrom
HOTFIX/worker_fix
Closed

HOTFIX/worker fix#34
wailbentafat wants to merge 10 commits intodevelopfrom
HOTFIX/worker_fix

Conversation

@wailbentafat
Copy link
Copy Markdown
Collaborator

this pr introcduce some fixes for the worker code

@wailbentafat wailbentafat changed the base branch from main to develop March 25, 2026 22:30
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

hhh why u hav removed embed and compute evens?

Comment thread app/service/face_match.py
logger.warning("Missing image_ref in event payload for photo %s", job.photo_id)
return

embedding_literal = self._vector_literal(embedding)
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

this two function make them as flat logique embedding_literal = self._vector_literal(embeddi bbox_payload = self._serialize_bbox(bbox) remove them and replace them with flat logique

Comment thread app/service/face_match.py
matched_user: ClosestUserMatch | None = None

try:
async with self.conn.begin():
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

also remove those two function and raplce them. with flat logiqure no neeed for helper fucntions

raw = job.image_ref[len(MINIO_URL_PREFIX) :]
parts = raw.split("/", 1)
if len(parts) != 2 or not parts[0] or not parts[1]:
raise ValueError("Invalid MinIO image_ref format")
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

u will get storage key not minio url try to read the content of the class before start vibe codding

@wailbentafat
Copy link
Copy Markdown
Collaborator Author

i will handle this code from here until the our next meeting

@ademboukabes ademboukabes self-requested a review March 27, 2026 10:26
@Tyjfre-j Tyjfre-j closed this Apr 11, 2026
@Tyjfre-j Tyjfre-j deleted the HOTFIX/worker_fix branch April 11, 2026 20:47
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