Brandon Castellano
Brandon Castellano
There's quite a bit of weird path manipulation happening within the codebase using `os.path`, which should probably be changed to use `pathlib.Path`. I'll try to make path handling consistent across...
> EDIT: After looking at the mediainfo of the input file, it reports variable frame rate for the video. Could it explain the mismatch between the frames written in SceneTiming.csv...
If you are using OpenCV for video input, you could try the [`VideoCaptureAdapter`](https://www.scenedetect.com/docs/latest/api/backends.html#scenedetect.backends.opencv.VideoCaptureAdapter) which is intended for this use case. Could you let me know if that works for you?...
> Now, I'm testing scene detection time via a callback function in PySceneDetect. I'm wondering if a custom callback passed to detect_scenes is called after each scene cut. In scene_manager.py,...
I've been experimenting with overhauling the PySceneDetect timecode interfaces [in a separate branch](https://github.com/Breakthrough/PySceneDetect/blob/issue-168-time-overhaul/scenedetect/time.py), but don't have a timeline for that yet (it's a rather major update). > You may want...
> But, since PySceneDetect use a video, it can technically get all the pts + timescale (or timebase) of the video PySceneDetect *does* use a fractional representation for timecodes, although...
As per #168, in v0.7, frame rates will now be stored as fractions internally, and there will be support for getting exact framerates if you use the PyAV backend (since...
Perhaps this should be an API only change and not affect the CLI. *API Sketch* Ideally we could have a concept of data sources (detectors) and filters (what scene manager...
Right now it would be a bit verbose but it is possible. With the threshold detector, you can find ranges where there are black frame sequences. However, by default, the...
> The table of results looks promising and the approach is interesting. They haven't published their code yet (coming soon according to their homepage). Ideally, they could also release a...