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

ab loop not working as expected #10640

Closed
hastyeagle opened this issue Sep 13, 2022 · 8 comments
Closed

ab loop not working as expected #10640

hastyeagle opened this issue Sep 13, 2022 · 8 comments
Labels

Comments

@hastyeagle
Copy link

hastyeagle commented Sep 13, 2022

Important Information

Provide following Information:

  • mpv version: mpv 0.34.1
  • Platform and Version: FreeBSD 13.1-RELEASE-p2
  • Source of the mpv binary: Package

Reproduction steps

Open an MP4 file.
Press "l" to set the A point.
Wait an indeterminte amount of time and press "l" again to set the B point.

I did have another set of files (security camera recordings made via RTSP and ffmpeg) that work as expected. Other MP4 files that I've tried downloaded from Youtube experience the ab loop failure behavior.

Expected behavior

After pressing "l" to set the B point, mpv should jump back to A and play until it reaches B and then repeat the loop.

Actual behavior

After setting the B loop point, mpv will switch back to the A point, but pauses. Pressing play/pause again doesn't start playback. If you press "l" again to clear the ab loop, mpv can start playing again.

Log file

Log when ab loop doesn't work: https://storage.googleapis.com/iqgxbry4pqfbybtp/log.txt

@hastyeagle
Copy link
Author

Just wanted to check in on this. Do I need to add anything or can I provide any other information that would be helpful? Can others test this on various MP4 files to see if they experience the same issue?

As I mentioned, it's not on all the MP4 files I've tried, but a large portion do exhibit this issue. I'm guessing it's something with the codec used, but I'm not sure yet.

@Dudemanguy
Copy link
Member

Could you upload a sample where it doesn't work for you?

@hastyeagle
Copy link
Author

Attached is a random sample I grabbed from Youtube. It's poor quality just to get a small file size.

test.mp4

@Dudemanguy
Copy link
Member

I have no problems with a-b loop on that clip. Perhaps the ao isn't handling the loop correctly? Could you try a different ao other than oss and see if you still get the same issue?

@hastyeagle
Copy link
Author

That looks like the issue. Switching to ao=null or ao=pulse and a-b loop works fine.

@Dudemanguy
Copy link
Member

So presumably this:

blocked, waiting for old audio to play

blocks forever on oss.

@noiseless
Copy link
Contributor

Also noted the presence of this bug when working on fixing a similar bug in ao_sndio. Perhaps my changes for ao_sndio will be something useful for fixing ao_oss.
(sorry if I'm distracting you unnecessarily)

cc: @rozhuk-im

@rozhuk-im
Copy link
Contributor

Testers are welcome :)

rozhuk-im added a commit to rozhuk-im/mpv that referenced this issue Feb 12, 2023
rozhuk-im added a commit to rozhuk-im/mpv that referenced this issue Feb 16, 2023
FarisR99 pushed a commit to FarisR99/mpv that referenced this issue May 20, 2023
dyphire pushed a commit to dyphire/mpv that referenced this issue Jul 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants