Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Whole screen dims out when targeting capital ships #19

Closed
PatTheMav opened this issue Aug 18, 2019 · 13 comments
Closed

Whole screen dims out when targeting capital ships #19

PatTheMav opened this issue Aug 18, 2019 · 13 comments

Comments

@PatTheMav
Copy link

Using v1.5.10 rc1 on stock XWA (GOG install) leads to the whole screen being dimmed whenever a capital ship (e.g. cruiser/star destroyer) is targeted. Once a fighter-class ship is targeted, the dimming is gone.

No additional patches were activated, but I've tested with the 60fps patch and still had the same result.

The issue does not appear when @JeremyAnsel's latest version is used.

@rdoeffinger
Copy link
Owner

Can you confirm that it does not happen with 1.5.9?
There are relatively few changes since 1.5.9 actually, and none look like a really obvious candidate (though there's one change I did that is only in my branch I need to look at more closely).

@PatTheMav
Copy link
Author

PatTheMav commented Aug 18, 2019

Will do that in a bit - I'm currently trying to find out why XWAUCP makes cutscenes run at half their fps and music is gone from gameplay.. 🙈

Edit: So the video playback issue might be a side-effect of the timer hook that fixes the constant 100% CPU usage. Worth the tradeoff I guess.

As for my issue, it sounds like exactly what you talked about in this post: http://www.xwaupgrade.com/phpBB3/viewtopic.php?p=156377#p156377

@bguthrie1
Copy link

I don't have this issue with the latest ddraw. You can try using xwahacker to disable 32-bit rendering and see if that fixes it. But the ddraw was supposed to fix it regardless if that was checked or not.

@bguthrie1
Copy link

bguthrie1 commented Aug 18, 2019

Scratch that, I had 16-bit rendering turned on. I turned on 32-bit rendering through xwahacker and I get the dimming issue posted above. I've tried various DDraw versions and it seems every version is affected except 1.5.7. Which is the version that was intended to fix this issue. Something changed between 1.5.7 and 1.5.8 that allowed this issue to occur.

@PatTheMav
Copy link
Author

@bguthrie1 So possibly a regression starting with 1.5.8 then. It seems to work like a charm with XWAUCP, but I do wonder if it's possible to remove the timer hook to let the ddraw handle the frame rate limit and thus get fluid video playback. 🤔

@bguthrie1
Copy link

What timer hook are you referring to? You can try removing the refresh limit in the ddraw config to see if that helps.

@PatTheMav
Copy link
Author

The one installed by XWAUCP - it essentially tries to achieve the same thing (reduce 100% cpu usage by inserting sleep calls in an otherwise endless loop), but a (known) side effect is that video frame rates are more or less halved.

@bguthrie1
Copy link

bguthrie1 commented Aug 18, 2019

Ah thats the hook_time.dll which removes the frame limit in flight. You can try replacing hook_time.dll with my 60FPS hook and see if that has an effect.

EDIT: Hook_time.dll shouldn't be affecting concourse frame rates. The refresh limit option I stated above should fix the issue but if that fails try using my 60FPS hook.

@PatTheMav
Copy link
Author

@bguthrie1 - thanks hadn't thought of just renaming the 60fps-hook to get it to load instead of the timer hook. It fixed the video playback rate without impacting gameplay frame rates. Now I just need to check CPU usage.

(BTW - I take it that even with all changed DLLs the game is still forward rendering? Because just enabling something like MXAO in reshade leads to massive frame rate drops when e.g. a space station or star destroyer is in view).

@rdoeffinger
Copy link
Owner

Sorry about that, there is no automated testing an I regularly mess up manual testing.
It seems one of the patches in 1.5.8 actually broke the previous fixes for 1.5.8 again and I tested it wrong and did not notice.
Could try out rc2...
xwing_ddraw_d3d11_1.5.10_rc2.zip

@PatTheMav
Copy link
Author

@rdoeffinger Thanks! If @bguthrie1 doesn't beat me to the punch, I'll try and test it in a 2nd copy of XWA (thanks to his tip I have a well-working XWA installation with UCP working and don't want to revert everything on that copy).

@bguthrie1
Copy link

Works fine now. Thanks Reimar.

@rdoeffinger
Copy link
Owner

In the hopes that it is indeed fixed for real this time, closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants