seerge / g-helper

Lightweight Armoury Crate alternative for Asus laptops and ROG Ally. Control tool for ROG Zephyrus G14, G15, G16, M16, Flow X13, Flow X16, TUF, Strix, Scar and other models
https://seerge.github.io/g-helper/
GNU General Public License v3.0
7.04k stars 255 forks source link

Rog Ally Some Improvements #769

Closed Nidre closed 1 year ago

Nidre commented 1 year ago

Hello, I am the owner of Rog Ally and I have discovered some possible improvements to the program.

  1. From the general when setting fan speed you could do like this: 1

It would be cool if there were "barriers" for example 10% for each slider.

Now about Rog Ally:

  1. By default we have 1920x1080 and a scale of 150%, so the Extra window looks like this: 2

  2. There is no dGPU on Ally and the "Eco" and "Optimized" mods should ideally not be displayed and ignored for XG Mobile connectivity.

4, I would also like to see a function to disable tracking of "Mode" and "Fans" to give control of this to another app. For example, it's almost mandatory for Ally to use Handheld Companion, which better adjusts the TDP for handheld computers.

  1. The maximum CPU limit looks like this: 3

  2. There are 2 fans (left and right) on the Ally, both cooling the CPU, the program also shows CPU/GPU fans.

Also can I ask what you use to connect/disconnect XG Mobile?

Invoke-CimMethod (Get-CimInstance -Namespace root/wmi -ClassName AsusAtkWmi_WMNB) -MethodName DEVS -Arguments @{Device_ID=0x090019; Control_status=1}

Or something a bit trickier?

And thank you for your program)

seerge commented 1 year ago

@Nidre hello

  1. Do you mean locking dots to 10% , 20%, 30% only instead of smooth dragging? I'm not sure if it's a good idea, even AC allows flexible settings :) (but i can imagine dots can be small enough for a small touch screen)
  2. In latest build I have made Extras screen a bit more compact GHelper.zip so it should fit into 1080P
  3. I know about absence of dGPU but by some reason Eco flag exists in bios (and even does something, i just didn't figure out what ... mb it adjusts RAM dedicated to GPU?)
  4. In this case I would advice to set TDP to max in g-helper (as they are anyway linked to one of predefined modes, and device is always in one of those modes, as they are baked in bios) and then apply custom limits via other apps.
  5. In AC maximum values are a bit lower than they can be in reality, when I cranked sliders in g-helper on my Ally it was sustaining 43-45W all the time (just in case in g-helper sPPT and SPL are merged into 1 slider cause i don't see sense to separate 2 min "boost" from sustained limit)
  6. I think even AC calls second fan "GPU fan", right ?
  7. Yes, that command is correct one (1 to connect, 0 to disconnect in Control_status), but g-helper also does that
Invoke-CimMethod (Get-CimInstance -Namespace root/wmi -ClassName AsusAtkWmi_WMNB) -MethodName DEVS -Arguments @{Device_ID=0x090019; Control_status=1}
Nidre commented 1 year ago
  1. In AC sliders just have barriers, the point is that it is impossible to overlap them, because they can be separated only by the reset button. There are probably no scenarios where one-degree control is required.

  2. Its latest build: 5

  3. We have a separate VRAM adjustment. I would assume "asus team" just left it "for later", and now it's really useless.

  4. I wanna use handless companion for cpu and msi afterburner for xg mobile, but on gpu tab there's no apply checkbox, like a cpu. I think it's not a bad idea to add a checkbox to the GPU tab.

  5. Fan 1 and Fan 2. (Fan 1 i think near the SD reader, its left and fan 2 is right)

In addition you have the files bloat.bat and debloat.bat There's no Ally service there: ArmouryCrateSEService image

seerge commented 1 year ago
  1. Dots can be dragged independently anyway
  2. Currently extra screen has multiple bindings not really relevant for ally, but since ally is like 0.5% i don't plan to adjust app specifically for that
  3. GPU tab will pick up afterburner settings

Not all services in debloat/bloat has to be on system, it's universal to cover most cases

Closing as it's not relevant atm