Skip to content
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

arrow keys to modify fan curve points #2312

Closed
2 tasks done
moms-spaghetti opened this issue Mar 27, 2024 · 8 comments
Closed
2 tasks done

arrow keys to modify fan curve points #2312

moms-spaghetti opened this issue Mar 27, 2024 · 8 comments

Comments

@moms-spaghetti
Copy link

Rules

  • I made myself familiar with the Readme, FAQ and Troubleshooting.
  • I understand that, if insufficient information will be provided, my issue will be closed without an answer.

Is your feature request related to a problem? Please describe

granular adjustment of fan curves value can be difficult when using a pointing device as small movements when releasing mouse buttons / trackpad are still recorded as inputs

Describe the solution you'd like

clicking on a fan curve point focuses it and allows arrow keys to be used for granular adjustment based on lowest unit graph increments

Describe alternatives you've considered

Double click a fan curve box to increase it's size so finer adjustments with a pointing device are easier

Device and Model

Asus Strix G16 G614JZ

Additional information.

No response

@seerge
Copy link
Owner

seerge commented Mar 27, 2024

@moms-spaghetti hello,

I can try to add something like that but

  • BIOS never follows fan curve precisely, and often has deviations. You should understand it only as approximate guidance. So it doesn't make much sense in pixel-peeping here
  • Current UI logic is to immediately apply custom fan curve as soon as you release left mouse button (if "apply" is already checked). It won't work like that for keyboard obviously

@moms-spaghetti
Copy link
Author

Thanks @seerge I didn't know the bios mapping of fan curves wasn't precise - good to know.
Will the implementation for using keys to adjust points require an apply button? Maybe some sort of debounce before applying changes for each keystroke will help?

@seerge
Copy link
Owner

seerge commented Mar 28, 2024

@moms-spaghetti ok, I have checked deeper, and it's actually harder to do than I thought :) whole fan-editor is actually just a "modified" windows charts component. And it's a lot of work to allow selecting dots manually, handing their movements from keys, etc.

So on this stage I'm not a big fan of that.

In addition, as I have initially mentioned fan curve (set in % points under the hood), is quite approximate guidance for BIOS, not precise. So couple of pixels up or down in reality won't make any difference.

What UI scaling do you use ? cause on mine 150% or 200% it's really easy to drag dots quite precisely.

Did you know you can drag whole curve by holding SHIFT ?

@moms-spaghetti
Copy link
Author

moms-spaghetti commented Mar 28, 2024

@seerge Ah ok does sound like it'll be more pain than it's worth to implement for little gain so can understand you not wanting to go down that route.

I am using 1920x1200 100% scale screen size 16", The windows are quite small, even with a mouse it's 'fiddly' to use but not impossible. Maybe an alternative could be a pop out for the graphs which can be resized if it's not a rabbit hole to implement?

@seerge
Copy link
Owner

seerge commented Mar 28, 2024

@moms-spaghetti I can add maximize button to Fans+Power, so it will go full-screen
GHelper.zip

seerge added a commit that referenced this issue Mar 28, 2024
@seerge
Copy link
Owner

seerge commented Mar 29, 2024

@moms-spaghetti does it solve your problem ?

@moms-spaghetti
Copy link
Author

moms-spaghetti commented Mar 29, 2024

@seerge ah! that's excellent thank you! was the implementation of this easier?

edit: I'd say this one is completed ✅

@seerge seerge closed this as completed Mar 29, 2024
@seerge
Copy link
Owner

seerge commented Mar 29, 2024

@moms-spaghetti yes, that’s easier, you can check actual charges in a linked commit :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants