-
Notifications
You must be signed in to change notification settings - Fork 162
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
Automate Driver Install Experience within nidaqmx Module #512
Automate Driver Install Experience within nidaqmx Module #512
Conversation
Adding file to install DAQmxDriver
FYI @dharaniprakashkm - I get a lot of review requests, so until this is marked "Ready for Review", I don't intend to look at it. That doesn't mean you should just smash that button, peer review and iteration is great! Also, definitely feel free to ping me on a specific issue if you like! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really great start. Sorry I missed the demo :(
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't get to the code yet. I'll post another review when I do.
Co-authored-by: Brad Keryan <[email protected]>
…b.com/dharaniprakashkm/nidaqmx-python into users/inkurdid/python_daqmx_installer
Co-authored-by: Brad Keryan <[email protected]>
…b.com/dharaniprakashkm/nidaqmx-python into users/inkurdid/python_daqmx_installer
What does this Pull Request accomplish?
Why should this Pull Request be merged?
We need to install the NI-DAQmx Driver to have Python library functional.
This script will improve the driver install experience for Windows Users.
What testing has been done?
Verified the script works fine on both Windows 11 and Windows Server 2016 OS.
Verified script handles/returns proper error info in the following cases.