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

--audio-file with ordered chapters cause strange behaviour #1018

Closed
Bilalh opened this issue Aug 15, 2014 · 5 comments
Closed

--audio-file with ordered chapters cause strange behaviour #1018

Bilalh opened this issue Aug 15, 2014 · 5 comments

Comments

@Bilalh
Copy link
Contributor

Bilalh commented Aug 15, 2014

When using a mkv file with ordered chapter (for the opening and ending) and using an external audio file (from an mp4):

  • audio from the mp4 is restarted (i.e played from the beginning) when playing the ordered chapter
  • When the ordered chapter is finished, the audio from the mp4 is reset back to location before the ordered chapter was played

I would have expected the audio (from the mp4) to not be effected by the ordered chapter.

command used

mpv  anime.mkv  --audio-file=anime_with_eng_audio.mp4

version info

mpv --version
mpv git-364e062 (C) 2000-2014 mpv/MPlayer/mplayer2 projects
 built on 2014-08-15T18:12:32
ffmpeg library versions:
   libavutil       52.92.100
   libavcodec      55.69.100
   libavformat     55.48.100
   libswscale      2.6.100
   libavfilter     4.11.100
   libavresample   1.3.0
@ChrisK2
Copy link
Member

ChrisK2 commented Aug 15, 2014

So the audio-file position should be aligned with the virtual timeline, not the position inside each segment file. Makes sense.

@Bilalh
Copy link
Contributor Author

Bilalh commented Aug 15, 2014

@ChrisK2 that a much better description

@ghost
Copy link

ghost commented Aug 15, 2014

So, like the change that was made for external subs?

@Bilalh
Copy link
Contributor Author

Bilalh commented Aug 15, 2014

yes

@ghost ghost closed this as completed in fa7c421 Aug 15, 2014
@Bilalh
Copy link
Contributor Author

Bilalh commented Aug 15, 2014

I tested with fa7c421 and it works well, thanks.

This issue was closed.
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

No branches or pull requests

2 participants