Skip to content

Feat/detection pose#329

Open
fe51 wants to merge 5 commits intodevelopfrom
feat/detection-pose-id
Open

Feat/detection pose#329
fe51 wants to merge 5 commits intodevelopfrom
feat/detection-pose-id

Conversation

@fe51
Copy link
Member

@fe51 fe51 commented Mar 18, 2026

The PR to close #328 Adapt detection creation to pyro-api updates

It introduces the following modifications :

  • Replace azimuth/azimuths with pose_ids in credentials.json, src/run.py, engine, README
  • Update create_detection call: (media, azimuth, bboxes) → (media, bboxes, pose_id) to match new pyro-api endpoint
  • Fix fill_empty_bboxes: use .tolist() to produce Python floats instead of np.float64
  • Skip alerts with empty bboxes instead of blocking the queue -> Added but should not happen
  • Add response.raise_for_status() on occlusion mask fetch to handle 404 gracefull
  • Log API response body on KeyError('id') for easier debug
  • Bump pinned pyroclient commit in requirements.txt to version with new create_detection signature

Now, it has to be tested on few sites, and need to work on updating credentials on all rpi fleet.

Happy to discuss it

filled = src.copy()
filled[:, -1] = 0.0 # force confidence to 0 for duplicated boxes
self._alerts[i]["bboxes"] = [tuple(row) for row in filled]
self._alerts[i]["bboxes"] = [tuple(row) for row in filled.tolist()]
Copy link
Member

Choose a reason for hiding this comment

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

why are we updating this here ?

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.

Adapt detection creation to pyro-api updates

2 participants