Page 1 of 1

thd+ac3 file in dgdecnv 240

Posted: Tue Apr 05, 2022 2:32 pm
by renols
Hi.

Sorry if this should have been in another thread.

Just updated to 240, and oh-my.

Could the change to write thd+ac3 track please be either reverted, or made optional?

It takes 3-4 mínutes to do that file, and with all the other speed optimazations that has been done over the past year or so, for many people I believe it is a waste of time to write that file.

I surely have never used or missed a thd+ac3 file.

Could it at least be made optional to create that file?

Imo this is a step backwards, to make someone else happy :-)

May have sounded like a good plan, but for me, not so much in real life.

renols

thd+ac3 file in dgdecnv 240

Posted: Wed Apr 06, 2022 7:57 am
by Rocky
We try to make everyone happy.

But yeah, I wondered why the merging operation is so slow, given how fast the gaps correction runs. I need to look into that. For now, I will add an option as you propose.

thd+ac3 file in dgdecnv 240

Posted: Wed Apr 06, 2022 10:09 am
by renols
Hi.

Thank you for adding an option.

Just did a quick test with a 4k movie a bit more than 2 hours long.

2.39 finnished in 2m 5s

2.40 changed to creating thd+ac3 after 2m 3s and then finnished after an additional 1m 37s. Almost the same time as it took to demux.

That is not 3-4 minutes like started yesterday, but it felt like forever, when just sitting there and waiting :-)

The thd+ac3 file is just above 7 GB in size.

Even if it only took 30 seconds for the thd+ac3 file, it is still 25% longer than without it. I am not sure, but I think that most users will just use the thd and the embedded ac3 to mux into mkv.

renols

thd+ac3 file in dgdecnv 240

Posted: Wed Apr 06, 2022 3:16 pm
by Guest
Actually a lot of us use BD structure
And most people do NOT use mkv
MP4 is the most commonly used container

thd+ac3 file in dgdecnv 240

Posted: Thu Apr 07, 2022 3:58 pm
by Guest 2
Err... where is the option? :scratch:

thd+ac3 file in dgdecnv 240

Posted: Thu Apr 07, 2022 4:40 pm
by Rocky
Patience, it is in progress.

thd+ac3 file in dgdecnv 240

Posted: Fri Apr 08, 2022 7:44 am
by Rocky
Released as build 241. Marking resolved.

thd+ac3 file in dgdecnv 240

Posted: Fri Apr 08, 2022 12:43 pm
by Rocky
Updated 241 to have thd+ac3 on by default.

thd+ac3 file in dgdecnv 240

Posted: Sat Apr 16, 2022 10:43 am
by Rocky
I have made the merge operation 10 times faster. Applied to thdmerge (1.2), DGDemux/DGDemuxGUI (1.0.0.61), and DGIndexNV (242).

Simply replaced low-level IO with stream IO. YMBDN ;)

thd+ac3 file in dgdecnv 240

Posted: Sat Apr 16, 2022 12:06 pm
by DG
Great stuff, Rocky. Thank you.