XenonMKV Forum
July 30, 2010, 08:12:54 pm *
Welcome, Guest. Please login or register.
Did you miss your activation email?

Login with username, password and session length
News: XenonMKV Home · Downloads · FAQ · Screenshots
 
   Home   Help Search Login Register  
Pages: 1 [2] 3 4 5
  Print  
Author Topic: Audio always out of sync  (Read 5125 times)
ddiangelo
Forum Member
*
Offline Offline

Posts: 8


View Profile
« Reply #15 on: January 06, 2009, 01:57:13 am »

It's an open-source project hosted at SourceForge, so sure, source code is available... you just need a Subversion client and a SourceForge account (which supports OpenID as well) along with the compiler. I happen to have access to Microsoft Visual Studio 2005 on my work machine - which according to Jake is required to compile the app (it's written in VB using .NET Framewrok 2.0). I happened to use the AnkhSVN add-in to access the Subversion repository from within Visual Studio, but there are other stand-alone clients and a command-line compiler is available with the .NET Framework SDK (free to download from MS - msdn.microsoft.com) if you don't have access to Visual Studio.

I'm wondering if the problem isn't coming from the FPS value being rounded to an integer for some reason -- i.e. 23.976 rounded to 24 will eventually get the audio out-of-sync to the point where it's noticable. Based on my debugging session, that shouldn't happen, but I'm looking at the latest code - the same bug may have existed in earlier builds before Jake implemented the default 23.976 value.

I personally haven't dug too deep into this issue - I only had 3 HDTV rips that I found with the issue. I do have a 1080p BluRay file with a DTS audio track (4.5GB - 23.976FPS). I'll start running that through the non-patched XMKV and see if the audio gets out of sync and report back tomorrow.

-- Drew
Logged
ddiangelo
Forum Member
*
Offline Offline

Posts: 8


View Profile
« Reply #16 on: January 07, 2009, 06:41:20 pm »

Well - my 23.976fps file with DTS audio worked just fine with build 35.

norrgrd, can you post the info about one of the MKVs you're working with (i.e. the text spit out from the 'MKV Information' item in the Tools menu)? That might give us a clue about what might be happening...i.e. some difference between what you're working with and what RadioHowie and I are seeing with our files that are working (and my files that did have the problem).

-- Drew
Logged
norrgrd
Forum Member
*
Offline Offline

Posts: 12


View Profile
« Reply #17 on: January 07, 2009, 07:13:20 pm »

Alright here you go. Everything from temp.txt;

