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

[PR0] Update NIDAQmx.h and other nidaqmx metadata files to v23.0 #881

Merged
merged 16 commits into from
Mar 14, 2023

Conversation

vigkre
Copy link
Contributor

@vigkre vigkre commented Mar 9, 2023

What does this Pull Request accomplish?

Updates daqmx metadata files to version 23.0

The source files that are updated to the version include,

  • NIDAQmx.h
  • enums.py
  • attributes.py
  • functions.py

The files are updated to the version in this https://github.com/ni/grpc-device-scrapigen/pull/136

  • Added a suppression logic ENUMS_SHOULD_BE_USED to exclude enums which are not used in grpc-device but are required in nidaqmx-python files.

Why should this Pull Request be merged?

Testing done

  • Regenerated the files in generated/nidaqmx directory and add to the PR

@vigkre vigkre changed the title Update NIDAQmx.h file to v23.0 [PR0] Update NIDAQmx.h file to v23.0 Mar 9, 2023
imports/include/NIDAQmx.h Show resolved Hide resolved
@vigkre vigkre changed the title [PR0] Update NIDAQmx.h file to v23.0 [PR0] Update NIDAQmx.h and other nidaqmx metadata files to v23.0 Mar 9, 2023
source/codegen/metadata/nidaqmx/functions.py Outdated Show resolved Hide resolved
source/codegen/metadata/nidaqmx/functions.py Outdated Show resolved Hide resolved
generated/nidaqmx/nidaqmx.proto Outdated Show resolved Hide resolved
generated/nidaqmx/nidaqmx.proto Outdated Show resolved Hide resolved
generated/nidaqmx/nidaqmx.proto Outdated Show resolved Hide resolved
generated/nidaqmx/nidaqmx.proto Outdated Show resolved Hide resolved
generated/nidaqmx/nidaqmx.proto Show resolved Hide resolved
generated/nidaqmx/nidaqmx.proto Show resolved Hide resolved
@vigkre vigkre force-pushed the users/vikram/updatedaqmx-sourcefiles-v23.0 branch from b025bbc to 8012f44 Compare March 10, 2023 07:30
@vigkre vigkre requested a review from bkeryan March 10, 2023 20:45
@vigkre vigkre requested a review from reckenro March 12, 2023 15:22
source/codegen/metadata/nidaqmx/functions.py Outdated Show resolved Hide resolved
generated/nidaqmx/nidaqmx.proto Outdated Show resolved Hide resolved
generated/nidaqmx/nidaqmx.proto Outdated Show resolved Hide resolved
generated/nidaqmx/nidaqmx.proto Outdated Show resolved Hide resolved
generated/nidaqmx/nidaqmx.proto Show resolved Hide resolved
generated/nidaqmx/nidaqmx.proto Show resolved Hide resolved
@reckenro reckenro added the source-breaking Change to proto file that will break client if they update label Mar 13, 2023
Copy link
Collaborator

@reckenro reckenro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I want to circle back after the comments are resolved mainly to double check how the .proto file is shaping up.

@vigkre vigkre requested review from reckenro and bkeryan March 14, 2023 10:39
@vigkre vigkre merged commit 1faa210 into main Mar 14, 2023
@vigkre vigkre deleted the users/vikram/updatedaqmx-sourcefiles-v23.0 branch March 14, 2023 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
source-breaking Change to proto file that will break client if they update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants