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

Add codegen support for Daqmx Write apis which have multiple in params. #1130

Merged
merged 27 commits into from
Nov 28, 2024

Conversation

Raghav-NI
Copy link
Contributor

What does this Pull Request accomplish?

This PR adds codegen support for remaining Daqmx Write apis which have multiple in params.

Why should this Pull Request be merged?

Generated streaming counterpart for all the remaining Daqmx write apis needed for streaming.

What testing has been done?

I am able to build and ran server without any errors. We will test the apis with the integration test we are working on.

@Raghav-NI Raghav-NI marked this pull request as ready for review November 26, 2024 14:55
Base automatically changed from users/rrawat/multiple_param_daqmx to main November 27, 2024 17:04
@Raghav-NI Raghav-NI merged commit 8f66808 into main Nov 28, 2024
9 of 10 checks passed
@Raghav-NI Raghav-NI deleted the users/rrawat/multiple_param_daqmx_write_apis branch November 28, 2024 09:04
doshirohan added a commit that referenced this pull request Dec 4, 2024
…tegration-tests

* origin/main:
  handle null (#1135)
  cleanup (#1133)
  Add codegen support for Daqmx Write apis which have multiple in params. (#1130)
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