Skip to content

Max N Count Support #1493

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.

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

Merged
merged 7 commits into from
May 26, 2025
Merged

Max N Count Support #1493

merged 7 commits into from
May 26, 2025

Conversation

BryonLewis
Copy link
Collaborator

@BryonLewis BryonLewis commented May 20, 2025

resolves #1492

DIVE-MaxN-Counts_small.mp4

TODO:

  • Filtering the Track List based on the current frame visible
  • Updating selection of track using the keyboard shortcuts in the track list so it only jumps to the next track in the track list
  • Filter the Type list so it will show only the types for the current frame
  • Add option to prevent cascading types. I.E it uses only the highest tracktype and will be removed if it it goes below that trackType.
  • Add an optional button to the types that will jump to the frame with the highest Count for the type. This button would be off by default but in the client settings it can be turned on and enabled.
  • Update Detections TimeLine View with additional options
    • Option to hide the total count
    • Option to count only detections and not the tracks
    • Maybe rename it to 'counts' and have a label being either 'tracks' or 'detections'
    • Went with a 750ms hover brings up the settings for the panel
  • Update the filtering on types and tracks to make it so it is only active when the video/media is not 'playing'. This may require placing it in the UsetimeObserver instead of the media annotator because the time observer is easier to pass around.
  • The auto zoom track options for the tracklist with some additional settings (ZoomTime: we may want more than a snapping to the position, ZoomBounds: the spacing around the bbox for the item to zoom in)
  • Make a way to full screen the video or toggle between full screen and the basic view.
  • Fix Tests
  • Update Documentation for Detections in timeline.
  • Test in Desktop/Electron Version
  • Create new dive version and release

@BryonLewis BryonLewis merged commit aa38619 into main May 26, 2025
4 checks passed
@BryonLewis BryonLewis deleted the max-n-counts branch May 26, 2025 12:15
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.

Easier Max-N Counts for classification types
1 participant