TDE Settings module shows wrong monitor refresh rates #75

Open
opened 9 months ago by Rademes · 1 comments

Basic information

  • TDE version: R14.0.11
  • Distribution: Debian GNU/Linux 11 (bullseye), Q4OS 4.7.1-n1
  • Hardware: 5.10.0-10-amd64 1 SMP Debian 5.10.84-1 (2021-12-08) x86_64 GNU/Linux

What is severity level?

Description

When I open settings:/Peripherals/Display, I have wrong monitor refresh rate numbers, like 50, 51, 52, 53... but no 60, no 75. As result, I can not set up proper monitor refresh rate. I have ASUS VA24DQ monitor which supports 60 and 75Hz refresh rates and connected using HDMI interface.
This bug was confirmed by Q4OS Team here: https://www.q4os.org/forum/viewtopic.php?id=3939

Steps to reproduce

  1. Open settings:/Peripherals/Display
  2. Try to change minitor refresh rate.

Screenshots

## Basic information - TDE version: R14.0.11 - Distribution: Debian GNU/Linux 11 (bullseye), Q4OS 4.7.1-n1 - Hardware: 5.10.0-10-amd64 1 SMP Debian 5.10.84-1 (2021-12-08) x86_64 GNU/Linux What is severity level? ## Description When I open **settings:/Peripherals/Display**, I have wrong monitor refresh rate numbers, like 50, 51, 52, 53... but no 60, no 75. As result, I can not set up proper monitor refresh rate. I have ASUS VA24DQ monitor which supports 60 and 75Hz refresh rates and connected using HDMI interface. This bug was confirmed by Q4OS Team here: https://www.q4os.org/forum/viewtopic.php?id=3939 ## Steps to reproduce 1. Open **settings:/Peripherals/Display** 2. Try to change minitor refresh rate. ## Screenshots

I can reproduce it.

Looks like XRRRates() over here is lying to us about the refresh rates… Never trust a function with 3 R's in the name… As far as I can see most implementation replace it with manually calculating the refresh rate. It shouldn't be very hard to fix…

I can reproduce it. Looks like `XRRRates()` [over here](https://mirror.git.trinitydesktop.org/gitea/TDE/tdelibs/src/commit/0631f76fb7238b2e0da61beb0b6422ac3d6fa0aa/tderandr/randr.cpp#L496) is lying to us about the refresh rates… Never trust a function with 3 R's in the name… As far as I can see most implementation replace it with manually calculating the refresh rate. It shouldn't be very hard to fix…
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: TDE/tde#75
Loading…
There is no content yet.