Skip to content

Annotation context display is unstable with multiple classes #2403

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Closed
abey79 opened this issue Jun 13, 2023 · 2 comments
Closed

Annotation context display is unstable with multiple classes #2403

abey79 opened this issue Jun 13, 2023 · 2 comments
Labels
😤 annoying Something in the UI / SDK is annoying to use 🏎️ Quick Issue Can be fixed in a few hours or less 📺 re_viewer affects re_viewer itself ui concerns graphical user interface

Comments

@abey79
Copy link
Member

abey79 commented Jun 13, 2023 •
edited by emilk
Loading

Describe the annoyance

When annotation contexts have multiple classes, their display is flickering due to random ordering of the classes.

Export-1686641959919.mp4

To Reproduce
Steps to reproduce the behavior:

This demo RRD:
demo.rrd.zip

Also, the mp_face example (#2338) can be used:

examples/python/face_tracking/main.py --demo-image

In the timeline, click "video/landmarker/annotation_context" to display the glitch.
@abey79 abey79 added the 😤 annoying Something in the UI / SDK is annoying to use label Jun 13, 2023
@abey79 abey79 mentioned this issue Jun 13, 2023
Open
@nikolausWest nikolausWest added ui concerns graphical user interface 📺 re_viewer affects re_viewer itself labels Jun 21, 2023
@emilk emilk added the 🏎️ Quick Issue Can be fixed in a few hours or less label Jun 26, 2023
@emilk emilk self-assigned this Jul 4, 2023
@emilk
Copy link
Member

emilk commented Jul 4, 2023

This is really weird. The UI comes from ClassDescription::keypoint_connections which is defined as:

pub keypoint_connections: Vec<(KeypointId, KeypointId)>

So there is no hashmap there. It must somehow get populated in a random order, but I don't immediately see how.

@emilk emilk removed their assignment Jul 4, 2023
@abey79
Copy link
Member Author

abey79 commented Oct 17, 2023

I can't reproduce anymore, this has likely been fixed.

@abey79 abey79 closed this as completed Oct 17, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
😤 annoying Something in the UI / SDK is annoying to use 🏎️ Quick Issue Can be fixed in a few hours or less 📺 re_viewer affects re_viewer itself ui concerns graphical user interface
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@emilk @nikolausWest @abey79