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

eGPUs identifiy as notebook #182

Open
jwatel1 opened this issue Jun 14, 2023 · 8 comments
Open

eGPUs identifiy as notebook #182

jwatel1 opened this issue Jun 14, 2023 · 8 comments
Assignees
Labels
bug Something isn't working (as intended) help wanted Extra attention is needed

Comments

@jwatel1
Copy link

jwatel1 commented Jun 14, 2023

Hi, looks like the program is getting confused when I'm trying to use it to update my eGPU drivers. I'm using an RTX 3060Ti in a Razer core X Chroma connected via Thunderbolt 3 to my laptop.

Here's the output I'm getting:

TinyNvidiaUpdateChecker v1.16.5

Arg: --debug

configFile: C:\Users\jules\AppData\Local\Hawaii_Beach\TinyNvidiaUpdateChecker\app.config
CHECK_UPDATE: true
MINIMAL_INSTALL: true
DOWNLOAD_LOCATION: fr

Verifying internet connection . . . OK!

Searching for Updates . . . OK!
offlineVer: 1.16.5
onlineVer:  1.16.5

Retrieving GPU information . . . ERROR!
GPU metadata for your card does not exist! Please file an issue on GitHub and include the following information:

gpuName:    GeForce RTX 3060 Ti
isNotebook: True

Press any key to exit...

Thanks!

@ElPumpo
Copy link
Owner

ElPumpo commented Jun 17, 2023

Hi and first thanks for the bug report.

Correct TNUC does currently not support eGPUs and neither more than one NVIDIA GPU.

TNUC uses code to determine if your computer is a laptop or desktop no matter if the GPU accually is.. not sure how to solve it without some special command line argument

@ElPumpo ElPumpo changed the title Can't properly find GPU driver when using eGPU eGPUs identifiy as notebook Aug 20, 2023
@ElPumpo ElPumpo self-assigned this Aug 20, 2023
@ElPumpo ElPumpo added the bug Something isn't working (as intended) label Aug 20, 2023
@ElPumpo
Copy link
Owner

ElPumpo commented Aug 20, 2023

[..] I don't know if there's a way to identify if a GPU is an "eGPU". If there is no way then the only solution I see is using command line arguments, like other projects have used in the past.

I belive the inital fix would be to provide a command line argument overriding notebook/desktop identify code. I'll keep this issue open until automatic solution can be implemented.

I'll start development on the workaround.

@ElPumpo ElPumpo modified the milestone: 1.17.0 Aug 20, 2023
@ElPumpo
Copy link
Owner

ElPumpo commented Aug 20, 2023

v1.17.0 has been released with the following new command line arguments:

  • --override-desktop
  • --override-notebook

@ElPumpo
Copy link
Owner

ElPumpo commented Jun 7, 2024

Perhaps a check box in the new GPU selector "Override desktop/notebook identification"

@ElPumpo ElPumpo added the help wanted Extra attention is needed label Jun 7, 2024
@jwatel1 jwatel1 closed this as completed Jun 10, 2024
@ElPumpo ElPumpo reopened this Jun 10, 2024
@ElPumpo
Copy link
Owner

ElPumpo commented Jun 10, 2024

J I wish not to close this issue until a better solution then arguments are added

@ElPumpo
Copy link
Owner

ElPumpo commented Jun 27, 2024

Animation

@ElPumpo
Copy link
Owner

ElPumpo commented Jun 27, 2024

Now I need code that saves the overriden settings for the GPU because we use Chassies Type only currently.. it's not ideal

@ElPumpo
Copy link
Owner

ElPumpo commented Jun 28, 2024

Accually that idea was scapped, and the referenced commit has a fix for this which is very simple. Just invert the notebook check if no Gpu Id is found for that type. Having a override Gpu Type code was too complex for no good reason.

I have pushed the code to branch type-override maybe I'll review it in the future

Because if the GPU name exists in both Desktop and Notebook, then we still need a proper override like the GIF shows, and the referenced commit wont fix that issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working (as intended) help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants