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

daqmx: Pick up scrapigen daqmx metadata update #1115

Merged

Conversation

SiuFong-NI
Copy link
Contributor

What does this Pull Request accomplish?

Pull latest scrapigen daqmx metadata from this PR: daqapi: Add new attribute and enum value for mioDAQ configurable digital voltage

Why should this Pull Request be merged?

This is needed to make sure the grpc-device repo proto gen for any changes with the updated metadata.

What testing has been done?

Local build successfully.

Signed-off-by: Siu Fong Tan <siu.fong.tan@ni.com>
Signed-off-by: Siu Fong Tan <siu.fong.tan@ni.com>
zhindes
zhindes previously requested changes Nov 5, 2024
Copy link
Contributor

@zhindes zhindes left a comment

Choose a reason for hiding this comment

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

We are gonna remove the new power stuff since its not fully baked; see Brad/my comments on the scrapigen Pr

@bkeryan bkeryan linked an issue Nov 6, 2024 that may be closed by this pull request
@maxxboehme maxxboehme added the source-breaking Change to proto file that will break client if they update label Nov 6, 2024
@SiuFong-NI
Copy link
Contributor Author

We are gonna remove the new power stuff since its not fully baked; see Brad/my comments on the scrapigen Pr

Updated in latest commit, verified that generated files excluded CalculatedPower related stuff.

@SiuFong-NI SiuFong-NI dismissed zhindes’s stale review November 7, 2024 02:53

Updated in latest commit, verified that generated files excluded CalculatedPower related stuff.

@SiuFong-NI
Copy link
Contributor Author

Pending grpc-device-scrapigen https://github.com/ni/grpc-device-scrapigen/pull/251 to be merged before merging this PR.

@SiuFong-NI SiuFong-NI merged commit 44fde0f into ni:main Nov 8, 2024
10 checks passed
@SiuFong-NI SiuFong-NI deleted the users/siutan/pickupPortLogicFamilyMetadata branch November 8, 2024 15:55
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.

Update DAQmx support to set runtime environment for CEIP
5 participants