faces: add person_id filter to /faces/embeddings; remove tag-bootstrap #89
Reference in New Issue
Block a user
Delete Branch "feature/faces-tab"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Pairs with the Apollo FACES-tab change. The new
POST /api/persons/{id}/similar-unassigned route on Apollo needs to
fetch one person's embeddings cheaply to compute the centroid;
adding a person_id query param to /faces/embeddings keeps that to a
single round-trip instead of paging the whole detected set
client-side. When both person_id and unassigned=true are supplied,
person_id wins (the explicit filter is the more specific intent).
Tag-bootstrap removal: bootstrap_candidates_handler,
bootstrap_persons_handler, /persons/bootstrap and
/tags/people-bootstrap-candidates route registrations, and the
heuristic helpers (is_plausible_name_token, looks_like_person) plus
their tests. Only Apollo called these; the migration is complete.
The persons.created_from_tag column stays - it's informational on
existing rows and removing it would be a destructive migration for
no benefit.
Co-Authored-By: Claude Opus 4.7 (1M context) noreply@anthropic.com
Pairs with the Apollo FACES-tab change. The new POST /api/persons/{id}/similar-unassigned route on Apollo needs to fetch one person's embeddings cheaply to compute the centroid; adding a person_id query param to /faces/embeddings keeps that to a single round-trip instead of paging the whole detected set client-side. When both person_id and unassigned=true are supplied, person_id wins (the explicit filter is the more specific intent). Tag-bootstrap removal: bootstrap_candidates_handler, bootstrap_persons_handler, /persons/bootstrap and /tags/people-bootstrap-candidates route registrations, and the heuristic helpers (is_plausible_name_token, looks_like_person) plus their tests. Only Apollo called these; the migration is complete. The persons.created_from_tag column stays - it's informational on existing rows and removing it would be a destructive migration for no benefit. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>