Skip to content
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

If use_tqdm bool doesn't work with a string #19

Closed
aaroncherian opened this issue Mar 8, 2024 · 0 comments · Fixed by #21
Closed

If use_tqdm bool doesn't work with a string #19

aaroncherian opened this issue Mar 8, 2024 · 0 comments · Fixed by #21
Assignees
Labels
bug Something isn't working

Comments

@aaroncherian
Copy link
Contributor

aaroncherian commented Mar 8, 2024

Hey, noticed that for the base_tracker process_video function, the input video path is optionally a string or a Path. I was trying to use a string, but ran into an error with the if use_tqdm bool, where the line desc=f"processing video: {input_video_filepath.name}" fails, because the .name function only works with a Path, not a string. I'm happy to make a fix, but wasn't sure if you'd rather it be changing what the description line of tqdm would be, or to change the type hint to be just a string.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants