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

Dynamic Response Time aka Overdrive missing #1291

Closed
3 tasks done
SpectreDev1 opened this issue May 17, 2024 · 16 comments · Fixed by #1296
Closed
3 tasks done

Dynamic Response Time aka Overdrive missing #1291

SpectreDev1 opened this issue May 17, 2024 · 16 comments · Fixed by #1296
Assignees
Labels
bug Something isn't working

Comments

@SpectreDev1
Copy link

Rules

  • I made myself familiar with the README.
  • I checked closed issues for similar ones and did not find a similar problem mentioned before.
  • I understand that, if insufficient information was provided, my issue will be closed without an answer.

Version

Latest Beta

OS

Win 11 Latest

Device

Legion 5 Gen 9

BIOS version

NMCN22WW

What's wrong?

well lenovo renamed the overdrive mode to dynamic response time, overdrive mode doesnt show up in LLT at all for Legion 5 Gen 9

image

How to reproduce the bug?

Its missing entirely

What is the behavior that you expected?

Expect Overdrive/Dynamic Response Time to be there

Logs

dont think its possible to log

Additional information

No response

@SpectreDev1 SpectreDev1 added the bug Something isn't working label May 17, 2024
@SpectreDev1 SpectreDev1 changed the title Dynamic Response Time aka Overdrive Dynamic Response Time aka Overdrive missing May 17, 2024
@BartoszCichecki BartoszCichecki added the needs-information More information is needed label May 17, 2024
@BartoszCichecki
Copy link
Owner

Hmmmm that is a tricky... what was the default value for it? Was it off by default?

@BartoszCichecki
Copy link
Owner

Anyway, let's start with a WMI dump from Gen 9, because I don't have it yet. Please post the output of this powershell script:

$wmi_classes = Get-WmiObject -Namespace 'ROOT/WMI' -List -Class "*LENOVO*"
foreach ($wmi_class in $wmi_classes){
  Write-Host "########################################"
  Write-Host "########################################"
  Write-Host "########################################"
  Write-Host "Name:" $wmi_class.Name
  Write-Host "Class Name:" $wmi_class.Name
  Write-Host "Class GUID:" $wmi_class.Qualifiers["guid"].Value
  Write-Host "Description:" $wmi_class.Methods.Count
  Write-Host "Methods:"
  foreach ($method in $wmi_class.Methods){
    Write-Host "Name:" $method.Name
    Write-Host "WmiMethodId:" $method.Qualifiers["WmiMethodId"].Value
    Write-Host "Class Name:" $wmi_class.Name
    Write-Host "Class GUID:" $wmi_class.Qualifiers["guid"].Value
    Write-Host "Description:" $method.Qualifiers["Description"].Value
    Write-Host "Implemented:" $method.Qualifiers["Implemented"].Value
    Write-Host ""
  }
  Write-Host ""
}

@SpectreDev1
Copy link
Author

Hmmmm that is a tricky... what was the default value for it? Was it off by default?

Yep it's off by default

@SpectreDev1
Copy link
Author

Anyway, let's start with a WMI dump from Gen 9, because I don't have it yet. Please post the output of this powershell script:

$wmi_classes = Get-WmiObject -Namespace 'ROOT/WMI' -List -Class "*LENOVO*"
foreach ($wmi_class in $wmi_classes){
  Write-Host "########################################"
  Write-Host "########################################"
  Write-Host "########################################"
  Write-Host "Name:" $wmi_class.Name
  Write-Host "Class Name:" $wmi_class.Name
  Write-Host "Class GUID:" $wmi_class.Qualifiers["guid"].Value
  Write-Host "Description:" $wmi_class.Methods.Count
  Write-Host "Methods:"
  foreach ($method in $wmi_class.Methods){
    Write-Host "Name:" $method.Name
    Write-Host "WmiMethodId:" $method.Qualifiers["WmiMethodId"].Value
    Write-Host "Class Name:" $wmi_class.Name
    Write-Host "Class GUID:" $wmi_class.Qualifiers["guid"].Value
    Write-Host "Description:" $method.Qualifiers["Description"].Value
    Write-Host "Implemented:" $method.Qualifiers["Implemented"].Value
    Write-Host ""
  }
  Write-Host ""
}

I'll do this tomorrow for you

@SpectreDev1
Copy link
Author

https://pastebin.com/ECyw0hKU here is the dump

@SpectreDev1
Copy link
Author

btw i have the 240hz panel, not sure if that makes any difference for you

@BartoszCichecki
Copy link
Owner

Hmmm... ok let's try something...

Check if this returns different values depending on what you select in the dropdown. Make sure that you click "Done" after each change:

(Get-WmiObject -namespace root\WMI -class LENOVO_OTHER_METHOD).GetFeatureValue(1703936).value

If it works, write down the values it returns for each of the options from the dropdown, so I can map them correctly.

@SpectreDev1
Copy link
Author

Hmmm... ok let's try something...

Check if this returns different values depending on what you select in the dropdown. Make sure that you click "Done" after each change:

(Get-WmiObject -namespace root\WMI -class LENOVO_OTHER_METHOD).GetFeatureValue(1703936).value

If it works, write down the values it returns for each of the options from the dropdown, so I can map them correctly.

1 for enabled 0 for disabled

@BartoszCichecki
Copy link
Owner

Great, we have a jackpot 😎 thanks for testing!

@SpectreDev1
Copy link
Author

no problem

@BartoszCichecki BartoszCichecki removed the needs-information More information is needed label May 18, 2024
@BartoszCichecki BartoszCichecki self-assigned this May 18, 2024
@BartoszCichecki
Copy link
Owner

Here is a test build you can try: installer

Please upload a log as well and see if some other options didn't disappear, because I did a small refactoring too.

@SpectreDev1
Copy link
Author

nothing has disappeared but overdrive still does not show up
log_2024_05_19_11_17_46.txt

@BartoszCichecki
Copy link
Owner

I think you had another instance running because log is broken.

@SpectreDev1
Copy link
Author

yep my bad, heres another log
log_2024_05_19_11_27_36.txt

@BartoszCichecki
Copy link
Owner

Shit, I made a typo :D Here's another one: installer

@SpectreDev1
Copy link
Author

yep it shows up now
log_2024_05_19_11_40_45.txt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants