Skip to content
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

examples: Update NI-Scope, NI-Switch and NI-Fgen TestStand sequences with the Update pin map custom step #540

Merged
merged 1 commit into from
Dec 8, 2023

Conversation

Avinash2Suresh
Copy link
Contributor

What does this Pull Request accomplish?

  • The Update Pin map and Clean up action steps are replaced with the "Update pin map" step.
  • Removed the teststand_pinmap.py file and an unused function in _helpers.py in the respective ../examples/{example_measurement_folder}/ directory.
  • Removed the FileGlobals.MeasurementLink.PinMapPath from the sequences.

Why should this Pull Request be merged?

Updates the NIScopeAcquireWaveform.seq, NISwitchControlRelays.seq, and NIFgenStandardFunction.seq to use the "Update pin map" step - Task 2586982.

What testing has been done?

Ran the NI-Scope, NI-Switch, and NI-Fgen sequences with the simulated instruments by using the TestStand measurement client that is available in the main branch.

Copy link

github-actions bot commented Dec 7, 2023

Test Results

       30 files  ±0         30 suites  ±0   35m 25s ⏱️ - 3m 21s
     544 tests ±0       544 ✔️ ±0         0 💤 ±0  0 ±0 
13 300 runs  ±0  12 240 ✔️ ±0  1 060 💤 ±0  0 ±0 

Results for commit 79c4c15. ± Comparison against base commit 82494b5.

@Avinash2Suresh Avinash2Suresh changed the title [Internal] examples: Update NI-Scope, NI-Switch and NI-Fgen TestStand sequences with the Update pin map custom step examples: Update NI-Scope, NI-Switch and NI-Fgen TestStand sequences with the Update pin map custom step Dec 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants