@neuron2 : Bug in DGindex with bad .ts file

[Dark-Cracker]

Hi,

i have a problem to create a .d2v file with a .ts file (apparently my friend have made a bad cut of the full file, he have try to get only the trailer part).

i explain the bug :
when i open the .ts file with dgindex it do nothink and when i click to get the PID value it say me my file is not a transport stream.

i have try to open the same file with DVD2AVI_NIC version and it have open the .ts file correctly (but failed to demux audio track).

i think this problem is because the .ts file have not a correct strat frame and this bug should be easy to fix.

i have host the first 4MB of the bugged .ts file i hope this will help neuron2 to fix this bug :)

URL :
-----
http://dark.pluridis.org/neuron2/bug_short_sample.zip

PS : dvd2avi_nic say me the video pid is at 10 and audio pid at 11

i hope this will help.


Keep up the good work.
Bye.

[Guest]

That's weird. When I run my debug executable directly, it fails as you describe. But when I run it through VC6++ it works fine. Is that the Sears building in Chicago?

My General Parser utility seems quite happy with it, too.

I never saw anything fail like this. I will try to figure it out.

EDIT: I'm having to debug with printf's via OutputDebugString(). :confused:

[Guest]

Hoo boy, that was a tough one. You can tell because it is 3:00am. :eek:

I finally discovered through OutputDebugString() that my desktop shortcut icon for DGIndex was using a different INI file from the executable in my debug directory, even though the shortcut pointed to the same executable (the shortcut makes an INI file on the desktop). That explained the difference between behavior in and outside the debugger!

The debugger's ini file coincidentally already had 0x10/0x11 as the PIDs! So I did this: Start DGIndex but don't load the file. Go into the set PIDs menu and set 0x10/0x11. Then load the file. Don't do anything but wait patiently. After some parsing, the GOP warning box will pop up and you'll get the video. I demuxed the audio just fine.

The reason it did not work with the PIDs not set was that while parsing for good video start codes, it somehow stumbled on what looked like a PACK_START code. That's for a program stream so it royally messes things up. I'm assessing what to do about this and why there appears to be a PACK_START code. Maybe it is just coincidental data. I need to investigate more about that.

At least you have a workaround. :)

[Dark-Cracker]

hi,

thank u for you quick answer :) you solution (set pid at 10 & 11 before open the file) have work like a charm :) i hope you will find this bug and fix it.
a word about the PACK_START code, my friend have send me 3 trailers and they all have the same problem with dgindex, i can host the 2 other headers if this can help you to solve this problem.
however i don't know which tool he have used to cut this .ts file and it will not be online before a long period.

PS : the sample was from the IRobot movie so the first frame is surely chicago :)

Bye.

[Guest]

Mystery solved!

DGIndex defaults to pids:

video 0x11
audio 0x14

But your stream has:

video 0x10
audio 0x11

That means DGIndex starts parsing the audio stream as a video stream. The audio AC3 private stream has data that appears as a video pack start code, so DGIndex thinks he has found a video program stream. I looked at the MPEG2 spec Part 1 and it says in Annex G that emulation of start codes is possible in audio and data streams, so it is not surprising that this could happen.

I handled this by ignoring the pseudo pack start and killing the decode thread when I know I've got a transport stream. I also pop up a message box saying "Strange video data! Check your PIDs." After you dismiss that, you can detect PIDs and proceed normally.

So it's a pathological case caused by trying to decode the audio stream as the video stream!

I'll put the fix in the next release. Thank you for calling this to my attention. It was an interesting problem.

Oh, here's a question for you. How do you get I Robot in 1920 x 1088 HD? :confused:

[Dark-Cracker]

happy to see you have fix this bug :) and that i have be usefull :)

Keep up the good work :)
Bye.

[Guest]

Just for you, Dark-Cracker, I've placed a DGIndex 1.0.13b1 beta here:

http://neuron2.net/fixd2v/DGIndex1.0.13b1.zip

It has your fix. It also has some other changes I will describe later. Let me know if it appears I have broken anything. Thank you.

And, oh, did you not see my question about I Robot?

[Dark-Cracker]

lol sorry i have not see you question, in fact it's an australian friend who have send me the sample, in fact it's not the IRobot movie but only the trailer.

Thank you for the fixed .exe file, i will try it and let you know if i encounter some problems :)

Bye.

[Nic]

@Dark-Cracker: DVD2AVI_Nic seems to demux the audio fine (Load File, Select Audio->Track Number->Track 1, Select File->Demux/Convert Audio Track...Did that not work for you?)

-Nic

[Dark-Cracker]

Hi Nic,

i will test this at the weekend (i am a school for the moment) but it seems to me it has failed but i have test different with the PID so perhaps i have made a wrong setting before demuxing i will retry and let you know.

PS : while you are here it seems to me your dvd2avi have crashed when i have press 2 times on the preview button, i will made more tests and let you know if there is some way to reproduce this crash and post a msg if you are interested to solve it and continue to work on you alpha version :)

Have a nice week.
Bye.