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

RFmx VNA and SpecAn 24C4 API changes #1140

Merged
merged 1 commit into from
Dec 13, 2024
Merged

Conversation

RahulBenaka
Copy link
Contributor

What does this Pull Request accomplish?

Updates niRFmxSpecAn in grpc-device-scrapigen to 25.0.0
Updates niRFmxVNA in grpc-device-scrapigen to 25.0.0
Updates the gRPC Scrapigen Device Code for SPecAn AMPM properties
Updates the gRPC Scrapigen Device Code for VNA LoadDataToMeasurementMemoryFromFile API parameter's name change from memoryDataName to measurementMemoryDataName

Why should this Pull Request be merged?

Updated to match the upcoming release of niRFmxSpecAn 25.0.0
Updated to match the upcoming release of niRFmxVNA 25.0.0

What testing has been done?

Manually inspected generated files.
Copied files from grpc-device-scrapigen/out/rfmxvna/export/ to grpc-device/source/codegen/metadata/nirfmxvna/ and built grpc-device successfully.
Manually inspected nirfmxvna.proto file.
Copied files from grpc-device-scrapigen/out/rfmxspecan/export/ to grpc-device/source/codegen/metadata/nirfmxspecan/ and built grpc-device successfully.
Manually inspected nirfmxspecan.proto file.

@reckenro reckenro added the source-breaking Change to proto file that will break client if they update label Dec 12, 2024
@RahulBenaka RahulBenaka merged commit 1b17c38 into main Dec 13, 2024
10 checks passed
@RahulBenaka RahulBenaka deleted the users/rnagaraj/api_addition branch December 13, 2024 05:14
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.

4 participants