FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix CTD empty conditions edge case by maximpavliv · Pull Request #3076 · DeepLabCut/DeepLabCut · GitHub

Fix CTD empty conditions edge case - #3076

Merged
AlexEMG merged 46 commits into
mainfrom
maxim/cond_kpts_bug
Sep 10, 2025
Merged

Fix CTD empty conditions edge case#3076
AlexEMG merged 46 commits into
mainfrom
maxim/cond_kpts_bug

Conversation

maximpavliv commented Aug 19, 2025
edited
Loading

Copy link
Copy Markdown
Contributor

This Pull Request fixes the edge case in the CTD pipeline when the conditions passed to the CTDInferenceRunner.predict() method are empty. This typically can happen when the FilterLowConfidencePoses processing stage filters out all poses.
Thus, this Pull Request solves #3001.

The following changes were introduced to fix the edge case:

  • Additionally, the TopDownCrop processing stage has been modified to always output images of the cropped shape, even in the case of an empty context["bboxes"] list. The first dimension length of the returned images (corresponding to the number of bounding boxes) obviously remains 0 in that case.

The frames indexes start at 0 and go up to (n_frames - 1)
maximpavliv added DLC3.0🔥 CTD Contidional Top-Down labels Aug 19, 2025
maximpavliv requested a review from AlexEMG August 26, 2025 11:05

arashsm79 commented Sep 10, 2025
edited
Loading

Copy link
Copy Markdown
Contributor

Thanks for this.
I ran these changes on the trimouse dataset with the following variants:

  • ctd_prenet_hrnet_*
  • ctd_coam_*

and everything worked as expected.
LGTM.

AlexEMG merged commit 382cdc7 into main Sep 10, 2025
5 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CTD Contidional Top-Down DLC3.0🔥

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL