fix(libvideo2x): fixed errors in estimating the total number of frames

Signed-off-by: k4yt3x <i@k4yt3x.com>
This commit is contained in:
k4yt3x 2024-11-03 00:00:00 +00:00
parent fbe3b44139
commit 851f13bd4d
No known key found for this signature in database

View File

@ -55,9 +55,18 @@ static int process_frames(
spdlog::debug("Read total number of frames from 'nb_frames': {}", proc_ctx->total_frames); spdlog::debug("Read total number of frames from 'nb_frames': {}", proc_ctx->total_frames);
} else { } else {
spdlog::warn("Estimating the total number of frames from duration * fps"); spdlog::warn("Estimating the total number of frames from duration * fps");
// Calculate duration in seconds
double duration_secs = static_cast<double>(ifmt_ctx->streams[vstream_idx]->duration) * // Get the duration of the video
double duration_secs = 0.0;
if (ifmt_ctx->duration != AV_NOPTS_VALUE) {
duration_secs =
static_cast<double>(ifmt_ctx->duration) / static_cast<double>(AV_TIME_BASE);
} else if (ifmt_ctx->streams[vstream_idx]->duration != AV_NOPTS_VALUE) {
duration_secs = static_cast<double>(ifmt_ctx->streams[vstream_idx]->duration) *
av_q2d(ifmt_ctx->streams[vstream_idx]->time_base); av_q2d(ifmt_ctx->streams[vstream_idx]->time_base);
} else {
spdlog::warn("Unable to determine video duration");
}
spdlog::debug("Video duration: {}s", duration_secs); spdlog::debug("Video duration: {}s", duration_secs);
// Calculate average FPS // Calculate average FPS
@ -74,8 +83,8 @@ static int process_frames(
spdlog::debug("Unable to estimate the average frame rate with 'av_guess_frame_rate'"); spdlog::debug("Unable to estimate the average frame rate with 'av_guess_frame_rate'");
fps = av_q2d(ifmt_ctx->streams[vstream_idx]->time_base); fps = av_q2d(ifmt_ctx->streams[vstream_idx]->time_base);
} }
if (fps <= 0) { if (fps <= 0 || duration_secs <= 0) {
spdlog::debug("Unable to estimate the video's average frame rate"); spdlog::warn("Unable to estimate the video's average frame rate");
} else { } else {
// Calculate total frames // Calculate total frames
proc_ctx->total_frames = static_cast<int64_t>(duration_secs * fps); proc_ctx->total_frames = static_cast<int64_t>(duration_secs * fps);
@ -83,7 +92,7 @@ static int process_frames(
} }
// Check if the total number of frames is still 0 // Check if the total number of frames is still 0
if (proc_ctx->total_frames == 0) { if (proc_ctx->total_frames <= 0) {
spdlog::warn("Unable to determine the total number of frames"); spdlog::warn("Unable to determine the total number of frames");
} else { } else {
spdlog::debug("{} frames to process", proc_ctx->total_frames); spdlog::debug("{} frames to process", proc_ctx->total_frames);