INTERACT FORUM

Please login or register.

Login with username, password and session length
Advanced search  
Pages: [1]   Go Down

Author Topic: madVR 0.86.2  (Read 3236 times)

6233638

  • Regular Member
  • Citizen of the Universe
  • *****
  • Posts: 5353
madVR 0.86.2
« on: June 04, 2013, 02:16:22 pm »

Just updated to the latest build of madVR, and have noticed an issue that seems to be caused by Media Center:

When Media Center's display mode switcher is disabled, allowing madVR to handle the switching, pressing the stop button inside Media Center does not appear to "release" the video correctly and the original refresh rate is not restored until both Media Center and Media Server are closed.


0.86.2 also implements a fix for madVR's display mode switcher on Windows 8, which allows it to switch to 24/60Hz correctly now.
I'm not sure how Madshi fixed this, but it would be nice if Media Center's display mode switcher could also be updated to correct this. (it switches to 23/59Hz rather than 24/60Hz)
Logged

Matt

  • Administrator
  • Citizen of the Universe
  • *****
  • Posts: 41927
  • Shoes gone again!
Re: madVR 0.86.2
« Reply #1 on: June 04, 2013, 02:33:52 pm »

When Media Center's display mode switcher is disabled, allowing madVR to handle the switching, pressing the stop button inside Media Center does not appear to "release" the video correctly and the original refresh rate is not restored until both Media Center and Media Server are closed.

madVR gets stopped and fully released when playback stops.  So if it makes a change that persists after that, it seems like it might be a madVR issue.
Logged
Matt Ashland, JRiver Media Center

Hendrik

  • Administrator
  • Citizen of the Universe
  • *****
  • Posts: 10710
Re: madVR 0.86.2
« Reply #2 on: June 04, 2013, 02:46:18 pm »

I have some distant memory of that being intentional in madVR..

madVR changelog says this:
* added option "restore original display mode when media player closes"
Logged
~ nevcairiel
~ Author of LAV Filters

Matt

  • Administrator
  • Citizen of the Universe
  • *****
  • Posts: 41927
  • Shoes gone again!
Re: madVR 0.86.2
« Reply #3 on: June 04, 2013, 02:57:12 pm »

madVR is special in that it never really unloads once it's loaded.

We release it, but if you look at a process spy, you'll see several threads, the madVR exe, etc.

I'm not a huge fan of this, since MC should be able to run (basically) forever and acts as a server.  However, I'm sure madshi has some good technical reasons for this.
Logged
Matt Ashland, JRiver Media Center

6233638

  • Regular Member
  • Citizen of the Universe
  • *****
  • Posts: 5353
Re: madVR 0.86.2
« Reply #4 on: June 05, 2013, 01:45:04 pm »

I was wondering: with the introduction of much larger decoder queue sizes, I also ended up increasing the number of backbuffers used in Windowed Mode from the default of 3 to 8.

With 24p content, I had to increase audio latency by ~208ms for sound to be in sync last night. (now 315ms total)

Was it just a coincidence that I had to do this (badly synced source? though it seems unlikely as it was a Blu-ray disc) or should Media Center perhaps be doing automatic correction for this?
Logged

Matt

  • Administrator
  • Citizen of the Universe
  • *****
  • Posts: 41927
  • Shoes gone again!
Re: madVR 0.86.2
« Reply #5 on: June 05, 2013, 01:54:29 pm »

I'm not certain about the latency.

It seems like it would be best if madVR could account for its own latency, but maybe that's not possible for some reason?  I've never noticed a difference between EVR and madVR with regards to timing, so I assumed it already did this.

Our audio renderer could talk to madVR to get its latency and account for it if necessary.  The audio engine already works with madVR to make VideoClock work well.
Logged
Matt Ashland, JRiver Media Center

6233638

  • Regular Member
  • Citizen of the Universe
  • *****
  • Posts: 5353
Re: madVR 0.86.2
« Reply #6 on: June 05, 2013, 01:57:23 pm »

