diff options
| author | 2023-06-17 11:19:39 +0200 | |
|---|---|---|
| committer | 2023-06-17 11:19:39 +0200 | |
| commit | c309a1c69b933bd196412cae854acb4837243806 (patch) | |
| tree | 4670dc235397f3b7a68f4ed00bb8f4a078146fe1 | |
| parent | video_core: Use sampler IDs instead pointers in the pipeline config (diff) | |
| download | yuzu-c309a1c69b933bd196412cae854acb4837243806.tar.gz yuzu-c309a1c69b933bd196412cae854acb4837243806.tar.xz yuzu-c309a1c69b933bd196412cae854acb4837243806.zip | |
video_core: Removed AF for all mip modes option as it's default now
| -rw-r--r-- | src/common/settings.cpp | 1 | ||||
| -rw-r--r-- | src/common/settings.h | 2 | ||||
| -rw-r--r-- | src/video_core/textures/texture.cpp | 8 | ||||
| -rw-r--r-- | src/yuzu/configuration/config.cpp | 2 | ||||
| -rw-r--r-- | src/yuzu/configuration/configure_graphics_advanced.cpp | 11 | ||||
| -rw-r--r-- | src/yuzu/configuration/configure_graphics_advanced.h | 1 | ||||
| -rw-r--r-- | src/yuzu/configuration/configure_graphics_advanced.ui | 13 | ||||
| -rw-r--r-- | src/yuzu_cmd/config.cpp | 1 | ||||
| -rw-r--r-- | src/yuzu_cmd/default_ini.h | 4 |
9 files changed, 3 insertions, 40 deletions
diff --git a/src/common/settings.cpp b/src/common/settings.cpp index da5cca589..9ff3edabb 100644 --- a/src/common/settings.cpp +++ b/src/common/settings.cpp | |||
| @@ -236,7 +236,6 @@ void RestoreGlobalState(bool is_powered_on) { | |||
| 236 | values.bg_blue.SetGlobal(true); | 236 | values.bg_blue.SetGlobal(true); |
| 237 | values.enable_compute_pipelines.SetGlobal(true); | 237 | values.enable_compute_pipelines.SetGlobal(true); |
| 238 | values.use_video_framerate.SetGlobal(true); | 238 | values.use_video_framerate.SetGlobal(true); |
| 239 | values.use_aggressive_anisotropic_filtering.SetGlobal(true); | ||
| 240 | 239 | ||
| 241 | // System | 240 | // System |
| 242 | values.language_index.SetGlobal(true); | 241 | values.language_index.SetGlobal(true); |
diff --git a/src/common/settings.h b/src/common/settings.h index 78ca73e42..9682281b0 100644 --- a/src/common/settings.h +++ b/src/common/settings.h | |||
| @@ -483,8 +483,6 @@ struct Values { | |||
| 483 | AstcRecompression::Uncompressed, AstcRecompression::Uncompressed, AstcRecompression::Bc3, | 483 | AstcRecompression::Uncompressed, AstcRecompression::Uncompressed, AstcRecompression::Bc3, |
| 484 | "astc_recompression"}; | 484 | "astc_recompression"}; |
| 485 | SwitchableSetting<bool> use_video_framerate{false, "use_video_framerate"}; | 485 | SwitchableSetting<bool> use_video_framerate{false, "use_video_framerate"}; |
| 486 | SwitchableSetting<bool> use_aggressive_anisotropic_filtering{ | ||
| 487 | false, "use_aggressive_anisotropic_filtering"}; | ||
| 488 | 486 | ||
| 489 | SwitchableSetting<u8> bg_red{0, "bg_red"}; | 487 | SwitchableSetting<u8> bg_red{0, "bg_red"}; |
| 490 | SwitchableSetting<u8> bg_green{0, "bg_green"}; | 488 | SwitchableSetting<u8> bg_green{0, "bg_green"}; |
diff --git a/src/video_core/textures/texture.cpp b/src/video_core/textures/texture.cpp index 63ebdfa82..d8b88d9bc 100644 --- a/src/video_core/textures/texture.cpp +++ b/src/video_core/textures/texture.cpp | |||
| @@ -62,14 +62,12 @@ std::array<float, 4> TSCEntry::BorderColor() const noexcept { | |||
| 62 | } | 62 | } |
| 63 | 63 | ||
| 64 | float TSCEntry::MaxAnisotropy() const noexcept { | 64 | float TSCEntry::MaxAnisotropy() const noexcept { |
| 65 | const bool is_suitable_mipmap_filter = Settings::values.use_aggressive_anisotropic_filtering | 65 | const bool is_suitable_mipmap_filter = mipmap_filter != TextureMipmapFilter::None; |
| 66 | ? mipmap_filter != TextureMipmapFilter::None | ||
| 67 | : mipmap_filter == TextureMipmapFilter::Linear; | ||
| 68 | const bool has_regular_lods = min_lod_clamp == 0 && max_lod_clamp >= 256; | 66 | const bool has_regular_lods = min_lod_clamp == 0 && max_lod_clamp >= 256; |
| 69 | const bool is_bilinear_filter = min_filter == TextureFilter::Linear && | 67 | const bool is_bilinear_filter = min_filter == TextureFilter::Linear && |
| 70 | reduction_filter == SamplerReduction::WeightedAverage; | 68 | reduction_filter == SamplerReduction::WeightedAverage; |
| 71 | if (max_anisotropy == 0 && (depth_compare_enabled || !has_regular_lods || !is_bilinear_filter || | 69 | if (max_anisotropy == 0 && (!is_suitable_mipmap_filter || !has_regular_lods || |
| 72 | !is_suitable_mipmap_filter)) { | 70 | !is_bilinear_filter || depth_compare_enabled)) { |
| 73 | return 1.0f; | 71 | return 1.0f; |
| 74 | } | 72 | } |
| 75 | const auto anisotropic_settings = Settings::values.max_anisotropy.GetValue(); | 73 | const auto anisotropic_settings = Settings::values.max_anisotropy.GetValue(); |
diff --git a/src/yuzu/configuration/config.cpp b/src/yuzu/configuration/config.cpp index 27839daaa..bac9dff90 100644 --- a/src/yuzu/configuration/config.cpp +++ b/src/yuzu/configuration/config.cpp | |||
| @@ -761,7 +761,6 @@ void Config::ReadRendererValues() { | |||
| 761 | ReadGlobalSetting(Settings::values.use_vulkan_driver_pipeline_cache); | 761 | ReadGlobalSetting(Settings::values.use_vulkan_driver_pipeline_cache); |
| 762 | ReadGlobalSetting(Settings::values.enable_compute_pipelines); | 762 | ReadGlobalSetting(Settings::values.enable_compute_pipelines); |
| 763 | ReadGlobalSetting(Settings::values.use_video_framerate); | 763 | ReadGlobalSetting(Settings::values.use_video_framerate); |
| 764 | ReadGlobalSetting(Settings::values.use_aggressive_anisotropic_filtering); | ||
| 765 | ReadGlobalSetting(Settings::values.bg_red); | 764 | ReadGlobalSetting(Settings::values.bg_red); |
| 766 | ReadGlobalSetting(Settings::values.bg_green); | 765 | ReadGlobalSetting(Settings::values.bg_green); |
| 767 | ReadGlobalSetting(Settings::values.bg_blue); | 766 | ReadGlobalSetting(Settings::values.bg_blue); |
| @@ -1418,7 +1417,6 @@ void Config::SaveRendererValues() { | |||
| 1418 | WriteGlobalSetting(Settings::values.use_vulkan_driver_pipeline_cache); | 1417 | WriteGlobalSetting(Settings::values.use_vulkan_driver_pipeline_cache); |
| 1419 | WriteGlobalSetting(Settings::values.enable_compute_pipelines); | 1418 | WriteGlobalSetting(Settings::values.enable_compute_pipelines); |
| 1420 | WriteGlobalSetting(Settings::values.use_video_framerate); | 1419 | WriteGlobalSetting(Settings::values.use_video_framerate); |
| 1421 | WriteGlobalSetting(Settings::values.use_aggressive_anisotropic_filtering); | ||
| 1422 | WriteGlobalSetting(Settings::values.bg_red); | 1420 | WriteGlobalSetting(Settings::values.bg_red); |
| 1423 | WriteGlobalSetting(Settings::values.bg_green); | 1421 | WriteGlobalSetting(Settings::values.bg_green); |
| 1424 | WriteGlobalSetting(Settings::values.bg_blue); | 1422 | WriteGlobalSetting(Settings::values.bg_blue); |
diff --git a/src/yuzu/configuration/configure_graphics_advanced.cpp b/src/yuzu/configuration/configure_graphics_advanced.cpp index 3dce73968..0463ac8b9 100644 --- a/src/yuzu/configuration/configure_graphics_advanced.cpp +++ b/src/yuzu/configuration/configure_graphics_advanced.cpp | |||
| @@ -31,7 +31,6 @@ void ConfigureGraphicsAdvanced::SetConfiguration() { | |||
| 31 | ui->use_asynchronous_shaders->setEnabled(runtime_lock); | 31 | ui->use_asynchronous_shaders->setEnabled(runtime_lock); |
| 32 | ui->anisotropic_filtering_combobox->setEnabled(runtime_lock); | 32 | ui->anisotropic_filtering_combobox->setEnabled(runtime_lock); |
| 33 | ui->enable_compute_pipelines_checkbox->setEnabled(runtime_lock); | 33 | ui->enable_compute_pipelines_checkbox->setEnabled(runtime_lock); |
| 34 | ui->use_aggressive_anisotropic_filtering->setEnabled(runtime_lock); | ||
| 35 | 34 | ||
| 36 | ui->async_present->setChecked(Settings::values.async_presentation.GetValue()); | 35 | ui->async_present->setChecked(Settings::values.async_presentation.GetValue()); |
| 37 | ui->renderer_force_max_clock->setChecked(Settings::values.renderer_force_max_clock.GetValue()); | 36 | ui->renderer_force_max_clock->setChecked(Settings::values.renderer_force_max_clock.GetValue()); |
| @@ -44,8 +43,6 @@ void ConfigureGraphicsAdvanced::SetConfiguration() { | |||
| 44 | ui->enable_compute_pipelines_checkbox->setChecked( | 43 | ui->enable_compute_pipelines_checkbox->setChecked( |
| 45 | Settings::values.enable_compute_pipelines.GetValue()); | 44 | Settings::values.enable_compute_pipelines.GetValue()); |
| 46 | ui->use_video_framerate_checkbox->setChecked(Settings::values.use_video_framerate.GetValue()); | 45 | ui->use_video_framerate_checkbox->setChecked(Settings::values.use_video_framerate.GetValue()); |
| 47 | ui->use_aggressive_anisotropic_filtering->setChecked( | ||
| 48 | Settings::values.use_aggressive_anisotropic_filtering.GetValue()); | ||
| 49 | 46 | ||
| 50 | if (Settings::IsConfiguringGlobal()) { | 47 | if (Settings::IsConfiguringGlobal()) { |
| 51 | ui->gpu_accuracy->setCurrentIndex( | 48 | ui->gpu_accuracy->setCurrentIndex( |
| @@ -97,9 +94,6 @@ void ConfigureGraphicsAdvanced::ApplyConfiguration() { | |||
| 97 | enable_compute_pipelines); | 94 | enable_compute_pipelines); |
| 98 | ConfigurationShared::ApplyPerGameSetting(&Settings::values.use_video_framerate, | 95 | ConfigurationShared::ApplyPerGameSetting(&Settings::values.use_video_framerate, |
| 99 | ui->use_video_framerate_checkbox, use_video_framerate); | 96 | ui->use_video_framerate_checkbox, use_video_framerate); |
| 100 | ConfigurationShared::ApplyPerGameSetting(&Settings::values.use_aggressive_anisotropic_filtering, | ||
| 101 | ui->use_aggressive_anisotropic_filtering, | ||
| 102 | use_aggressive_anisotropic_filtering); | ||
| 103 | } | 97 | } |
| 104 | 98 | ||
| 105 | void ConfigureGraphicsAdvanced::changeEvent(QEvent* event) { | 99 | void ConfigureGraphicsAdvanced::changeEvent(QEvent* event) { |
| @@ -136,8 +130,6 @@ void ConfigureGraphicsAdvanced::SetupPerGameUI() { | |||
| 136 | Settings::values.enable_compute_pipelines.UsingGlobal()); | 130 | Settings::values.enable_compute_pipelines.UsingGlobal()); |
| 137 | ui->use_video_framerate_checkbox->setEnabled( | 131 | ui->use_video_framerate_checkbox->setEnabled( |
| 138 | Settings::values.use_video_framerate.UsingGlobal()); | 132 | Settings::values.use_video_framerate.UsingGlobal()); |
| 139 | ui->use_aggressive_anisotropic_filtering->setEnabled( | ||
| 140 | Settings::values.use_aggressive_anisotropic_filtering.UsingGlobal()); | ||
| 141 | 133 | ||
| 142 | return; | 134 | return; |
| 143 | } | 135 | } |
| @@ -165,9 +157,6 @@ void ConfigureGraphicsAdvanced::SetupPerGameUI() { | |||
| 165 | ConfigurationShared::SetColoredTristate(ui->use_video_framerate_checkbox, | 157 | ConfigurationShared::SetColoredTristate(ui->use_video_framerate_checkbox, |
| 166 | Settings::values.use_video_framerate, | 158 | Settings::values.use_video_framerate, |
| 167 | use_video_framerate); | 159 | use_video_framerate); |
| 168 | ConfigurationShared::SetColoredTristate(ui->use_aggressive_anisotropic_filtering, | ||
| 169 | Settings::values.use_aggressive_anisotropic_filtering, | ||
| 170 | use_aggressive_anisotropic_filtering); | ||
| 171 | ConfigurationShared::SetColoredComboBox( | 160 | ConfigurationShared::SetColoredComboBox( |
| 172 | ui->gpu_accuracy, ui->label_gpu_accuracy, | 161 | ui->gpu_accuracy, ui->label_gpu_accuracy, |
| 173 | static_cast<int>(Settings::values.gpu_accuracy.GetValue(true))); | 162 | static_cast<int>(Settings::values.gpu_accuracy.GetValue(true))); |
diff --git a/src/yuzu/configuration/configure_graphics_advanced.h b/src/yuzu/configuration/configure_graphics_advanced.h index 4b52e5d61..a4dc8ceb0 100644 --- a/src/yuzu/configuration/configure_graphics_advanced.h +++ b/src/yuzu/configuration/configure_graphics_advanced.h | |||
| @@ -48,7 +48,6 @@ private: | |||
| 48 | ConfigurationShared::CheckState use_vulkan_driver_pipeline_cache; | 48 | ConfigurationShared::CheckState use_vulkan_driver_pipeline_cache; |
| 49 | ConfigurationShared::CheckState enable_compute_pipelines; | 49 | ConfigurationShared::CheckState enable_compute_pipelines; |
| 50 | ConfigurationShared::CheckState use_video_framerate; | 50 | ConfigurationShared::CheckState use_video_framerate; |
| 51 | ConfigurationShared::CheckState use_aggressive_anisotropic_filtering; | ||
| 52 | 51 | ||
| 53 | const Core::System& system; | 52 | const Core::System& system; |
| 54 | }; | 53 | }; |
diff --git a/src/yuzu/configuration/configure_graphics_advanced.ui b/src/yuzu/configuration/configure_graphics_advanced.ui index 06771c080..e7f0ef6be 100644 --- a/src/yuzu/configuration/configure_graphics_advanced.ui +++ b/src/yuzu/configuration/configure_graphics_advanced.ui | |||
| @@ -260,19 +260,6 @@ Compute pipelines are always enabled on all other drivers.</string> | |||
| 260 | </layout> | 260 | </layout> |
| 261 | </widget> | 261 | </widget> |
| 262 | </item> | 262 | </item> |
| 263 | <item> | ||
| 264 | <widget class="QCheckBox" name="use_aggressive_anisotropic_filtering"> | ||
| 265 | <property name="toolTip"> | ||
| 266 | <string>Enable this option for a more aggressive approach to applying Anisotropic Filtering to textures. | ||
| 267 | By toggling this, Anisotropic Filtering is added to textures with both nearest and linear mipmapping modes. | ||
| 268 | This may result in improved visual quality for a wider range of textures, but can also introduce artifacts in | ||
| 269 | some titles.</string> | ||
| 270 | </property> | ||
| 271 | <property name="text"> | ||
| 272 | <string>Apply Anisotropic Filtering for all mipmap modes</string> | ||
| 273 | </property> | ||
| 274 | </widget> | ||
| 275 | </item> | ||
| 276 | </layout> | 263 | </layout> |
| 277 | </widget> | 264 | </widget> |
| 278 | </item> | 265 | </item> |
diff --git a/src/yuzu_cmd/config.cpp b/src/yuzu_cmd/config.cpp index f5a5364da..c5bc472ca 100644 --- a/src/yuzu_cmd/config.cpp +++ b/src/yuzu_cmd/config.cpp | |||
| @@ -321,7 +321,6 @@ void Config::ReadValues() { | |||
| 321 | ReadSetting("Renderer", Settings::values.astc_recompression); | 321 | ReadSetting("Renderer", Settings::values.astc_recompression); |
| 322 | ReadSetting("Renderer", Settings::values.use_fast_gpu_time); | 322 | ReadSetting("Renderer", Settings::values.use_fast_gpu_time); |
| 323 | ReadSetting("Renderer", Settings::values.use_vulkan_driver_pipeline_cache); | 323 | ReadSetting("Renderer", Settings::values.use_vulkan_driver_pipeline_cache); |
| 324 | ReadSetting("Renderer", Settings::values.use_aggressive_anisotropic_filtering); | ||
| 325 | 324 | ||
| 326 | ReadSetting("Renderer", Settings::values.bg_red); | 325 | ReadSetting("Renderer", Settings::values.bg_red); |
| 327 | ReadSetting("Renderer", Settings::values.bg_green); | 326 | ReadSetting("Renderer", Settings::values.bg_green); |
diff --git a/src/yuzu_cmd/default_ini.h b/src/yuzu_cmd/default_ini.h index f12e2dfc9..911d461e4 100644 --- a/src/yuzu_cmd/default_ini.h +++ b/src/yuzu_cmd/default_ini.h | |||
| @@ -325,10 +325,6 @@ aspect_ratio = | |||
| 325 | # 0: Default, 1: 2x, 2: 4x, 3: 8x, 4: 16x | 325 | # 0: Default, 1: 2x, 2: 4x, 3: 8x, 4: 16x |
| 326 | max_anisotropy = | 326 | max_anisotropy = |
| 327 | 327 | ||
| 328 | # Apply Anisotropic Filtering to all mipmap modes. | ||
| 329 | # 0 (default): Off, 1: On | ||
| 330 | use_aggressive_anisotropic_filtering = | ||
| 331 | |||
| 332 | # Whether to enable VSync or not. | 328 | # Whether to enable VSync or not. |
| 333 | # OpenGL: Values other than 0 enable VSync | 329 | # OpenGL: Values other than 0 enable VSync |
| 334 | # Vulkan: FIFO is selected if the requested mode is not supported by the driver. | 330 | # Vulkan: FIFO is selected if the requested mode is not supported by the driver. |