FFBitrateViewer icon indicating copy to clipboard operation
FFBitrateViewer copied to clipboard

Add true fps

Open tormento opened this issue 11 months ago • 7 comments

Would be possibile to add a graph for the true fps of a video?

There are some mixed ones, such as 23.976 for the main video and 29.97 for opening and ending or much more complex ones where it varies depending on content, such as CGI parts or animated ones.

As FFBitrateViewer already does a scan with ffprobe, I think it could be feasible.

Thanks in advance for any effort ;)

tormento avatar Jan 24 '25 11:01 tormento

The feature you want sounds reasonable and should not be hard to implement. However, at the same time it is not exactly fit into programe named FFBitrateViewer. I will think about this. No any promises at this moment.

fifonik avatar Feb 20 '25 06:02 fifonik

To advocate my cause, please notice that in the frame based view, as far as I can understand — and I am probably wrong — the program implies that all the frames are of the same time lenght but it's not always correct. Perhaps a frame (and not time) based X axis could be more correct OR give every frame the appropriate time span according to real FPS. Moreover the mediainfo in the "path to file" should show the average real frame rate and not what the container reports.

tormento avatar Feb 20 '25 09:02 tormento

as far as I can understand — and I am probably wrong — the program implies that all the frames are of the same time lenght but it's not always correct

Correct, you are wrong :-P FFProbe returns start time & duration for every frame/packet. FFMetrics use this to draw per frame plot. Notice axis X is not frame numbers, but time and in my example line is not started from 0 (as video stream offset is also taken into account when "adjust" checkbox is not ticket):

Image

fifonik avatar Feb 20 '25 10:02 fifonik

Perhaps a frame (and not time) based X axis could be more correct

I do not think so as in this case user will not be able to see how long is the frame that take the size. I think 'Per second' is more relevant for situations that you talking about. It is possible to introduce plot "per average framerate" that probably what you want, but I'm not sure that could be used by many people.

fifonik avatar Feb 20 '25 10:02 fifonik

Moreover the mediainfo in the "path to file" should show the average real frame rate and not what the container reports.

Make sense. Will try to add this into the program (added issue #30).

fifonik avatar Feb 20 '25 10:02 fifonik

Correct, you are wrong :-P

😊

tormento avatar Feb 20 '25 10:02 tormento

mediainfo in the "path to file" should show the average real frame rate

Implemented in v0.9.3b1

fifonik avatar Feb 21 '25 09:02 fifonik