FSR: only set in Gamescope (remove Wine FSR)

This moves the FSR settings in Gamescope, and removes support for
setting FSR in Wine.

wine-ge-custom already enables FSR by default, and discourages
the user from disabling it (#2636).

Since Gamescope is generally more independent and is a micro-compositor
with first-class support with FSR, it's safer to only have an option there,
and have the runner decide for the user if Gamescope isn't being used.

Having two separate FSR enablers is messy, as we need logic to avoid
enabling both of them at the same time, which Bottles isn't even doing.

The conditions mentioned above aren't communicated at all by having FSR
as a standalone setting.

Fixes #2755, #2636

Author: Martino Fontana

bottles/backend/managers/manager.py

@@ -1096,7 +1096,6 @@ def get_programs(self, config: BottleConfig) -> List[dict]:
                     "dxvk": _program.get("dxvk"),
                     "vkd3d": _program.get("vkd3d"),
                     "dxvk_nvapi": _program.get("dxvk_nvapi"),
-                    "fsr": _program.get("fsr"),
                     "gamescope": _program.get("gamescope"),
                     "pulseaudio_latency": _program.get("pulseaudio_latency"),
                     "virtual_desktop": _program.get("virtual_desktop"),

bottles/backend/models/config.py

@@ -89,8 +89,7 @@ class BottleParams(DictCompatMixIn):
     gamescope_custom_options: str = ""
     sync: str = "wine"
     fsr: bool = False
-    fsr_sharpening_strength: int = 2
-    fsr_quality_mode: str = "none"
+    fsr_sharpening_strength: int = 8
     custom_dpi: int = 96
     renderer: str = "gl"
     discrete_gpu: bool = False

bottles/backend/models/samples.py

@@ -64,9 +64,6 @@ class Samples:
         "ENABLE_VKBASALT": ("vkbasalt", True),
         "WINEESYNC": ("sync", "esync"),
         "WINEFSYNC": ("sync", "fsync"),
-        "WINE_FULLSCREEN_FSR": ("fsr", True),
-        "WINE_FULLSCREEN_FSR_STRENGTH": ("fsr_sharpening_strength", 2),
-        "WINE_FULLSCREEN_FSR_MODE": ("fsr_quality_mode", "none"),
         "GAMESCOPE": ("gamescope", False),
         "DRI_PRIME": ("discrete_gpu", True),
         "__NV_PRIME_RENDER_OFFLOAD": ("discrete_gpu", True),

bottles/backend/wine/executor.py

@@ -57,7 +57,6 @@ def __init__(
         program_dxvk: Optional[bool] = None,
         program_vkd3d: Optional[bool] = None,
         program_nvapi: Optional[bool] = None,
-        program_fsr: Optional[bool] = None,
         program_gamescope: Optional[bool] = None,
         program_virt_desktop: Optional[bool] = None,
         program_winebridge: Optional[bool] = None,
@@ -116,16 +115,6 @@ def __init__(
                 override_nvapi = NVAPIComponent.get_override_keys() + "=b"
                 env_dll_overrides.append(override_nvapi)
 
-        if program_fsr is not None and program_fsr != self.config.Parameters.fsr:
-            self.environment["WINE_FULLSCREEN_FSR"] = "1" if program_fsr else "0"
-            self.environment["WINE_FULLSCREEN_FSR_STRENGTH"] = str(
-                self.config.Parameters.fsr_sharpening_strength
-            )
-            if self.config.Parameters.fsr_quality_mode:
-                self.environment["WINE_FULLSCREEN_FSR_MODE"] = str(
-                    self.config.Parameters.fsr_quality_mode
-                )
-
         if (
             program_gamescope is not None
             and program_gamescope != self.config.Parameters.gamescope
@@ -167,7 +156,6 @@ def _resolve(field: str):
             program_dxvk=program.get("dxvk"),
             program_vkd3d=program.get("vkd3d"),
             program_nvapi=program.get("dxvk_nvapi"),
-            program_fsr=program.get("fsr"),
             program_gamescope=program.get("gamescope"),
             program_virt_desktop=program.get("virtual_desktop"),
             program_winebridge=program.get("winebridge"),

bottles/backend/wine/winecommand.py

@@ -446,13 +446,6 @@ def get_env(
         # if params["aco_compiler"]:
         #     env.add("ACO_COMPILER", "aco")
 
-        # FSR
-        if params.fsr:
-            env.add("WINE_FULLSCREEN_FSR", "1")
-            env.add("WINE_FULLSCREEN_FSR_STRENGTH", str(params.fsr_sharpening_strength))
-            if params.fsr_quality_mode:
-                env.add("WINE_FULLSCREEN_FSR_MODE", str(params.fsr_quality_mode))
-
         # PulseAudio latency
         if params.pulseaudio_latency:
             env.add("PULSE_LATENCY_MSEC", "60")
@@ -699,10 +692,8 @@ def _get_gamescope_cmd(self, return_steam_cmd: bool = False) -> str:
                 gamescope_cmd.append("-S integer")
             if params.fsr:
                 gamescope_cmd.append("-F fsr")
-                # Upscaling sharpness is from 0 to 20. There are 5 FSR upscaling levels,
-                # so multiply by 4 to reach 20
                 gamescope_cmd.append(
-                    f"--fsr-sharpness {params.fsr_sharpening_strength * 4}"
+                    f"--fsr-sharpness {params.fsr_sharpening_strength}"
                 )
             if params.gamescope_fps > 0:
                 gamescope_cmd.append(f"-r {params.gamescope_fps}")

bottles/frontend/cli/cli.py

@@ -793,7 +793,6 @@ def run_program(self):
             program.get("dxvk")
             program.get("vkd3d")
             program.get("dxvk_nvapi")
-            program.get("fsr")
             program.get("gamescope")
             program.get("virtual_desktop")
 

bottles/frontend/ui/bottles.gresource.xml

@@ -47,7 +47,6 @@
     <file preprocess="xml-stripblanks">dialog-rename.ui</file>
     <file preprocess="xml-stripblanks">dialog-gamescope.ui</file>
     <file preprocess="xml-stripblanks">dialog-vkbasalt.ui</file>
-    <file preprocess="xml-stripblanks">dialog-fsr.ui</file>
     <file preprocess="xml-stripblanks">dialog-mangohud.ui</file>
     <file preprocess="xml-stripblanks">dialog-display.ui</file>
     <file preprocess="xml-stripblanks">dialog-drives.ui</file>

bottles/frontend/ui/details-preferences.blp

@@ -98,26 +98,6 @@ template $DetailsPreferences: Adw.PreferencesPage {
       }
     }
 
-    Adw.ActionRow {
-      activatable-widget: switch_fsr;
-      title: _("FidelityFX Super Resolution");
-      subtitle: _("Increase performance at the expense of visuals. Only works on Vulkan.");
-
-      Button btn_manage_fsr {
-        tooltip-text: _("Manage FidelityFX Super Resolution settings");
-        valign: center;
-        icon-name: "applications-system-symbolic";
-
-        styles [
-          "flat",
-        ]
-      }
-
-      Switch switch_fsr {
-        valign: center;
-      }
-    }
-
     Adw.ActionRow row_discrete {
       visible: false;
       activatable-widget: switch_discrete;

bottles/frontend/ui/dialog-fsr.blp

@@ -107,6 +107,33 @@ template $GamescopeDialog: Adw.Window {
         }
       }
 
+      Adw.PreferencesGroup {
+        title: _("FSR");
+
+        Adw.ActionRow {
+          title: _("Enable");
+          activatable-widget: switch_fsr;
+
+          Switch switch_fsr {
+            valign: center;
+          }
+        }
+
+        Adw.ActionRow {
+          title: _("Sharpening Strength");
+
+          SpinButton {
+            numeric: true;
+            valign: center;
+
+            adjustment: Adjustment spin_sharpening_strength {
+              step-increment: 4;
+              upper: 20;
+            };
+          }
+        }
+      }
+
       Adw.PreferencesGroup {
         title: _("Miscellaneous");