I'm just not sure if the sync error I saw last night was caused by me not having my latency set right for my new DAC yet (though it seemed to be OK before then) if it was the film I was watching, or if something else in my setup had changed which caused this. (though I'm not sure what else that could be)

I understand that it should be a relatively painless fix if that's the cause, but I was wondering if anyone else had noticed something similar?

Note: I seem to recall that years ago when I also ran into lip-sync issues, they were solved by putting madVR into Fullscreen Exclusive Mode, rather than running it in Windowed Mode - but that was on a completely different system, and I don't want to be using FSE with MC18.

I need to do some more testing on this, but I don't know that I will have the opportunity to for a couple of days.


Edit: Oh, and to make it clear, 208ms is approximately 5 frames at 24Hz, which is why I suspect that this was the cause. (1 frame is 41.67ms)
Logged

Hendrik

  • Administrator
  • Citizen of the Universe
  • *****
  • Posts: 10710
Re: madVR 0.86.2
« Reply #7 on: June 05, 2013, 02:17:48 pm »

madVR syncs to the audio reference clock. Longer queues would not cause any desync, unless something can't keep up.
Logged
~ nevcairiel
~ Author of LAV Filters

6233638

  • Regular Member
  • Citizen of the Universe
  • *****
  • Posts: 5353
Re: madVR 0.86.2
« Reply #8 on: June 05, 2013, 02:42:33 pm »

madVR syncs to the audio reference clock. Longer queues would not cause any desync, unless something can't keep up.
Must have been a coincidence then, or something else causing the delay. Thanks.
Logged

mdav

  • Junior Woodchuck
  • **
  • Posts: 59
Re: madVR 0.86.2
« Reply #9 on: June 06, 2013, 02:58:27 am »

Strange, I'm also having A/V sync problems with the update to .195 (running RO HQ, madVR in windowed overlay mode).
Logged

madshi

  • Galactic Citizen
  • ****
  • Posts: 376
Re: madVR 0.86.2
« Reply #10 on: June 06, 2013, 08:06:02 am »

When Media Center's display mode switcher is disabled, allowing madVR to handle the switching, pressing the stop button inside Media Center does not appear to "release" the video correctly and the original refresh rate is not restored until both Media Center and Media Server are closed.

The failure to restore the original refresh rate is not a consequence of MC18 doing something wrong. It was a not-fully-implemented feature in madVR. Basically I restored the refresh rate when switching from fullscreen to windowed mode and when the media player process closes, but I "forgot" to also restore the display mode when the madVR instance is released without leaving fullscreen. This will be fixed in the next build.

madVR is special in that it never really unloads once it's loaded.

We release it, but if you look at a process spy, you'll see several threads, the madVR exe, etc.

I'm not a huge fan of this, since MC should be able to run (basically) forever and acts as a server.  However, I'm sure madshi has some good technical reasons for this.

I might still improve this in a future version. But to my best knowledge the current behaviour doesn't harm in any way, so it's not high in my priority list. FWIW, some of the behaviour has good reasons. E.g. when madVR switches to another display mode, it starts a new madHcCtrl.exe helper process which sits idle and waits for the media player process to end. When the media player process does end, the helper process makes sure that the original display mode gets restored, just in case the madVR instance inside of the media player failed to restore it. This could happen e.g. if there's a bug in madVR (like the one found by 6233638), or if the media player crashes and terminates without giving madVR a chance to restore the original display mode. By using the external madHcCtrl.exe process it is guaranteed that the original display mode is restored at some point.

madVR syncs to the audio reference clock. Longer queues would not cause any desync, unless something can't keep up.

Correct.

Logged

6233638

  • Regular Member
  • Citizen of the Universe
  • *****
  • Posts: 5353
Re: madVR 0.86.2
« Reply #11 on: June 06, 2013, 09:18:01 am »

This will be fixed in the next build.
Excellent, thank you. I wasn't meaning to point "blame" at anyone.
Logged
Pages: [1]   Go Up