-
Notifications
You must be signed in to change notification settings - Fork 97
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
Expose supervised ID models to GUI #1213
Conversation
…/supervised_id_gui
…/supervised_id_gui
…/supervised_id_gui
Codecov Report
@@ Coverage Diff @@
## develop #1213 +/- ##
===========================================
- Coverage 72.36% 72.31% -0.05%
===========================================
Files 132 132
Lines 23502 23526 +24
===========================================
+ Hits 17008 17014 +6
- Misses 6494 6512 +18
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few comments that I will handle quickly, looks great!
sleap/gui/learning/dialog.py
Outdated
if recent_cfg_info.head_name in ("multi_class_topdown"): | ||
return "top-down-id" | ||
if recent_cfg_info.head_name in ("centroid", "centered_instance"): | ||
return "top-down" | ||
if recent_cfg_info.head_name in ("multi_instance"): | ||
return "bottom-up" | ||
if recent_cfg_info.head_name in ("single_instance"): | ||
return "single" | ||
if recent_cfg_info.head_name in ("multi_class_bottomup"): | ||
return "bottom-up-id" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Most of these are not tuples (except for the "top-down" check). Thus, if the recent_cfg_info.head_name
were "multi_class"
, then it would be in "multi_class_topdown"
and in "multi_class_bottomup"
. This likely won't be a problem since the recent_cfg_info.head_name
is set from only a few options, but we may not be doing the check we thought we were.
Description
Add supervised ID models to GUI
Types of changes
Does this address any currently open issues?
[list open issues here]
Outside contributors checklist
Thank you for contributing to SLEAP!
❤️