lfreist / hwinfo

cross platform C++ library for hardware information (CPU, RAM, GPU, ...)
MIT License
433 stars 74 forks source link

GPU: Add VendorID and DeviceID information under Windows #83

Closed julianxhokaxhiu closed 5 months ago

julianxhokaxhiu commented 5 months ago

Hello,

I managed to find some time and look at how the information is obtained in Windows. This is my proposal for the issue I've opened. Let me know what you think about it.

Also, the new example looks like this now after the patch:

----------------------------------- GPU -----------------------------------
GPU 0:
  vendor:           NVIDIA
  model:            NVIDIA GeForce RTX 3070 Laptop GPU
  driverVersion:    31.0.15.5186
  memory [MiB]:     4095
  frequency:        0
  cores:            0
  vendor_id:        10DE
  device_id:        249D
GPU 1:
  vendor:           Advanced Micro Devices, Inc.
  model:            AMD Radeon(TM) Graphics
  driverVersion:    31.0.21912.14
  memory [MiB]:     512
  frequency:        0
  cores:            0
  vendor_id:        1002
  device_id:        1638

Fixes #82

julianxhokaxhiu commented 5 months ago

Hello @lfreist thanks a lot for the nice feedback :)

Added one more commit to fix the format using clang-format. Hopefully I did the right thing.

Let me know, thanks!