From 2d64108af5fa0b64c8d8a7630af5cbdc97ff58fd Mon Sep 17 00:00:00 2001
From: Tyler Wilding <xTVaser@users.noreply.github.com>
Date: Wed, 2 Jul 2025 22:57:57 -0400
Subject: [PATCH] display: be more generous when filtering resolutions by
 refresh rate (#3974)

Some monitors are very specific about their reported refresh rates:
```
[54:37] [debug] [DISPLAY]: Skipping 640x480 as it requires 360.11hz but the monitor is currently set to 360hz
```

The current code truncated the display mode, but would compare against
the reported refresh rate. As per the log, they are obviously not equal.
- Retain the float component and only round it off when sending the
value to GOAL
- Make the comparison more generous, +/- 1.0hz
---
 game/system/hid/display_manager.cpp | 9 ++++++---
 game/system/hid/display_manager.h   | 2 +-
 2 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/game/system/hid/display_manager.cpp b/game/system/hid/display_manager.cpp
index 23d57d9863..c2072dcbe1 100644
--- a/game/system/hid/display_manager.cpp
+++ b/game/system/hid/display_manager.cpp
@@ -1,5 +1,8 @@
 #include "display_manager.h"
 
+#include <cmath>
+#include <cstdlib>
+
 #include "sdl_util.h"
 
 #include "common/global_profiler/GlobalProfiler.h"
@@ -160,7 +163,7 @@ std::string DisplayManager::get_connected_display_name(int id) {
 int DisplayManager::get_active_display_refresh_rate() {
   const auto display_index = get_active_display_index();
   if (m_current_display_modes.size() > display_index) {
-    return m_current_display_modes.at(display_index).refresh_rate;
+    return round(m_current_display_modes.at(display_index).refresh_rate);
   }
   return 0;
 }
@@ -450,7 +453,7 @@ void DisplayManager::update_video_modes() {
     }
 
     DisplayMode new_mode = {display_id,   display_name_str, curr_mode->format,
-                            curr_mode->w, curr_mode->h,     (int)curr_mode->refresh_rate,
+                            curr_mode->w, curr_mode->h,     curr_mode->refresh_rate,
                             orient};
     m_current_display_modes.push_back(new_mode);
     lg::info(
@@ -492,7 +495,7 @@ void DisplayManager::update_resolutions() {
     // Skip resolutions that aren't using the current refresh rate, they won't work.
     // For example if your monitor is currently set to `60hz` and the monitor _could_ support
     // resolution X but only at `30hz`...then there's no reason for us to consider it as an option.
-    if (display_mode->refresh_rate != active_refresh_rate) {
+    if (std::abs(display_mode->refresh_rate - active_refresh_rate) < 1.0) {
       lg::debug(
           "[DISPLAY]: Skipping {}x{} as it requires {}hz but the monitor is currently set to {}hz",
           display_mode->w, display_mode->h, display_mode->refresh_rate, active_refresh_rate);
diff --git a/game/system/hid/display_manager.h b/game/system/hid/display_manager.h
index a7d2803f12..c5077fd62f 100644
--- a/game/system/hid/display_manager.h
+++ b/game/system/hid/display_manager.h
@@ -30,7 +30,7 @@ struct DisplayMode {
   int screen_width;
   int screen_height;
   /// refresh rate (in Hz), or 0 for unspecified
-  int refresh_rate;
+  float refresh_rate;
   Orientation orientation;
 };