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

Updated gRPC Device code for all 24C1 RFmx APIs #1047

Merged
merged 6 commits into from
Mar 20, 2024

Conversation

RahulBenaka
Copy link
Contributor

@RahulBenaka RahulBenaka commented Mar 13, 2024

gRPC Supported personalities - BT, LTE, NR, SpecAn and WLAN

What does this Pull Request accomplish?
Updates niRFmxSpecAn in grpc-device-scrapigen to 24.3.0
Updates niRFmxLTE in grpc-device-scrapigen to 24.3.0
Updates niRFmxNR in grpc-device-scrapigen to 24.3.0
Updates niRFmxWLAN in grpc-device-scrapigen to 24.3.0
Updates the gRPC Scrapigen Device Code for SpecAn Marker APIs.
Updates the gRPC Scrapigen Device Code for LTE List Mode APIs, attributes and enums.
Updates the gRPC Scrapigen Device Code for new BT ModSpectrum API and attributes.

Why should this Pull Request be merged?
Updated to match the upcoming release of niRFmxSpecAn 24.3.0
Updated to match the upcoming release of niRFmxLTE 24.3.0
Updated to match the upcoming release of niRFmxNR 24.3.0
Updated to match the upcoming release of niRFmxWLAN 24.3.0

What testing has been done?
Manually inspected generated files.
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.
Copied files from grpc-device-scrapigen/out/rfmxlte/export/ to grpc-device/source/codegen/metadata/nirfmxlte/ and built grpc-device successfully.
Manually inspected nirfmxlte.proto file.
Copied files from grpc-device-scrapigen\out\rfmxbluetooth\export to grpc-device/source/codegen/metadata/ nirfmxbluetooth / and built grpc-device successfully.
Manually inspected nirfmxbluetooth.proto file.
Copied files from grpc-device-scrapigen\out\rfmxnr\export to grpc-device/source/codegen/metadata/ nirfmxnr/ and built grpc-device successfully.
Manually inspected nirfmxnr.proto file.
Copied files from grpc-device-scrapigen\out\rfmxwlan\export to grpc-device/source/codegen/metadata/ nirfmxwlan/ and built grpc-device successfully.
Manually inspected nirfmxwlan.proto file.
Manually inspected nirfmxbluetooth.proto file.

@RahulBenaka RahulBenaka changed the title Updated grpc device code for SpecAn 24C1 Marker APIs Updated gRPC Device code for all 24C1 APIs Mar 15, 2024
@maxxboehme
Copy link
Collaborator

@RahulBenaka @reckenro @astarche I am reruning the windows tests as they appeared to have failed due to some labview crash.

@RahulBenaka
Copy link
Contributor Author

RahulBenaka commented Mar 19, 2024

@RahulBenaka @reckenro @astarche I am reruning the windows tests as they appeared to have failed due to some labview crash.

I see that all the checks have passed successfully.
@astarche @reckenro @maxxboehme
Can you please approve the PR? I had reset the vote of approval as I pushed new changes. Sorry for the inconvenience. I need to submit these as soon as possible as it is required for the release.

@reckenro reckenro changed the title Updated gRPC Device code for all 24C1 APIs Updated gRPC Device code for all 24C1 RFmx APIs Mar 19, 2024
@RahulBenaka RahulBenaka added binary-breaking Change to proto file that requires client updates source-breaking Change to proto file that will break client if they update labels Mar 20, 2024
@RahulBenaka
Copy link
Contributor Author

@astarche Can you please review the PR? I need this PR to be merged before release branch is created.

@RahulBenaka RahulBenaka merged commit e034b69 into main Mar 20, 2024
11 checks passed
@RahulBenaka RahulBenaka deleted the users/rnagaraj/specan_24C1_apis branch March 20, 2024 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
binary-breaking Change to proto file that requires client updates 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

5 participants