pepperplus/pepperplus-cb
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

feat: visual emotion recognition agent #54

Merged
s.o.h.luijkx merged 27 commits from feat/visual-emotion-recognition into main 2026-01-30 16:53:16 +00:00
Conversation 20 Commits 27 Files Changed 12 +1533 -112
s.o.h.luijkx commented 2026-01-30 15:22:22 +00:00 (Migrated from git.science.uu.nl)
Copy Link

Implemented Visual Emotion Detection Agent.

ref: N25B-393

To verify:

  • Style checks pass
  • Pipeline (tests) pass
  • Documentation is up to date
  • Tests are up to date (new code is covered)

For empirical testing:
Run the RI using the feat/pause-functionality branch.
This branch will use laptop webcam to send video image. Emotion updates should appear in the CB logs.

Implemented Visual Emotion Detection Agent. ref: N25B-393 To verify: - [x] Style checks pass - [x] Pipeline (tests) pass - [x] Documentation is up to date - [x] Tests are up to date (new code is covered) For empirical testing: Run the RI using the `feat/pause-functionality` branch. This branch will use laptop webcam to send video image. Emotion updates should appear in the CB logs.
s.o.h.luijkx commented 2026-01-30 15:23:09 +00:00 (Migrated from git.science.uu.nl)
Copy Link

added 1 commit

  • cbbb80ea - chore: removed unnecessary imports

Compare with previous version

added 1 commit <ul><li>cbbb80ea - chore: removed unnecessary imports</li></ul> [Compare with previous version](/ics/sp/2025/n25b/pepperplus-cb/-/merge_requests/54/diffs?diff_id=138111&start_sha=3867d9e8b91cb1b604576d159cf15a4835ca5311)
s.o.h.luijkx commented 2026-01-30 15:40:21 +00:00 (Migrated from git.science.uu.nl)
Copy Link

added 1 commit

  • 14eb4da3 - chore: added deepface to test group

Compare with previous version

added 1 commit <ul><li>14eb4da3 - chore: added deepface to test group</li></ul> [Compare with previous version](/ics/sp/2025/n25b/pepperplus-cb/-/merge_requests/54/diffs?diff_id=138115&start_sha=cbbb80ea5ccf2321f0c0481728d8ac51509a0b44)
s.o.h.luijkx commented 2026-01-30 15:52:16 +00:00 (Migrated from git.science.uu.nl)
Copy Link

added 7 commits

  • 14eb4da3...516fda46 - 6 commits from branch main
  • c07efd37 - Merge branch 'main' into feat/visual-emotion-recognition

Compare with previous version

added 7 commits <ul><li>14eb4da3...516fda46 - 6 commits from branch <code>main</code></li><li>c07efd37 - Merge branch &#39;main&#39; into feat/visual-emotion-recognition</li></ul> [Compare with previous version](/ics/sp/2025/n25b/pepperplus-cb/-/merge_requests/54/diffs?diff_id=138118&start_sha=14eb4da300cce62c92163bc55b3fcc0e5872e4f6)
s.o.h.luijkx commented 2026-01-30 15:59:09 +00:00 (Migrated from git.science.uu.nl)
Copy Link

added 2 commits

  • 53ba8bb1 - 1 commit from branch main
  • d9d968ef - Merge branch 'main' into feat/visual-emotion-recognition

Compare with previous version

added 2 commits <ul><li>53ba8bb1 - 1 commit from branch <code>main</code></li><li>d9d968ef - Merge branch &#39;main&#39; into feat/visual-emotion-recognition</li></ul> [Compare with previous version](/ics/sp/2025/n25b/pepperplus-cb/-/merge_requests/54/diffs?diff_id=138121&start_sha=c07efd37b9e44db95aff70db6db50771a815b108)
s.o.h.luijkx commented 2026-01-30 16:06:05 +00:00 (Migrated from git.science.uu.nl)
Copy Link

added 2 commits

  • 68f445c8 - 1 commit from branch main
  • 5411dbe6 - Merge branch 'main' into feat/visual-emotion-recognition

Compare with previous version

