Open Arthurmayo opened 4 months ago
The recent change in video.py
involves simplifying the logic that determines the self.selectable
attribute within a class. Previously, the attribute depended on the state of self.predicted
and the color manager's prediction settings. Now, self.selectable
is always set to True
, streamlining the code and potentially enhancing usability by making all items selectable regardless of other conditions.
File | Change Summary |
---|---|
sleap/gui/widgets/video.py |
Simplified the logic for setting self.selectable to be always True . |
In the code where widgets play,
Selectable options lead the way,
No longer bound by past conditions,
Simplified with clear decisions.
Now every item can take the stage,
A brighter path on the coding page. ππ
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Description
Predicted instances were not selectable if they were not colored. This fix allows all instances to always be selectable by the user.
Types of changes
Does this address any currently open issues?
[list open issues here]
Outside contributors checklist
Thank you for contributing to SLEAP!
:heart:
Summary by CodeRabbit