Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Annotation context display is unstable with multiple classes #2403

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in 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