Skip to content

Operator role introduced in web ui - #209

Merged
stefanodellosa-personal merged 6 commits into
mainfrom
operator_role_in_web_ui
Jul 25, 2026
Merged

stefanodellosa-personal merged 6 commits into
mainfrom
operator_role_in_web_ui

Conversation

@stefanodellosa-personal

Copy link
Copy Markdown
Collaborator

Operator role is now introduced to access and configure actuators, but does not have full admin rights

Operator role is now introduced to access and configure actuators, but does not have full admin rights
@alessandropalla

Copy link
Copy Markdown
Collaborator

@copilot , test test/test_models.py::test_classification_dog_overlapping fails but it is not related to this PR. Please, can you add a commit where you, instead of matching the bounding box, compute the IoU and make it such that IoU needs to be greater than a threshold (maybe 95%)

Here the failing test

=========================== short test summary info ============================
FAILED test/test_models.py::test_classification_dog_overlapping - assert [555, 368, 1045, 616] == [554, 368, 1045, 616]
  
  At index 0 diff: 555 != 554
  
  Full diff:
    [
  -     554,
  ?       ^
  +     555,
  ?       ^
        368,
        1045,
        616,
    ]
============ 1 failed, 183 passed, 11 warnings in 177.44s (0:02:57) ============

Copilot AI commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

@copilot , test test/test_models.py::test_classification_dog_overlapping fails but it is not related to this PR. Please, can you add a commit where you, instead of matching the bounding box, compute t...

Done in commit where I replaced the exact bounding box assertion with compute_iou(classified_animals[0]["xyxy"], [554, 368, 1045, 616]) > 0.95, reusing the existing compute_iou helper already in the test file.

@stefanodellosa-personal
stefanodellosa-personal merged commit 9e7903a into main Jul 25, 2026
2 of 3 checks passed
@stefanodellosa-personal
stefanodellosa-personal deleted the operator_role_in_web_ui branch July 25, 2026 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants