NVlabs / FPSci

Aim Training Experiments
Other
67 stars 23 forks source link

Add `logSystemInfo` parameter and USB VID/PID lookup #331

Open bboudaoud-nv opened 2 years ago

bboudaoud-nv commented 2 years ago

This branch adds support for logging SystemInfo to the output database as a new System_Info table. It also adds support for getting the VID/PID of all USB devices attached to the system and logging this as part of SystemInfo (which is also written to log.txt as it has been historically).

In order to get VID/PID this branch adds a dependency on setupapi.lib which will now need to be packaged with the project. This is currently a windows-only feature.

There is no support built-in for identifying mice/keyboards from other USB devices, but the VID/PID of every connected device is reported. An optional vid_pid.json file has been added to the data-files directory to provide a lookup of vendor/product names from the VID/PID pair. This file is produced using a Python web-scraping tool.

Merging this PR closes #288.

jspjutNV commented 2 years ago

Is setupapi.lib a G3D project? This is a great idea, but we need to make sure we can package the library before we can merge this one.