Importing Recordings (4 Viewers)

[snakeman65]

Where I am so far with this: I have found that if I set MP to use the VideoPlayerVMR9 to play dvr-ms instead of the StreamBufferPlayer9 then it happily plays! Now, unfortunately, I don't understand video stuff enough to know what the differences between videoplayervmr9 and streambufferplayer9 are and what the reason for using streambufferplayer9 is so if anyone could fill me in it'd be a great help!

I'm getting there though... slowly!

Chris

I've now confirmed that it is DEFINATELY the DirectShowUtil.cs that has the bug. Restoring an old build of this file allows it to work so now its just spot the bug time between revision 16124 and 13892!

Ok, so now I know where the problem is, don't understand the code though

The code working:
hr = graphBuilder.Render(pins[0]);
if (hr == 0)

and broken:

hr = 0;
if (TryConnect(graphBuilder, info.achName, pins[0]))

Ho hum

 

[snakeman65]

Ok, from what I understand so far, the difference that is causing the problem is that in the earlier revision IGraphBuilder.Render was called, which allows IGraphBuilder to build its own Graph, in the later code, the graph is built manually, since there are no comments, I can't say why this was done, however, I at least have my own personal build that works!!

Anyone able to add any of this to the bug report?

 

[iosub]

Ok, from what I understand so far, the difference that is causing the problem is that in the earlier revision IGraphBuilder.Render was called, which allows IGraphBuilder to build its own Graph, in the later code, the graph is built manually, since there are no comments, I can't say why this was done, however, I at least have my own personal build that works!!

Anyone able to add any of this to the bug report?

can you tell me what file did you modify and what line of code? I want to test also.

Thanks!!

 

[snakeman65]

Please bear in mind that this changes anything rendered by this file to use the default graph rather than the mp created one. I don't know what affect this will have (other than fixing the bug!) but as the svn change was listed as dvd navigation not working I suspect this will break dvd navigation!

In the latest svn, take a look at core/dshownew/helper/directshowutil.cs around line 599.

change:

hr = 0;
if (TryConnect(graphBuilder, info.achName, pins[0]))
//if ((hr=graphBuilder.Render(pins[0])) == 0)
{
Log.Info(" render ok");
}
else
{
Log.Error(" render {0} failed:{1:x}", pinInfo.name, hr);

to:

if ((hr=graphBuilder.Render(pins[0])) == 0)
{
Log.Info(" render ok");
}
else
{
Log.Error(" render failed:{0:x}", hr);

 

[Paranoid Delusion]

Hi snakeman65

Any chance of putting your point of view regarding this on IRC, link in MP config, really may prove beneficial to all, as a lot of Dev's do not read all posts.

Hope everything works out with the Mrs as the WAF has to be high.

 

[iosub]

Ok, from what I understand so far, the difference that is causing the problem is that in the earlier revision IGraphBuilder.Render was called, which allows IGraphBuilder to build its own Graph, in the later code, the graph is built manually, since there are no comments, I can't say why this was done, however, I at least have my own personal build that works!!

Anyone able to add any of this to the bug report?

Please bear in mind that this changes anything rendered by this file to use the default graph rather than the mp created one. I don't know what affect this will have (other than fixing the bug!) but as the svn change was listed as dvd navigation not working I suspect this will break dvd navigation!

In the latest svn, take a look at core/dshownew/helper/directshowutil.cs around line 599.

change:

hr = 0;
if (TryConnect(graphBuilder, info.achName, pins[0]))
//if ((hr=graphBuilder.Render(pins[0])) == 0)
{
Log.Info(" render ok");
}
else
{
Log.Error(" render {0} failed:{1:x}", pinInfo.name, hr);

to:

if ((hr=graphBuilder.Render(pins[0])) == 0)
{
Log.Info(" render ok");
}
else
{
Log.Error(" render failed:{0:x}", hr);

THANKS! THANKS! MILION OF THANKS!!!!
WORKS!
Now some developer should check this...

 

[snakeman65]

Hi snakeman65

Any chance of putting your point of view regarding this on IRC, link in MP config, really may prove beneficial to all, as a lot of Dev's do not read all posts.

Hope everything works out with the Mrs as the WAF has to be high.

I'll have a go! It seems though that because i'm in the UK the devs don't seem to be online at the times that I am

If I could get any of this on the bug report then i'm sure it would get looked at quickly, how do I get a mantis acount?

 

[joystick]

Hi,
I just tried your Importer with MySQL:

It does actually nothing...
thats what I did:
Unpacked the file to a Folder
started the Recordings Importer exe
set the Directory where my "new" recordings are located
set MySql hock
set the Mysql Server and password
tested connection -> succeed
click on ok

nothing really happened
No recordings imported....

Hints?


cheers

joystick

 

[snakeman65]

I'll look into it! I haven't done a huge amount of testing since I realised that I couldn't actually use mp but now i've got a bugfix i'll get back into it.

Chris

 

[Marcusb]

Glad to hear you got it working
As was said earlier, Eabin was the one who made the changes, so it may be worthwhile to PM him, since he will have more information and it may draw his attention to this thread.

Also, strange about the devs on IRC. They are mostly in Germany so shouldn't be too far out of your time zone.
You should try living in Australia