FanControl.Releases icon indicating copy to clipboard operation
FanControl.Releases copied to clipboard

FR: Ability to prevent FanControl window from opening window after waking PC

Open hapklaar opened this issue 1 year ago • 15 comments

Every time I wake my PC from sleep, the FanControl window opens and doesn't minimize by itself. Start minimized is enabled.

Can this be prevented somehow, so the window remains minimized to tray?

hapklaar avatar Jan 23 '24 18:01 hapklaar

Do you get any popup error when the window is opened up automatically?

Rem0o avatar Jan 24 '24 01:01 Rem0o

Currently there are no errors when resuming from sleep.

Sent from Nine


Van: "Rem0o @.)" @.> verzonden: woensdag 24 januari 2024 02:49 Aan: Rem0o/FanControl.Releases Cc: hapklaar; Author Onderwerp: Re: [Rem0o/FanControl.Releases] FR: Ability to prevent FanControl window from opening window after waking PC (Issue #2265)

Do you get any popup error when the window is opened up automatically? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>

hapklaar avatar Jan 24 '24 18:01 hapklaar

Strangely enough when I just woke the PC, the window did not pop up... Should it only happen when there are errors?

hapklaar avatar Jan 24 '24 20:01 hapklaar

Should it only happen when there are errors?

Yes, well, if some part of the program asked to show a dialog, which upon wake from sleep to happen automatically, is an error in the refresh process .

Rem0o avatar Jan 24 '24 20:01 Rem0o

Today it opened again after resuming from sleep. Nothing in the error log. Window appears like this: image

The bottom notification Configuration loaded: userConfig.json kept dropping down and reappearing. That eventually stopped. Maybe that is relevant?

hapklaar avatar Jan 26 '24 13:01 hapklaar

@hapklaar that means the app refreshed for every time that snackbar popped up, which means upon waking from sleep the app can't get hold of all the sensors it had before going to sleep.

Rem0o avatar Jan 26 '24 13:01 Rem0o

Ah I see. The level of sleep might play a part here.

Would it be possible to do this in the background and not show the screen?

hapklaar avatar Jan 26 '24 14:01 hapklaar

The intention is to only show the window if it couldn't restore properly, hence functionality was not restored, needing attention.

In your case there seem to be a bug. I will take a look.

Rem0o avatar Jan 26 '24 14:01 Rem0o

Any log.txt?

I made a debug version of the exe with logs of the whole wake from sleep sequence, if you could run that, reproduce the wake from sleep problem, then send back the logs.

Wake from sleep logs.zip

Rem0o avatar Jan 26 '24 15:01 Rem0o

sure, will do. Will it write to log.txt in the exe's directory?

hapklaar avatar Jan 26 '24 16:01 hapklaar

yes

Rem0o avatar Jan 26 '24 17:01 Rem0o

Today when I woke the pc from sleep (16:41), no FanControl window was opened, but I noticed after a while my radiator fans were spinning louder than they should. I opened FanControl from systray (16:51) and was greeted with a message that not all sensors were detected. After clicking refresh once, this was fixed.

So not the original issue I was facing, but maybe it is still useful.

I enclosed the log log.txt

hapklaar avatar Jan 27 '24 15:01 hapklaar

This tells me it takes more than 30 seconds for your system to be able to get back the sensors after sleep. When waking from sleep, fancontrol will refresh every 5 seconds for 30 seconds until all sensors that were previously there before sleep are restored. Maybe in your case that's just not enough, which is surprising.

Rem0o avatar Jan 27 '24 18:01 Rem0o

I made a debug version of the exe with logs of the whole wake from sleep sequence, if you could run that, reproduce the wake from sleep problem, then send back the logs.

Just to let you know. I haven't had the issue since I installed your debug version. Could the debugging itself have influence?

hapklaar avatar Feb 25 '24 09:02 hapklaar

I have the same problem and occasionally need to manually refresh the sensor, is there a solution now?

AzuMiWanHua avatar Apr 13 '24 06:04 AzuMiWanHua