Skip to content

HOTFIX/worker fix#34

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

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

Conversation

@wailbentafat
Copy link
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
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?

logger.warning("Missing image_ref in event payload for photo %s", job.photo_id)
return

embedding_literal = self._vector_literal(embedding)
Copy link
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

matched_user: ClosestUserMatch | None = None

try:
async with self.conn.begin():
Copy link
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
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
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
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