+ EBML head
|+ Doc type: matroska
|+ Doc type version: 1
|+ Doc type read version: 1
+ Segment, size 64460182
|+ Seek head (subentries will be skipped)
|+ EbmlVoid (size: 4012)
|+ Segment information
| + Timecode scale: 1000000
| + Muxing application: libebml v0.7.7 + libmatroska v0.8.1
| + Writing application: mkvmerge v2.0.2 ('You're My Flame') built on Feb 21 2007 23:40:55
| + Duration: 63.400s (00:01:03.400000000)
| + Date: Sun Jan 04 19:35:35 2009 UTC
| + Title: blahblahblah (2008)
| + Segment UID: 0x8f 0xd4 0x91 0x79 0xb2 0x54 0x30 0xd2 0xb5 0x9c 0xed 0x93 0xfb 0x63 0xca 0x91
|+ Segment tracks
| + A track
|  + Track number: 1
|  + Track UID: 1
|  + Track type: video
|  + Enabled: 1
|  + Default flag: 1
|  + Forced flag: 0
|  + Lacing flag: 0
|  + MinCache: 1
|  + Timecode scale: 1.000000
|  + Max BlockAddition ID: 0
|  + Codec ID: V_MPEG4/ISO/AVC
|  + Codec decode all: 1
|  + CodecPrivate, length 169
|  + Default duration: 41.708ms (23.976 fps for a video track)
|  + Language: eng
|  + Video track
|   + Pixel width: 1280
|   + Pixel height: 534
|   + Interlaced: 0
|   + Display width: 640
|   + Display height: 267
| + A track
|  + Track number: 2
|  + Track UID: 15035289
|  + Track type: audio
|  + Enabled: 1
|  + Default flag: 1
|  + Forced flag: 0
|  + Lacing flag: 1
|  + MinCache: 0
|  + Timecode scale: 1.000000
|  + Max BlockAddition ID: 0
|  + Codec ID: A_DTS
|  + Codec decode all: 1
|  + Language: eng
|  + Audio track
|   + Sampling frequency: 48000.000000
|   + Channels: 6
| + A track
|  + Track number: 3
|  + Track UID: 3146169787
|  + Track type: audio
|  + Enabled: 1
|  + Default flag: 0
|  + Forced flag: 0
|  + Lacing flag: 1
|  + MinCache: 0
|  + Timecode scale: 1.000000
|  + Max BlockAddition ID: 0
|  + Codec ID: A_VORBIS
|  + Codec decode all: 1
|  + CodecPrivate, length 3562
|  + Language: eng
|  + Name: Commentary
|  + Audio track
|   + Sampling frequency: 48000.000000
|   + Channels: 2
| + A track
|  + Track number: 4
|  + Track UID: 3409595326
|  + Track type: subtitles
|  + Enabled: 1
|  + Default flag: 1
|  + Forced flag: 0
|  + Lacing flag: 0
|  + MinCache: 0
|  + Timecode scale: 1.000000
|  + Max BlockAddition ID: 0
|  + Codec ID: S_TEXT/ASS
|  + Codec decode all: 1
|  + CodecPrivate, length 810
|  + Language: eng
| + A track
|  + Track number: 5
|  + Track UID: 4001495939
|  + Track type: subtitles
|  + Enabled: 1
|  + Default flag: 0
|  + Forced flag: 0
|  + Lacing flag: 0
|  + MinCache: 0
|  + Timecode scale: 1.000000
|  + Max BlockAddition ID: 0
|  + Codec ID: S_TEXT/ASS
|  + Codec decode all: 1
|  + CodecPrivate, length 784
|  + Language: eng
|  + Name: Commentary
| + A track
|  + Track number: 6
|  + Track UID: 3531003034
|  + Track type: subtitles
|  + Enabled: 1
|  + Default flag: 0
|  + Forced flag: 0
|  + Lacing flag: 0
|  + MinCache: 0
|  + Timecode scale: 1.000000
|  + Max BlockAddition ID: 0
|  + Codec ID: S_TEXT/ASS
|  + Codec decode all: 1
|  + CodecPrivate, length 810
|  + Language: dan
| + A track
|  + Track number: 7
|  + Track UID: 693334147
|  + Track type: subtitles
|  + Enabled: 1
|  + Default flag: 0
|  + Forced flag: 0
|  + Lacing flag: 0
|  + MinCache: 0
|  + Timecode scale: 1.000000
|  + Max BlockAddition ID: 0
|  + Codec ID: S_TEXT/ASS
|  + Codec decode all: 1
|  + CodecPrivate, length 810
|  + Language: fin
| + A track
|  + Track number: 8
|  + Track UID: 4272085678
|  + Track type: subtitles
|  + Enabled: 1
|  + Default flag: 0
|  + Forced flag: 0
|  + Lacing flag: 0
|  + MinCache: 0
|  + Timecode scale: 1.000000
|  + Max BlockAddition ID: 0
|  + Codec ID: S_TEXT/ASS
|  + Codec decode all: 1
|  + CodecPrivate, length 810
|  + Language: nor
| + A track
|  + Track number: 9
|  + Track UID: 3685052842
|  + Track type: subtitles
|  + Enabled: 1
|  + Default flag: 0
|  + Forced flag: 0
|  + Lacing flag: 0
|  + MinCache: 0
|  + Timecode scale: 1.000000
|  + Max BlockAddition ID: 0
|  + Codec ID: S_TEXT/ASS
|  + Codec decode all: 1
|  + CodecPrivate, length 784
|  + Language: rum
| + A track
|  + Track number: 10
|  + Track UID: 569734324
|  + Track type: subtitles
|  + Enabled: 1
|  + Default flag: 0
|  + Forced flag: 0
|  + Lacing flag: 0
|  + MinCache: 0
|  + Timecode scale: 1.000000
|  + Max BlockAddition ID: 0
|  + Codec ID: S_TEXT/ASS
|  + Codec decode all: 1
|  + CodecPrivate, length 810
|  + Language: spa
| + A track
|  + Track number: 11
|  + Track UID: 2609895870
|  + Track type: subtitles
|  + Enabled: 1
|  + Default flag: 0
|  + Forced flag: 0
|  + Lacing flag: 0
|  + MinCache: 0
|  + Timecode scale: 1.000000
|  + Max BlockAddition ID: 0
|  + Codec ID: S_TEXT/ASS
|  + Codec decode all: 1
|  + CodecPrivate, length 810
|  + Language: swe
|+ EbmlVoid (size: 1024)
|+ Attachments
| + Attached
|  + File name: CronosPro-Bold.ttf
|  + Mime type: application/x-truetype-font
|  + File data, size: 108928
|  + File UID: 2410273686
| + Attached
|  + File name: CronosPro-Semibold.ttf
|  + Mime type: application/x-truetype-font
|  + File data, size: 109724
|  + File UID: 3724802181
| + Attached
|  + File name: CronosPro-SemiboldIt.ttf
|  + Mime type: application/x-truetype-font
|  + File data, size: 135684
|  + File UID: 3883746487
|+ Cluster


And this is from another one:

+ EBML head
|+ Doc type: matroska
|+ Doc type version: 1
|+ Doc type read version: 1
+ Segment, size 52798280
|+ Seek head (subentries will be skipped)
|+ EbmlVoid (size: 4027)
|+ Segment information
| + Timecode scale: 1000000
| + Muxing application: libebml v0.7.7 + libmatroska v0.8.1
| + Writing application: mkvmerge v2.0.2 ('You're My Flame') built on Feb 21 2007 23:40:55
| + Duration: 60.816s (00:01:00.816000000)
| + Date: Mon Nov 12 18:07:39 2007 UTC
| + Segment UID: 0xbc 0xb9 0x11 0x48 0xd3 0x0d 0xa7 0x6f 0xae 0x7f 0xdc 0xce 0xb4 0x0e 0x6f 0xf0
|+ Segment tracks
| + A track
|  + Track number: 1
|  + Track UID: 2399976121
|  + Track type: audio
|  + Enabled: 1
|  + Default flag: 1
|  + Forced flag: 0
|  + Lacing flag: 1
|  + MinCache: 0
|  + Timecode scale: 1.000000
|  + Max BlockAddition ID: 0
|  + Codec ID: A_AC3
|  + Codec decode all: 1
|  + Default duration: 32.000ms (31.250 fps for a video track)
|  + Language: eng
|  + Audio track
|   + Sampling frequency: 48000.000000
|   + Channels: 2
| + A track
|  + Track number: 2
|  + Track UID: 1
|  + Track type: video
|  + Enabled: 1
|  + Default flag: 1
|  + Forced flag: 0
|  + Lacing flag: 0
|  + MinCache: 1
|  + Timecode scale: 1.000000
|  + Max BlockAddition ID: 0
|  + Codec ID: V_MPEG4/ISO/AVC
|  + Codec decode all: 1
|  + CodecPrivate, length 172
|  + Default duration: 40.000ms (25.000 fps for a video track)
|  + Language: eng
|  + Video track
|   + Pixel width: 1280
|   + Pixel height: 720
|   + Interlaced: 0
|   + Display width: 16
|   + Display height: 9
|+ EbmlVoid (size: 1024)
|+ Cluster

I have plenty more if it helps.

EDIT: Could you maybe send me one of your mkv's and I could try that one out?
« Last Edit: January 07, 2009, 07:19:05 pm by norrgrd » Logged
ddiangelo
Forum Member
*
Offline Offline

Posts: 8


View Profile
« Reply #18 on: January 08, 2009, 01:25:28 am »

Well - I had a whole post written out and tried to attach a file and got an error when it submitted - guess the forum wasn't setup to accept attachments Huh

Anyway -- there are a few differences between what I see in your files and the ones I have, but nothing that appears to be significant (at least without further review of the code).

I'd send you a file or two, but none of the MKVs I currently have are short clips, plus your email address is hidden in the forum profile. I was trying to attach my patched EXE for you to test with (only 73KB in a zip), but I can't do that either.

Why don't you send me a few of your short MKV clips and I can test them to see what happens? Email is in my profile...

-- Drew
Logged
norrgrd
Forum Member
*
Offline Offline

Posts: 12


View Profile
« Reply #19 on: January 09, 2009, 12:51:12 pm »

Hm.

I tried pulling some files now too from the temp folder and muxed with mp4box (same version) and voila, it works! Though some require me to use -fps 23.976, some require me to use -fps 25 and some -fps 29.97.

The -fps 25 one is the tricky one really since the default, if not specified in mp4box, is -fps 25 then why is that one not working with XMKV? And if your findings are true, ddiangelo, about the default coded by Jake to be 23.976 fps then why isn't that specific sample working with XMKV?

I'm very confused about all this but at least I figured out how to get a flawless mp4 and that's what most important to me.
Logged
ddiangelo
Forum Member
*
Offline Offline

Posts: 8


View Profile
« Reply #20 on: January 10, 2009, 12:40:43 pm »

That's good news norrgrd! At least you got a way to get a good MP4... I was thinking it might be helpful to have XMKV spit out the command line params it uses when it calls out to MP4Box (probably the other tools as well)... hopefully I'll get a chance to add that to the code sometime this weekend. That would at least give you an idea what XMKV is trying to do...

-- Drew
Logged
ddiangelo
Forum Member
*
Offline Offline

Posts: 8


View Profile
« Reply #21 on: January 13, 2009, 07:19:11 pm »

Was just looking through the code and found that the feature I mentioned - spitting out the command line params - is already available with Jake's original code. Just check the box for 'Log all tool output' and you should see extra log entries when the process gets started by XMKV... I'll test it later when I am on my home machine.

-- Drew
Logged
norrgrd
Forum Member
*
Offline Offline

Posts: 12


View Profile
« Reply #22 on: January 19, 2009, 01:12:04 pm »

Alright I have some new input.

I think I have narrowed down the issue. The output log shows that the fps parameter for mp4box is using a comma, like this "-fps 23,976". If I pull those files from the temp folder during the conversion and I use the exact same mp4box-parameters except for the comma, and instead I am using a dot, like this "-fps 23.976", it works great!

A problem that still persists is that this method above only work for clips with 23.976 fps. Any other movie I have to pull them and change the fps to either 25 or 29.97 since mp4box always use "-fps 23,976", at least for me.

Am I onto something? Could anyone else try it out?
Logged
jbillo
Administrator
Forum Member
*****
Offline Offline

Posts: 504


View Profile WWW
« Reply #23 on: January 19, 2009, 06:40:24 pm »

Hi all,

Just got a chance to look at this thread. I've been swamped with school and my real job so far, but my buddy Phil suggested I take a look at some new stuff for XenonMKV.

Drew - thanks a lot for your input and suggested fixes. Do you have your modified source files, or would you like commit access to SVN?

norrgrd: Are you running a regional/localized version of Windows that uses commas instead of decimal places? If so, we may be able to automate this based on system language.

In the meantime, I'll implement the Double.TryParse fix and see if I can release a new standalone EXE for people who don't want to/can't compile this themselves.
Logged
jbillo
Administrator
Forum Member
*****
Offline Offline

Posts: 504


View Profile WWW
« Reply #24 on: January 19, 2009, 06:58:26 pm »

Attempted to re-enable attachment support.
Logged
BagronkeN
Forum Member
*
Offline Offline

Posts: 4


View Profile
« Reply #25 on: January 20, 2009, 10:20:30 am »

Hi all!

I've been lurking the forum having this same problem for about a week, and I can confirm that changing the regional settings in Windows to use "." as the decimal symbol instead of "," (which is default in many languages) will solve the problem.

So feel free to change this on your Windows installation (Control Panel -> Regional and Language Options) until jbillo can make a new build that counters the problem.

Thanks for an awesome application!
Logged
norrgrd
Forum Member
*
Offline Offline

Posts: 12


View Profile
« Reply #26 on: January 22, 2009, 04:34:40 pm »

Thanks for the input jbillo,

I can also confim that changing the regional settings in Windows to use "." instead of "," solved the problem!

I feel pretty stupid right now, not to have thought about that earlier, since the Windows version was the only common between the systems.

I can finally go to sleep at night!
Logged
jbillo
Administrator
Forum Member
*****
Offline Offline

Posts: 504


View Profile WWW
« Reply #27 on: January 25, 2009, 05:16:53 pm »

Interesting that mp4box takes the decimal separator based on system language. In any event I'll try and have a patch out for this problem along with the Double.TryParse item - a friend and I are actually restructuring the entire codebase, but it'd be nice to have a fixed EXE for people with this problem.
Logged
lars667
Forum Member
*
Offline Offline

Posts: 2


View Profile
« Reply #28 on: February 02, 2009, 02:39:49 am »

Thanks, this solution also works for me. I have Norwegian setup on my XP box, and changed from ',' to '.'. Ah, the beauty of open and free software!
Logged
carpenn
Forum Member
*
Offline Offline

Posts: 5


View Profile
« Reply #29 on: February 18, 2009, 08:20:23 pm »

Any word on a fix for the out of sync issue? I know that any time i try to convert any 29.97 fps video it slows the video down in the coverted file and it's way off.
Logged
Pages: 1 [2] 3 4 5
  Print  
 
Jump to:  

Powered by MySQL Powered by PHP Powered by SMF 1.1.11 | SMF © 2006-2009, Simple Machines LLC Valid XHTML 1.0! Valid CSS!