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

Update DAQmx enums and attributes scrapigen metadata & generate proto files #880

Merged
merged 45 commits into from
Mar 15, 2023

Conversation

vigkre
Copy link
Contributor

@vigkre vigkre commented Mar 9, 2023

What does this Pull Request accomplish?

  • Created this PR to test the grpc-device with the updated scrapigen metadata (enums.py and attributes.py)

Notes:

  • Added ENUMS_SHOULD_BE_USED suppression for enums which is not used in grpc-device but are required in nidaqmx-python files.
  • Added Optional parameters in metadata_validation.py file to align with newly added fields in enums.py and attributes.py metadata files

Testing done

  • Regenerated the files in generated/nidaqmx directory.
  • No change in Proto files 😊

@vigkre vigkre force-pushed the users/vikram/generate-proto-with-updated-scrapigen branch from 6e10426 to fb655d0 Compare March 9, 2023 14:54
@vigkre vigkre requested a review from keer1111 March 9, 2023 15:03
@vigkre vigkre force-pushed the users/vikram/generate-proto-with-updated-scrapigen branch from fb655d0 to 9354ed9 Compare March 9, 2023 15:28
@vigkre vigkre changed the base branch from main to users/vikram/updatedaqmx-sourcefiles-v23.0 March 9, 2023 20:05
Copy link
Contributor

@bkeryan bkeryan left a comment

Choose a reason for hiding this comment

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

This looks pretty good, but there are a couple of unexplained compatibility breakages that you should fix.

generated/nidaqmx/nidaqmx.proto Outdated Show resolved Hide resolved
generated/nidaqmx/nidaqmx.proto Outdated Show resolved Hide resolved
source/codegen/metadata_validation.py Outdated 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
…vikram/generate-proto-with-updated-scrapigen
@vigkre vigkre changed the title [DRAFT] Updated enums and attributes scrapigen metadata & proto files generated Updated enums and attributes scrapigen metadata & proto files generated Mar 10, 2023
@vigkre vigkre marked this pull request as ready for review March 10, 2023 20:34
@vigkre vigkre requested a review from bkeryan March 10, 2023 20:45
@vigkre vigkre changed the title Updated enums and attributes scrapigen metadata & proto files generated Update DAQmx enums and attributes scrapigen metadata & generate proto files Mar 13, 2023
Base automatically changed from users/vikram/updatedaqmx-sourcefiles-v23.0 to main March 14, 2023 16:57
@vigkre vigkre merged commit 54aa081 into main Mar 15, 2023
@vigkre vigkre deleted the users/vikram/generate-proto-with-updated-scrapigen branch March 15, 2023 15:48
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.

None yet

4 participants