-
-
Notifications
You must be signed in to change notification settings - Fork 978
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bug] Pointless power plan change #2319
Comments
The bug must me somewhere here: https://github.com/dotnet/BenchmarkDotNet/blob/master/src/BenchmarkDotNet/Running/PowerManagementApplier.cs Please send a PR with a fix |
From the brief look at it I am not seeing the problem, must be in the calling code. Another thing, what happens when user presses Ctrl+C to interrupt test execution? I don't see the code hooking console event handler to restore power plan when process is interrupted? Absolutely the whole idea of messing with user settings is IMO bad if you don't guarantee they will be restored. Another scenario -- Windows Update decides to restart the PC while the benchmark is running. Are you intercepting |
Just had to come here and say as someone that just tried this framework the changing of power plans feels very weird. On my laptop I use balanced, yet when running these tests it changes the profile to performance, which increases my screen brightness (I have custom settings I'd like to retain between the different performance profiles). Is there a reason this is done, that performance profiling cannot be done on the user's current power plan? |
You should not change power plant to High Performance if it is already Ultimate Performance.
The text was updated successfully, but these errors were encountered: