Visitar URL original
ModelZoo GUI Tab fixes by maximpavliv · Pull Request #3055 · DeepLabCut/DeepLabCut · GitHub
Skip to content

ModelZoo GUI Tab fixes - #3055

Merged
AlexEMG merged 59 commits into
mainfrom
maxim/modelzoo_tab_fixes
Sep 10, 2025
Merged

AlexEMG merged 59 commits into
mainfrom
maxim/modelzoo_tab_fixes

Conversation

@maximpavliv

@maximpavliv maximpavliv commented Jul 8, 2025 •

Copy link
Copy Markdown
Contributor

This Pull Request makes several fixes to bugs in the ModelZoo Tab:

  • adds pseudo threshold widgets back to the _build_tf_attributes
  • restores showing / hiding the pytorch-specific widgets (torch_widget) on engine change
  • restores showing / hiding the detector settings on engine change
  • hides the detector row when humanbody supermodel is selected
  • hides correctly the torch adaptation widget when superanimal_humanbody is selected

It also adds the following improvements from #3038 [outdated]:

  • adds create_labeled_video argument to the video_inference_superanimal() method, and adds a "Create labeled video" checkbox
  • hides adaptation settings when adaptation checkbox is unchecked
  • adds batch size combo boxes for pose model and detector
  • uses set_combo_items() method introduced in BUCTD GUI integration #3016 to change the contents of Qt combo boxes - this is a cleaner approach and avoids callback loops, thus improving the reactivity of the tab

It also refactors deeplabcut/gui/tabs/modelzoo.py:

  • splits long methods into smaller ones
  • removes code dupplication
  • restores VideoSelectionWidget and adds hide_videotype option to it)

@maximpavliv maximpavliv changed the title ModelZoo GUI Tab fixes [WIP] ModelZoo GUI Tab fixes Jul 8, 2025
@maximpavliv maximpavliv added GUI issues relating to GUI WORK IN PROGRESS! developers are currently working on this feature... stay tuned. ModelZoo related to models/code for modezoo.deeplabcut.org labels Jul 8, 2025
@maximpavliv

Copy link
Copy Markdown
Contributor Author

The code snippet that displays the created labeled videos (introduced in #3043) does not work correctly if the user created other labeled video from the same video in the past - it will list all labeled videos in the folder instead of the ones that were just created in this specific run. How should we handle this in your opinion?

@maximpavliv
maximpavliv force-pushed the maxim/modelzoo_tab_fixes branch from 5094020 to 1435920 Compare July 10, 2025 14:38
@maximpavliv maximpavliv changed the title [WIP] ModelZoo GUI Tab fixes ModelZoo GUI Tab fixes Jul 10, 2025
@maximpavliv maximpavliv removed the WORK IN PROGRESS! developers are currently working on this feature... stay tuned. label Jul 10, 2025
@maximpavliv
maximpavliv requested a review from AlexEMG July 10, 2025 14:52
@maximpavliv maximpavliv changed the title ModelZoo GUI Tab fixes [WIP] ModelZoo GUI Tab fixes Jul 25, 2025
@maximpavliv maximpavliv added the WORK IN PROGRESS! developers are currently working on this feature... stay tuned. label Jul 25, 2025
@maximpavliv maximpavliv removed the WORK IN PROGRESS! developers are currently working on this feature... stay tuned. label Jul 26, 2025
@maximpavliv maximpavliv changed the title [WIP] ModelZoo GUI Tab fixes ModelZoo GUI Tab fixes Jul 26, 2025
@maximpavliv
maximpavliv force-pushed the maxim/modelzoo_tab_fixes branch from f03c54a to c6ecb0f Compare July 31, 2025 11:39
@AlexEMG

AlexEMG commented Sep 4, 2025

Copy link
Copy Markdown
Member

Please resolve conflicts.

@maximpavliv

Copy link
Copy Markdown
Contributor Author

Please resolve conflicts.

@AlexEMG done!

@AlexEMG AlexEMG left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent!

@AlexEMG
AlexEMG merged commit 130f028 into main Sep 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GUI issues relating to GUI ModelZoo related to models/code for modezoo.deeplabcut.org

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants