TinyNvidiaUpdateChecker icon indicating copy to clipboard operation
TinyNvidiaUpdateChecker copied to clipboard

eGPUs identifiy as notebook

Open jwatel1 opened this issue 1 year ago • 8 comments

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!

jwatel1 avatar Jun 14 '23 13:06 jwatel1

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 avatar Jun 17 '23 21:06 ElPumpo

[..] 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 avatar Aug 20 '23 08:08 ElPumpo

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

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

ElPumpo avatar Aug 20 '23 12:08 ElPumpo

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

ElPumpo avatar Jun 07 '24 11:06 ElPumpo

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

ElPumpo avatar Jun 10 '24 16:06 ElPumpo

Animation

ElPumpo avatar Jun 27 '24 06:06 ElPumpo

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

ElPumpo avatar Jun 27 '24 06:06 ElPumpo

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

ElPumpo avatar Jun 28 '24 09:06 ElPumpo