added 2 commits <ul><li>68f445c8 - 1 commit from branch <code>main</code></li><li>5411dbe6 - Merge branch &#39;main&#39; into feat/visual-emotion-recognition</li></ul> [Compare with previous version](/ics/sp/2025/n25b/pepperplus-cb/-/merge_requests/54/diffs?diff_id=138123&start_sha=d9d968ef2756ba71f1d825b742928acd72ee58ea)
s.o.h.luijkx commented 2026-01-30 16:06:43 +00:00 (Migrated from git.science.uu.nl)
Copy Link

added 1 commit

  • 1bdfe343 - chore: added tf-keras to group

Compare with previous version

added 1 commit <ul><li>1bdfe343 - chore: added tf-keras to group</li></ul> [Compare with previous version](/ics/sp/2025/n25b/pepperplus-cb/-/merge_requests/54/diffs?diff_id=138125&start_sha=5411dbe62827a766270d87b697dcd11da4a973fc)
8464960 commented 2026-01-30 16:44:26 +00:00 (Migrated from git.science.uu.nl)
Copy Link

marked the checklist item Tests are up to date (new code is covered) as completed

marked the checklist item **Tests are up to date \(new code is covered\)** as completed
8464960 commented 2026-01-30 16:44:29 +00:00 (Migrated from git.science.uu.nl)
Copy Link

marked the checklist item Documentation is up to date as completed

marked the checklist item **Documentation is up to date** as completed
8464960 commented 2026-01-30 16:44:29 +00:00 (Migrated from git.science.uu.nl)
Copy Link

marked the checklist item Pipeline (tests) pass as completed

marked the checklist item **Pipeline \(tests\) pass** as completed
8464960 commented 2026-01-30 16:44:33 +00:00 (Migrated from git.science.uu.nl)
Copy Link

marked the checklist item Style checks pass as completed

marked the checklist item **Style checks pass** as completed
8464960 commented 2026-01-30 16:47:18 +00:00 (Migrated from git.science.uu.nl)
Copy Link

Either remove print statements or make use of the logger

Either remove print statements or make use of the logger
8464960 commented 2026-01-30 16:47:18 +00:00 (Migrated from git.science.uu.nl)
Copy Link

I checked code quality and seems to adhere to standards, especially if you resolve the one thread that I have.

Could not test it with my own camera, but this is probably a security issue. We have already tested it on Pepper's camera.

I checked code quality and seems to adhere to standards, especially if you resolve the one thread that I have. Could not test it with my own camera, but this is probably a security issue. We have already tested it on Pepper's camera.
8464960 commented 2026-01-30 16:47:18 +00:00 (Migrated from git.science.uu.nl)
Copy Link

approved this merge request

approved this merge request
s.o.h.luijkx commented 2026-01-30 16:48:50 +00:00 (Migrated from git.science.uu.nl)
Copy Link

added 1 commit

  • 0d1f43df - chore: removed print statements

Compare with previous version

added 1 commit <ul><li>0d1f43df - chore: removed print statements</li></ul> [Compare with previous version](/ics/sp/2025/n25b/pepperplus-cb/-/merge_requests/54/diffs?diff_id=138131&start_sha=1bdfe343500bccede920464f2f13bad5841579c9)
s.o.h.luijkx commented 2026-01-30 16:51:45 +00:00 (Migrated from git.science.uu.nl)
Copy Link

resolved all threads

resolved all threads
k.marinus commented 2026-01-30 16:53:11 +00:00 (Migrated from git.science.uu.nl)
Copy Link

resolved all threads

resolved all threads
k.marinus commented 2026-01-30 16:53:12 +00:00 (Migrated from git.science.uu.nl)
Copy Link

approved this merge request

approved this merge request
k.marinus (Migrated from git.science.uu.nl) merged commit 21690da679 into main 2026-01-30 16:53:16 +00:00
k.marinus commented 2026-01-30 16:53:17 +00:00 (Migrated from git.science.uu.nl)
Copy Link

mentioned in commit 21690da679

mentioned in commit 21690da679011a97a4e19c003bf6ee5bc9c6c3ea
8464960 (Migrated from git.science.uu.nl) approved these changes 2026-02-02 13:29:04 +00:00
k.marinus (Migrated from git.science.uu.nl) approved these changes 2026-02-02 13:29:04 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
8464960
k.marinus
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
kasper twirre pepperplus
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: pepperplus/pepperplus-cb#54
Delete Branch "feat/visual-emotion-recognition"

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?