Skip to content

Conversation

@gitttt-1234
Copy link
Collaborator

@gitttt-1234 gitttt-1234 commented Nov 21, 2025

Summary

  • Adds video name prefix to prediction output filenames when running inference with --video_index
  • Prevents filename collisions when running inference on multiple videos from the same project

Changes

Updated make_predict_cli_call in runners.py (lines 263-273):

  • Extracts video index from CLI args when --video_index is present
  • Prepends video filename to prediction output path
  • Ensures unique output files for each video in multi-video inference

Before vs After

Before

When running inference on multiple videos from project.slp:

predictions/project.slp.250121_142530.predictions.slp  # All videos overwrite same file ❌

After

predictions/video1.mp4_project.slp.250121_142530.predictions.slp
predictions/video2.mp4_project.slp.250121_142530.predictions.slp
predictions/video3.mp4_project.slp.250121_142530.predictions.slp

Why

Previously, when running inference on multiple videos from the same project, all predictions used the same output filename (based on the project name), causing each video's predictions to overwrite the previous one. This fix ensures each video gets a unique output file by prepending the video filename.

Related

Addresses the filename collision issue discussed for v1.4.1: when running inference on all videos, predictions would overwrite each other.

🤖 Generated with Claude Code

@codecov
Copy link

codecov bot commented Nov 21, 2025

Codecov Report

❌ Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 59.40%. Comparing base (a8684ca) to head (d58543c).
⚠️ Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
sleap/gui/learning/runners.py 0.00% 5 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2475      +/-   ##
===========================================
- Coverage    59.42%   59.40%   -0.03%     
===========================================
  Files           68       68              
  Lines        13226    13231       +5     
===========================================
  Hits          7860     7860              
- Misses        5366     5371       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gitttt-1234 gitttt-1234 merged commit fee86ae into develop Nov 21, 2025
8 checks passed
@gitttt-1234 gitttt-1234 deleted the divya/add-video-name-to-path branch November 21, 2025 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants