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

Support piped input with QtAVPlayer #849

Merged
merged 1 commit into from
Dec 23, 2024
Merged

Support piped input with QtAVPlayer #849

merged 1 commit into from
Dec 23, 2024

Conversation

g-maxime
Copy link
Collaborator

@g-maxime g-maxime commented Dec 21, 2024

Needs a modified version of QtAVPlayer, discussion valbok/QtAVPlayer#507

Snapshot

Please test and also check stats coherency with olders versions

@g-maxime g-maxime marked this pull request as ready for review December 21, 2024 16:51
@g-maxime g-maxime force-pushed the pipe branch 2 times, most recently from 60ae139 to c0a03da Compare December 21, 2024 20:43
Signed-off-by: Maxime Gervais <gervais.maxime@gmail.com>
@dericed
Copy link
Member

dericed commented Dec 23, 2024

currently this doesn't crash but hangs during processing

$ ffmpeg -f lavfi -i mandelbrot -f lavfi -i sine -ac 2 -pix_fmt yuv422p10le -c:v ffv1 -c:a flac -t 3 test.mkv

$ cat test.mkv | /usr/local/bin/qcli -i - -a -video all -audio all -f signalstats+cropdetect+psnr+ebur128+aphasemeter+astats+ssim+idet+deflicker+entropy+entropy-diff -o test.mkv.qctools.mkv
qcli 1.3.20241221, Dec 21 2024 11:08:09
filters selected: signalstats cropdetect psnr ebur128 aphasemeter astats ssim idet deflicker entropy entropy-diff 

analyzing input file... -
.                                                  2 of 100 %

just stays at 2%

@dericed
Copy link
Member

dericed commented Dec 23, 2024

will merge, the filter chain issue seems different than the pipe issue

@dericed dericed merged commit 80ed24f into bavc:main Dec 23, 2024
10 checks passed
@dericed dericed mentioned this pull request Dec 23, 2024
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

Successfully merging this pull request may close these issues.

2 participants