Subtitle demux issue

Post by **Rocky** » Wed Nov 23, 2022 11:39 am

OK, the fix for Peter was a regression. When I back it out your disk works fine. I have to re-rip his disk and re-investigate his issue.

downloaded · Post by **downloaded** » Wed Nov 23, 2022 3:44 pm

Sorry for the late reply and thank you for your time looking into this. It's very appreciated!

Post by **Rocky** » Thu Nov 24, 2022 6:19 am

Thanks for reply. I will try to fix this on Friday (tomorrow). I'm away for the holiday today.

Post by **Rocky** » Thu Nov 24, 2022 7:06 am

Decided to slipstream a temporary fix into 63, so re-download and test please. I'll come back to Peter's issue tomorrow. It seems like a rare corner case.

downloaded · Post by **downloaded** » Fri Nov 25, 2022 12:13 pm

Thank you for taking time to work on it.

I will test and let you know the result.

Post by **Rocky** » Fri Nov 25, 2022 4:01 pm

With the corrected version, i.e., the version 63 currently online, I am unable to reproduce Peter's original problem. So unless he posts again about it, I'll consider this resolved.

Peter · Post by **Peter** » Sun Dec 04, 2022 9:32 pm

Apologies for the delay. Using DGDemux v1.0.0.63, I still get the same behavior. In the demuxed subtitles, caption 1397 (the last caption, which was from the last m2ts file, 00010.m2ts) has the start/end times of 00:00:01.043/00:00:01.960.

Opening the file in BDSup2Sub/BDSup2Sub++ yields the following:
WARNING: end time of frame 1396 > start time of next frame

Post by **Rocky** » Mon Dec 05, 2022 9:22 am

Did you re-download DGDemux 63 to be sure you have the latest version?

Peter · Post by **Peter** » Mon Dec 05, 2022 9:51 am

Rocky wrote: ↑
Mon Dec 05, 2022 9:22 am
Did you re-download DGDemux 63 to be sure you have the latest version?

Binary date is 11/29. Is that not the latest?

Post by **Rocky** » Mon Dec 05, 2022 9:58 am

That is the correct one. I'll try again to duplicate this.

Post by **Rocky** » Mon Dec 05, 2022 11:03 am

Duplicated and fixed locally. Have to re-test All Creatures with -sf and any other disk without -sf. Then I'll give you a test version.

Post by **Rocky** » Mon Dec 05, 2022 11:43 am

@Peter and downloaded:

OK guys, I think I have fixed it correctly this time. Let's see. Please re-download build 63 and update DGDemux.exe. If you find it working correctly please let me know and I'll fix it for DGIndexNV also.

Peter · Post by **Peter** » Mon Dec 05, 2022 4:01 pm

It looks to be correct now, for the issue I reported. Thanks!

Post by **Rocky** » Mon Dec 05, 2022 5:03 pm

Great to hear, thank you. I tested All Creatures successfully too so it looks like this version is a keeper.

Post by **Rocky** » Wed Dec 07, 2022 9:25 am

DGIndexNV and the included DGDemux were updated in the 246 distribution to include this fix.