search

ValveSoftware / Proton

Compatibility tool for Steam Play based on Wine and additional components
Other
24.35k stars 1.06k forks source link

The "Compiling Shaders" DXVK HUD variable should be enabled by default #3564

Open dribbleondo opened 4 years ago

dribbleondo commented 4 years ago

Feature Request

I confirm:

Description

Enable the compiler variable in DXVK's HUD by default, so players can see when the game is compiling shaders.

Justification [optional]

DXVK HUD normally has some pretty niche variables, not everyone needs to know CPU or GPU usage all the time. However, it would probably divert a load of confusion to the right place if the option; compiler in DXVK HUD was to be on by default. The "compiling shaders" message coming up every time a game stutters/ hangs would hopefully reduce the frequency to say "This game runs poorly", when in actuality, said game is likely shader caching on the first run.

Even if players don't know what "compiling shaders" actually means, the phrase is documented in other areas (such as the DXVK readme, and one would assume appropriate linux gaming subreddits) to get a cursory understanding of what any given game is doing when it stutters/ runs really slowly.

Risks [optional]

Not many come to mind, though perhaps the text might overlay with an ingame UI, which might cause issues.

fsyy commented 4 years ago

would be the 'compiler' option.

dribbleondo commented 4 years ago

would be the 'compiler' option.

Apologies! I've corrected it.

fsyy commented 4 years ago

maybe you could use the user_settings.py which comes with proton.

/home/user/.steam/steam/steamapps/common/Proton 5.0/user_settings.sample.py

edit the line you want to change, in your case: # "DXVK_HUD": "devinfo,fps", and rename the file to user_settings.py try if it's on by default now.

you may have to do this to all proton versions.

dribbleondo commented 4 years ago

I am aware of how to install and use the commands and such, I just thought it'd be nice for the "compiling shaders" text to set to appear by default. Might give a "Oh, that's why it's stuttering!" to newbies, so they don't inherently blame Proton or the OS.

fsyy commented 4 years ago

Hi, i didn't want to offend you. I thought you might use this as a workaround if you want to have it on by default before this maybe is implemented in the client itself. But sure it would be nice for new users to know that it's doing something in the background.

dribbleondo commented 4 years ago

Considering I screwed up the variable in the original description, It's very justified in thinking that. In actuality, I just got the name wrong =P.

Beryesa commented 1 year ago

Never new that, really useful! Please!