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

Adding a 'frames' array in returned data for ffprobe() call with '-show_frames' option #1153

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pandominox
Copy link

@pandominox pandominox commented Oct 8, 2021

Adding a 'frames' array in the returned data when calling ffprobe() with the '-show_frames' option, containing the entire set of information about video/audio frames. The current call of ffprobe() allows '-show_frames' option to be added, but the returned data does not contain this information, because of lack of this property (as array of 'frames') in the 'data' variable and not parsing the 'frame' blocks in the data returned in 'stdout' from the 'ffprobe' command.

Add array of "frames" in returned data in case of call ffprobe() with option "-show_frames", containing all set of information about video/audio frames. Current call of ffprobe() allows adding "-show_frames" option, but returned data doesn't contain this information, because of lack this property (as array of 'frames') in 'data' variable and not parsing 'frame' blocks in data returned in 'stdout' from ffprobe command.
@pandominox pandominox changed the title Add array of "frames" for ffprobe option "-show_frames" Adding a 'frames' array for ffprobe() call with '-show_frames' option Oct 15, 2021
@pandominox pandominox changed the title Adding a 'frames' array for ffprobe() call with '-show_frames' option Adding a 'frames' array in returned data for ffprobe() call with '-show_frames' option Oct 15, 2021
@prsauer
Copy link

prsauer commented Nov 6, 2023

+1 for adding this - we have a patch package doing this in our repo now!

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