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

Debugging with process_clip.py #281

Closed
jasonmadigan opened this issue Nov 3, 2020 · 3 comments
Closed

Debugging with process_clip.py #281

jasonmadigan opened this issue Nov 3, 2020 · 3 comments

Comments

@jasonmadigan
Copy link

Describe the bug
I saw a really useful comment on #190 about how to process saved clips for debugging/tuning:

docker run --privileged --rm -v ${PWD}/config:/config -v /dev/bus/usb:/dev/bus/usb -v ${PWD}/debug:/debug -v /etc/localtime:/etc/localtime:ro --name=frigate --shm-size=1g blakeblackshear/frigate:reduce_false_positives-deec71f python3.7 process_clip.py --path /debug/false_positives/porch-1599803694.352481-73n4yg.mp4 --debug-path /debug/frames

Running with latest stable, I seem to get an error:

    capture_frames(ffmpeg_process, self.camera_name, self.frame_shape, self.frame_manager, self.frame_queue, 1, fps, skipped_fps, stop_event, detection_frame, current_frame)
TypeError: capture_frames() takes 10 positional arguments but 11 were given

With:

docker run --privileged --rm -v ${PWD}/config:/config -v /dev/bus/usb:/dev/bus/usb -v ${PWD}/debug:/debug -v /etc/localtime:/etc/localtime:ro --name=frigate --shm-size=1g blakeblackshear/frigate:stable-amd64 python3.8 process_clip.py --path /debug/false_negatives/trim.mp4 --debug-path /debug/frames
@blakeblackshear
Copy link
Owner

Try running an older version.

@jasonmadigan
Copy link
Author

Seems to work OK with the older frigate:reduce_false_positives-deec71f image. Be nice if it worked with current stuff too. Would also be nice if general config (maybe minus the camera config) could be used as well, rather than the hardcoded debugging config. Would help to be able to test things other than person movement etc.

@jasonmadigan
Copy link
Author

Closing this, but wanted to note that I've had good success in tuning Frigate with clips by streaming the clip as an RTSP stream on a "test" camera. Details here: https://gist.github.com/jasonmadigan/910d1fb0997c5fa69997340b84367aa8

luoj1 pushed a commit to luoj1/frigate that referenced this issue Apr 29, 2023
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

No branches or pull requests

2 participants