Update status message on status bar

[shrivaths16]

Description

While removing videos from the gui, the status bar at the bottom does not update correctly. Now trying to update it properly after every video is being removed.

Related PR:

• Update labelled frames in the GUI after removing a video from a project #1280

Types of changes

• Bugfix
• New feature
• Refactor / Code style update (no logical changes)
• Build / CI changes
• Documentation Update
• Other (explain)

Does this address any currently open issues?

• Removing videos from a project doesn't update Labeled frames: YYY in project #1261

Outside contributors checklist

• Review the guidelines for contributing to this repository
• Read and sign the CLA and add yourself to the authors list
• Make sure you are making a pull request against the develop branch (not main). Also you should start your branch off develop
• Add tests that prove your fix is effective or that your feature works
• Add necessary documentation (if appropriate)

Thank you for contributing to SLEAP!

❤️

[talmo]

Looks like we shouldn't be trying to query for the number of labeled frames for a video that does not exist:

Traceback (most recent call last):
  File "/home/runner/work/sleap/sleap/sleap/gui/app.py", line 1227, in _after_plot_change
    self.updateStatusMessage()
  File "/home/runner/work/sleap/sleap/sleap/gui/app.py", line 1270, in updateStatusMessage
    current_video, "predicted"
  File "/home/runner/work/sleap/sleap/sleap/io/dataset.py", line [107](https://github.com/talmolab/sleap/actions/runs/5661452334/job/15339366532?pr=1411#step:5:108)3, in get_labeled_frame_count
    return self._cache.get_frame_count(video, filter)
  File "/home/runner/work/sleap/sleap/sleap/io/dataset.py", line 289, in get_frame_count
    video, filter
  File "/home/runner/work/sleap/sleap/sleap/io/dataset.py", line 319, in get_filtered_frame_idxs
    video_idx = self.labels.videos.index(video)
ValueError: Video(backend=MediaVideo(filename='tests/data/videos/centered_pair_small.mp4', grayscale=True, bgr=True, dataset='', input_format='')) is not in list

Namely, in updateStatusMessage. Maybe we should grab it from state["video"]?

If state["video"] is still set to a video that does not exist, then it really should be set to None. If this is being handled elsewhere, it's possible that this callback is being triggered before state["video"] is appropriately updated to None. We should check why the RemoveVideo command isn't doing this, and if it's supposed to be, why it's not applied when this callback is called.

A quick patch would be to check for whether current_video in self.labels.videos, and if False, just bypass checking for frame count. That said, it would be preferable to find out why this workaround is necessary in the first place.

More hints:

• Check how Update labelled frames in the GUI after removing a video from a project #1280 handles this issue
• Check if it's a problem with the labels cache not updating appropriately

[codecov]

Codecov Report

Merging #1411 (52a27a2) into develop (3a01ef3) will increase coverage by 0.00%.
The diff coverage is 91.66%.

@@           Coverage Diff            @@
##           develop    #1411   +/-   ##
========================================
  Coverage    73.02%   73.02%           
========================================
  Files          133      133           
  Lines        23752    23757    +5     
========================================
+ Hits         17345    17349    +4     
- Misses        6407     6408    +1     

Files Changed	Coverage Δ
sleap/gui/commands.py	61.42% <83.33%> (-0.02%)	⬇️
sleap/gui/app.py	75.31% <100.00%> (+0.03%)	⬆️
sleap/io/dataset.py	83.86% <100.00%> (+0.03%)	⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[roomrys]

#1422 should resolve the strange error we were seeing in these tests. I don't think we need the extra "conditional state check" code here from the second commit.

[talmo]

LGTM! Good catches!