brndnmtthws / conky

Light-weight system monitor for X, Wayland (sort of), and other things, too
https://conky.cc
GNU General Public License v3.0
7.27k stars 620 forks source link

Scaling goes strange after commit ≈ 7f84662c and including in today's git version #1097

Closed LinuxOnTheDesktop closed 1 year ago

LinuxOnTheDesktop commented 3 years ago

Issue

I build Conky from git, on a machine with HiDPI and the OS set to scale everything to x2. Everything works until the commit immediately previous to 7f84662c. (I do not know the number of that previous, problematic commit. My git skills are very limited even though I build Conky via it! And the problematic commit might not be exactly the one immediately before 7f84662c.)

After the problematic commit, things look like this:

image

Before that commit, they look, as they should, like this:

image

There was a problem like this a while back and it got fixed; I would not mention the current problem were it not that I see you are heading for a release (and I do want the Intel backlight support that the new release will add - though I am unsure one is to use it).

Information

Linux Mint 20.1 x 64 Cinnamon

I can give my Conky config files but they are large.

genes1122 commented 3 years ago

I have the same issue. I have used the DPI setting in MATE for years to scale the display for high resolution monitors. Conky has ignored this setting until very recently. I'm not sure about the commit number, I'm using conky from the openSUSE Tumbleweed repository. Here are the openSUSE versions bracketing the issue.

Update that broke things. 2021-04-30 09:56:59 | install | conky | 1.12.2-1.1

Revert that fixed things. 2021-04-30 11:38:43 | install | conky | 1.12.1-1.3

Update now works with modified .conkyrc file. 2021-05-02 15:56:09 | install | conky | 1.12.2-1.1

Diff between the .conkyrc files:

diff -s conkyrc.old conkyrc.new

 gap_y = 5,
 maximum_width = 1920,
 minimum_width = 1920,

 gap_y = 3,
 maximum_width = 1536,
 minimum_width = 1536,

${goto 8}${color0}BAT:$color ${battery_short BAT1} ${battery_time BAT1}\ ${goto 445}${color0}CPU:$color $cpu%\ ${goto 555}${color0}CPU:$color ${hwmon k10temp temp 1}°C\ ${goto 675}${color0}Fan:$color ${hwmon dell_smm fan 1}rpm\ ${goto 830}${color0}NVMe:$color ${hwmon nvme temp 1}°C\ ${goto 970}${color0}SSD:$color ${hddtemp /dev/sda}°C\ ${goto 1095}${color0}RAM:$color $memperc%\ ${goto 1205}${color0}Down:$color ${downspeedf wlp3s0}KB/s\

${goto 6}${color0}BAT:$color ${battery_short BAT1} ${battery_time BAT1}\ ${goto 355}${color0}CPU:$color $cpu%\ ${goto 445}${color0}CPU:$color ${hwmon k10temp temp 1}°C\ ${goto 540}${color0}Fan:$color ${hwmon dell_smm fan 1}rpm\ ${goto 665}${color0}NVMe:$color ${hwmon nvme temp 1}°C\ ${goto 775}${color0}SSD:$color ${hddtemp /dev/sda}°C\ ${goto 875}${color0}RAM:$color $memperc%\ ${goto 965}${color0}Down:$color ${downspeedf wlp3s0}KB/s\

Please ignore the system DPI when moving to pixel locations.

Gene

LinuxOnTheDesktop commented 3 years ago

Please ignore the system DPI when moving to pixel locations.

Or, if Conky intends to change its behaviour - thereby breaking backwards compatibility - then at least the change should be documented, surely.

LinuxOnTheDesktop commented 2 years ago

There has no been a release involve the commit at issue - and, still (to my knowledge) no documentation.

LinuxOnTheDesktop commented 2 years ago

Let me myself provide some rough documentation. (In so doing I build upon what @genes1122 wrote above.)

Users need to adjust these values:

Users should not adjust the following:

I use 200% DPI scaling. Thus one might think that halving various values in Conky would do the trick; and, if I remember correctly, I think that so doing did indeed work.

github-actions[bot] commented 1 year ago

This issue is stale because it has been open 365 days with no activity. Remove stale label or comment, or this issue will be closed in 30 days.

github-actions[bot] commented 1 year ago

This issue was closed because it has been stalled for 30 days with no activity.