-
Notifications
You must be signed in to change notification settings - Fork 52
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add codegen support for Daqmx Write apis which have multiple in param…
…s. (#1130) * add new meesage to proto for moniker streaming call * work-in-progress: codegen changes for service.cpp generation with response on moniker out param * run codegen after resolving merge conflicts * return failure error code from moniker APIs * update proto and struct to contain request and response messages * standardize to send output in response parameter for moniker APIs * fix black formatter issues * fix build error * cleanup scripts * fix tests * script and cpp cleanup * codegen for read apis with multiple out params * added condition to call reserve and resize functions for some param types * add type hints for python helpers * fix black formatter issues * return error status from Write* moniker functions * fix type hints * codegen for write apis with multiple out params * merged latest main * cleanup * resolved build errors * fixed failing tests * removed comment from code --------- Co-authored-by: Rohan Doshi <rohan.doshi@ni.com> Co-authored-by: Raghav Rawat <raghavrawat@ni.com>
- Loading branch information
1 parent
b966222
commit 8f66808
Showing
11 changed files
with
1,660 additions
and
460 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.