ni / niveristand-scan-engine-ethercat-custom-device

Provides NI Scan Engine, EtherCAT and Remote IO support for NI VeriStand
MIT License
16 stars 22 forks source link

Move dialog handling for out-of-process application to calling VIs #189

Closed Karl-G1 closed 2 years ago

Karl-G1 commented 2 years ago

What does this Pull Request accomplish?

Major Changes

Minor Changes

Why should this Pull Request be merged?

The primary change is to move dialog handling for out-of-process application to calling VIs. This was done for 1) correctness and 2) it was stalling out the automated tests because LabVIEW could not squash the one-button dialog launched when downloading a 9145 bitfile.

What testing has been done?

Lots of hand testing with VeriStand 2020 R6. ATS tests now pass without user intervention (with LabVIEW set to unattended mode).

niveristand-diff-bot commented 2 years ago

Bleep bloop!

LabVIEW Diff Robot here with some diffs served up hot for your pull request.

Notice something funny? Help fix me on my GitHub repo.

SEECD System Explorer.lvlib--Build Command Line Arguments.vi.png ![capture](https://raw.githubusercontent.com/niveristand-diff-bot/diff-images/master/NI/niveristand-scan-engine-ethercat-custom-device/PR-189/2021-11-08/17%3A27%3A12/SEECD%20System%20Explorer.lvlib--Build%20Command%20Line%20Arguments.vi.png)
SEECD System Explorer.lvlib--Build Installed Folder Paths.vi.png ![capture](https://raw.githubusercontent.com/niveristand-diff-bot/diff-images/master/NI/niveristand-scan-engine-ethercat-custom-device/PR-189/2021-11-08/17%3A27%3A12/SEECD%20System%20Explorer.lvlib--Build%20Installed%20Folder%20Paths.vi.png)
SEECD System Explorer.lvlib--Call Check and Download Bitfile.vi.png ![capture](https://raw.githubusercontent.com/niveristand-diff-bot/diff-images/master/NI/niveristand-scan-engine-ethercat-custom-device/PR-189/2021-11-08/17%3A27%3A12/SEECD%20System%20Explorer.lvlib--Call%20Check%20and%20Download%20Bitfile.vi.png)
SEECD System Explorer.lvlib--Call Get HW Config.vi.png ![capture](https://raw.githubusercontent.com/niveristand-diff-bot/diff-images/master/NI/niveristand-scan-engine-ethercat-custom-device/PR-189/2021-11-08/17%3A27%3A12/SEECD%20System%20Explorer.lvlib--Call%20Get%20HW%20Config.vi.png)
SEECD System Explorer.lvlib--Call Revert to Scan Mode.vi.png ![capture](https://raw.githubusercontent.com/niveristand-diff-bot/diff-images/master/NI/niveristand-scan-engine-ethercat-custom-device/PR-189/2021-11-08/17%3A27%3A12/SEECD%20System%20Explorer.lvlib--Call%20Revert%20to%20Scan%20Mode.vi.png)
SEECD System Explorer.lvlib--Check and Download Bitfile.vi.png ![capture](https://raw.githubusercontent.com/niveristand-diff-bot/diff-images/master/NI/niveristand-scan-engine-ethercat-custom-device/PR-189/2021-11-08/17%3A27%3A12/SEECD%20System%20Explorer.lvlib--Check%20and%20Download%20Bitfile.vi.png)
SEECD System Explorer.lvlib--Display Message Dialog.vi.png ![capture](https://raw.githubusercontent.com/niveristand-diff-bot/diff-images/master/NI/niveristand-scan-engine-ethercat-custom-device/PR-189/2021-11-08/17%3A27%3A12/SEECD%20System%20Explorer.lvlib--Display%20Message%20Dialog.vi.png)
SEECD System Explorer.lvlib--ECAT Download Error Handle.vi.png ![capture](https://raw.githubusercontent.com/niveristand-diff-bot/diff-images/master/NI/niveristand-scan-engine-ethercat-custom-device/PR-189/2021-11-08/17%3A27%3A12/SEECD%20System%20Explorer.lvlib--ECAT%20Download%20Error%20Handle.vi.png)
SEECD System Explorer.lvlib--ECAT FPGA OD FoE.vi.png ![capture](https://raw.githubusercontent.com/niveristand-diff-bot/diff-images/master/NI/niveristand-scan-engine-ethercat-custom-device/PR-189/2021-11-08/17%3A27%3A12/SEECD%20System%20Explorer.lvlib--ECAT%20FPGA%20OD%20FoE.vi.png)
SEECD System Explorer.lvlib--ECAT Wipeback FoE.vi.png ![capture](https://raw.githubusercontent.com/niveristand-diff-bot/diff-images/master/NI/niveristand-scan-engine-ethercat-custom-device/PR-189/2021-11-08/17%3A27%3A12/SEECD%20System%20Explorer.lvlib--ECAT%20Wipeback%20FoE.vi.png)
SEECD System Explorer.lvlib--Generate Application Call Error.vi.png ![capture](https://raw.githubusercontent.com/niveristand-diff-bot/diff-images/master/NI/niveristand-scan-engine-ethercat-custom-device/PR-189/2021-11-08/17%3A27%3A12/SEECD%20System%20Explorer.lvlib--Generate%20Application%20Call%20Error.vi.png)
SEECD System Explorer.lvlib--Get HW Config.vi.png ![capture](https://raw.githubusercontent.com/niveristand-diff-bot/diff-images/master/NI/niveristand-scan-engine-ethercat-custom-device/PR-189/2021-11-08/17%3A27%3A12/SEECD%20System%20Explorer.lvlib--Get%20HW%20Config.vi.png)
SEECD System Explorer.lvlib--Host - Download.vi.png ![capture](https://raw.githubusercontent.com/niveristand-diff-bot/diff-images/master/NI/niveristand-scan-engine-ethercat-custom-device/PR-189/2021-11-08/17%3A27%3A12/SEECD%20System%20Explorer.lvlib--Host%20-%20Download.vi.png)
SEECD System Explorer.lvlib--Host - Get HW Config.vi.png ![capture](https://raw.githubusercontent.com/niveristand-diff-bot/diff-images/master/NI/niveristand-scan-engine-ethercat-custom-device/PR-189/2021-11-08/17%3A27%3A12/SEECD%20System%20Explorer.lvlib--Host%20-%20Get%20HW%20Config.vi.png)
SEECD System Explorer.lvlib--Host - Get Local and ECAT Masters With Retries.vi.png ![capture](https://raw.githubusercontent.com/niveristand-diff-bot/diff-images/master/NI/niveristand-scan-engine-ethercat-custom-device/PR-189/2021-11-08/17%3A27%3A12/SEECD%20System%20Explorer.lvlib--Host%20-%20Get%20Local%20and%20ECAT%20Masters%20With%20Retries.vi.png)
SEECD System Explorer.lvlib--Host - Get Local and ECAT Masters.vi.png ![capture](https://raw.githubusercontent.com/niveristand-diff-bot/diff-images/master/NI/niveristand-scan-engine-ethercat-custom-device/PR-189/2021-11-08/17%3A27%3A12/SEECD%20System%20Explorer.lvlib--Host%20-%20Get%20Local%20and%20ECAT%20Masters.vi.png)
SEECD System Explorer.lvlib--Host - Is Download Necessary.vi.png ![capture](https://raw.githubusercontent.com/niveristand-diff-bot/diff-images/master/NI/niveristand-scan-engine-ethercat-custom-device/PR-189/2021-11-08/17%3A27%3A12/SEECD%20System%20Explorer.lvlib--Host%20-%20Is%20Download%20Necessary.vi.png)
SEECD System Explorer.lvlib--Host - Revert.vi.png ![capture](https://raw.githubusercontent.com/niveristand-diff-bot/diff-images/master/NI/niveristand-scan-engine-ethercat-custom-device/PR-189/2021-11-08/17%3A27%3A12/SEECD%20System%20Explorer.lvlib--Host%20-%20Revert.vi.png)
SEECD System Explorer.lvlib--Revert to Scan Mode.vi.png ![capture](https://raw.githubusercontent.com/niveristand-diff-bot/diff-images/master/NI/niveristand-scan-engine-ethercat-custom-device/PR-189/2021-11-08/17%3A27%3A12/SEECD%20System%20Explorer.lvlib--Revert%20to%20Scan%20Mode.vi.png)
Karl-G1 commented 2 years ago
  • [x] Did the FoE VIs change? If not, please revert them.

This was a consequence of moving the sub VIs of the Applications into a Utilities folder.