Skip to content

Conversation

@akolpakov
Copy link
Contributor

See issue #187

@kkroening
Copy link
Owner

kkroening commented Apr 12, 2019

Looks great. Not sure why CI is failing - probably not related to your PR though. I'll merge once I have time to fix CI.

Thanks for the changes.

@cbitterfield
Copy link

This doesn't seem to be working:

extra = {"sexagesimal":None}

try:
probe = ffmpeg.probe(FILE,extra)
except ffmpeg.Error as e:
print(e.stderr, file=sys.stderr)
sys.exit(1)

Doesn't seem to work; however, it looks like it should

@akolpakov
Copy link
Contributor Author

This doesn't seem to be working:

extra = {"sexagesimal":None}

try:
probe = ffmpeg.probe(FILE,extra)
except ffmpeg.Error as e:
print(e.stderr, file=sys.stderr)
sys.exit(1)

Doesn't seem to work; however, it looks like it should

ffmpeg.probe accepts params as **kwargs. Can you try:

probe = ffmpeg.probe(FILE, sexagesimal=None)?

@153957
Copy link
Contributor

153957 commented Aug 5, 2019

probe = ffmpeg.probe(FILE, **extra) should also work.

@akolpakov
Copy link
Contributor Author

probe = ffmpeg.probe(FILE, **extra) should also work.

Yes, it is the same. But your example in the previous message is without **

Just tested

probe = ffmpeg.probe(FILE, sexagesimal=None)

It work properly. It shows duration in format 'duration': '0:01:33.994000'

@cbitterfield
Copy link

let me check. Got sidetracked on another project.

@cbitterfield
Copy link

That worked. My programming skills have also improved. I am starting to get a better understanding of the API.

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.

4 participants