Welcome to Doom9's Forum, THE in-place to be for everyone interested in DVD conversion. Before you start posting please read the forum rules. By posting to this forum you agree to abide by the rules. |
24th June 2011, 23:26 | #17581 | Link | |
Broadband Junkie
Join Date: Oct 2005
Posts: 1,859
|
Quote:
I did some testing, and in most cases gommorah's fixed string manipulation patch results in a 0-2% increase in avg framerate on difficult samples when VSFilter is CPU limited and prebuffering is disabled (r3255 vs r3262 from xvidvideo.ru). Some subtitle samples and CPUs likely benefit from the patch more than others, but at least this resolved one of many causes of VSFilter slowdowns. Since the code hasn't really changed much, it's likely the optimizations from VS2010 vs VS2005 results in most of the gain over Guliverkli2. AMD X2 4800+ (939) @2.64Ghz on WinXP SP3 VSFilter r3262 (xvidvideo.ru VS2010) KaraokeSample1: ~22 fps KaraokeSample2: ~20.5 fps TypesettingSample1: ~20.5 fps VSFilter r3255 (xvidvideo.ru VS2010) KaraokeSample1: ~21.75 fps KaraokeSample2: ~20 fps TypesettingSample1: ~20 fps VSFilter r3242 (VS2010 WPO+PGO) KaraokeSample1: ~22.5 fps KaraokeSample2: ~22.75 fps TypesettingSample1: ~21.25 fps VSFilter 2.39.5.2 (Guliverkli2 VS2005) KaraokeSample1: ~20.75 fps KaraokeSample2: ~16.75 fps TypesettingSample1: ~19.25 fps Note: The above are rough measurements, averaged over a few runs on 1000-2000 frame clips. They are accurate enough to see which builds are faster/slower with the samples tested, but not accurate enough to determine exact performance gains/losses in the way I tested. The r3242 WPO+PGO build I just added for comparisons sake. I haven't taken the time to make a new build, but I suspect a r3262 WPO+PGO build would have a similar 0-2% gain with the patch like the xvidvideo.ru builds do. |
|
25th June 2011, 08:17 | #17582 | Link | |
Registered User
Join Date: Oct 2004
Location: France
Posts: 567
|
Quote:
I never use animation in subtitles so I don't really know what to do ... |
|
25th June 2011, 09:00 | #17583 | Link | |
Registered User
Join Date: Jun 2009
Location: London
Posts: 378
|
Quote:
Maybe if a check is desired something like this, WinXP = 3 cores (Mostly to keep it away from older P4's with HT) Vista+Windows7 = 2 cores
__________________
Windows 11 Pro, CPU: Ryzen 9 5900X, GPU: GeForce RTX 3090 /w 512.95, Audio: HDMI to Yamaha RX-A3060 (5.1) Last edited by VipZ; 25th June 2011 at 09:11. |
|
25th June 2011, 14:58 | #17584 | Link |
*****
Join Date: Feb 2005
Posts: 5,643
|
I say commit with 4 as default in both VSFilter and MPC. Use true for the anim option. It can always be changed later if needed.
__________________
MPC-HC 2.1.7.2 |
25th June 2011, 15:09 | #17585 | Link |
Registered User
Join Date: Sep 2009
Posts: 16
|
Hi, since revision 3262 (fixed vertical padding) placement of Drawing is wrong (I don't know why and how).
Here is a self explanatory image, I used Aegisub to render it so we can see the anchor point, but I have tested on a video, and the result is the same: http://img808.imageshack.us/img808/8107/examplerb.png Thanks for reading. |
25th June 2011, 15:11 | #17586 | Link | |
Registered User
Join Date: Sep 2009
Location: Sydney, Australia
Posts: 1,073
|
Quote:
Last edited by namaiki; 25th June 2011 at 15:25. |
|
25th June 2011, 15:32 | #17587 | Link | |
Registered User
Join Date: Sep 2009
Posts: 16
|
Quote:
|
|
25th June 2011, 15:46 | #17588 | Link |
*****
Join Date: Feb 2005
Posts: 5,643
|
With the old VSFilter 2.39 it also isn't perfect. Without padding the blue boxes are slightly larger, but the black boxes are positioned correctly. With padding enabled, the black boxes are stretched vertically. But padding is something not really suitable for subs that need specific positioning above the video.
With new VSFilter, the black boxes are positioned wrong even with padding disabled.
__________________
MPC-HC 2.1.7.2 Last edited by clsid; 25th June 2011 at 15:48. |
25th June 2011, 15:54 | #17589 | Link |
Registered User
Join Date: Sep 2009
Posts: 16
|
Oh, sorry, forgot to mention the reference image, (I didn't made the square the same size), here is reference image, and the result with r3262, it's not a very good testcase:
http://img717.imageshack.us/img717/2954/resultu.png |
25th June 2011, 19:02 | #17590 | Link |
Broadband Junkie
Join Date: Oct 2005
Posts: 1,859
|
I can confirm what neb1236 is seeing when Vertical Padding is set to Original Height in r3262.
CCCP's r339/r1096 basevideofilter revert patches doesn't have the positioning bug seen in neb1236's r3262 image and fixes Vertical Padding in VSFilter. Identical results to Guliverkli2. Edit: My mistake, it turns out I tested the wrong build which only had the r339/r1096 basevideofilter revert patches. The build with r339/r1096 basevideofilter revert patches and gommorah's patch did have the bug. It looks like CLSID fixed this in r3270. CLSID's custom Vertical Padding Fix patch in r3261 does have the positioning bug seen in neb1236's r3262 image. Last edited by cyberbeing; 27th June 2011 at 22:17. |
26th June 2011, 00:28 | #17591 | Link |
*****
Join Date: Feb 2005
Posts: 5,643
|
r3261 should have the same effect as reverting 339/1096. Some fresh eyes looking at that code would be helpful. I might have overlooked something.
__________________
MPC-HC 2.1.7.2 |
26th June 2011, 10:34 | #17593 | Link |
Registered User
Join Date: Dec 2008
Posts: 1,959
|
mpc-hc_3268_dts_ac3_test1.7z
DTSAC3Source: + Added support DTSWAV (DTS-CD) + Added search DTS and AC3 headers (skip garbage in the beginning) + Improved support for AC3 and E-AC3 |
26th June 2011, 22:36 | #17595 | Link | |
Registered User
Join Date: May 2010
Posts: 236
|
Quote:
I have a different problem which is quite terrible. I can't change files with PgUp/PgDn when I'm in D3DFS. When I try to do so everything's normal except one thing: video is frozen on the last frame of the previous video. It's as if the video card fails to see the need to show any changes! Sound is running normal but seekbar as well as mouse cursor aren't showing up. Catalyst 11.6, everything is set to use application settings. All video enhancers are off. I use EVR CP and tried every option in renederer settings. Tried different splitters as well. Tried uninstalling the player with its settings and with a fresh install it's the same thing. |
|
26th June 2011, 22:55 | #17596 | Link | |
Registered User
Join Date: Oct 2010
Location: The Netherlands
Posts: 1,083
|
Quote:
__________________
development folder, containing MPC-HC experimental tester builds, pixel shaders and more: http://www.mediafire.com/?xwsoo403c53hv |
|
26th June 2011, 23:12 | #17597 | Link |
Registered User
Join Date: May 2010
Posts: 236
|
JanWillem32, are you a wizard? The problem is gone with your build! Why don't other development folks use your method to prevent this problem?
P.S. And I've almost made up my mind to delete the system. Last edited by TheElix; 26th June 2011 at 23:15. |
26th June 2011, 23:42 | #17598 | Link |
Registered User
Join Date: Oct 2010
Location: The Netherlands
Posts: 1,083
|
I'm not going to try to break up my re-written parts of the renderer for partial integration. I've re-written almost the complete rendering engine, and the allocator-presenter is re-written for more than a quarter of the total code. There's little resemblance in some parts of the rendering method. That causes incompatibility with a lot of the older code. The best I can do at the moment is to provide users with tester builds and interact in the thread I made. I use this method as a way to work towards offering the renewed version of the internal renderer sections as a complete replacement, rather than a patch of the original code. I most certainly know I have bend over backwards to ever get the three VSync items to work again as they used to. I'll just have to make it work, but it's very hard for me to understand the cohesion in the many 64-bit items it uses, with timescales in millisecond, 100 nanosecond, nanosecond, second and per second timing units. I would never have written it in this way. (I'm usually lazy in this regard and just activate the video card's own internal VSync, or let desktop composition handle the VSync in windowed mode, but I'm a lot more familiar with DirectX 10 and 11 than 9, anyway.)
__________________
development folder, containing MPC-HC experimental tester builds, pixel shaders and more: http://www.mediafire.com/?xwsoo403c53hv |
27th June 2011, 00:13 | #17599 | Link |
Registered User
Join Date: May 2010
Posts: 236
|
I see. Well, your method is the only one that makes internal autochange frequency work for me. That just makes one more reason for me to stick to your versions for now. By the way, does autofrequency in your build work for you with D3DFS?
Last edited by TheElix; 27th June 2011 at 00:27. |
27th June 2011, 00:54 | #17600 | Link |
Registered User
Join Date: Oct 2010
Location: The Netherlands
Posts: 1,083
|
The exclusive mode frequency adjustment part is officially on the "to do" list, but if the display frequency is already set while the program is in windowed mode, the code I wrote will happily accept normal (whole number) and NTSC refresh rates (those are 1/1.001 adjusted) from the currently active state in windowed mode. If the exclusive mode is activated before this kind of frequency change command is used, that command is simply negated.
I didn't know this function worked at all in my builds. I haven't paid much attention to this part, I just tried to fix the critical things first. I'll have to add this function to the "to test if it works in all cases" list then... If it doesn't really work in exclusive mode I can try to make a link to the allocator-presenter. It could then inherit the custom set frequency number directly from the user-set options and enable it on an output by itself, instead of inheriting the frequency from the windowed mode before it.
__________________
development folder, containing MPC-HC experimental tester builds, pixel shaders and more: http://www.mediafire.com/?xwsoo403c53hv |
Tags |
dxva, h264, home cinema, media player classic, mpc-hc |
Thread Tools | Search this Thread |
Display Modes | |
|
|