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 niRFmxLTE to 23.0 #853

Merged

Conversation

WesleyTangNationalInstruments
Copy link
Contributor

@WesleyTangNationalInstruments WesleyTangNationalInstruments commented Feb 15, 2023

What does this Pull Request accomplish?

Updates niRFmxLTE support to 23.0 API. Metadata files were sourced from grpc-device-scrapigen (see this PR).

The adds the following:

  • Enums

    • SemUplinkMaskType
      • SEM_UPLINK_MASK_TYPE_NS28, SEM_UPLINK_MASK_TYPE_CANS09, SEM_UPLINK_MASK_TYPE_CANS10
      • Note: SEM_UPLINK_MASK_TYPE_NS27 was changed to SEM_UPLINK_MASK_TYPE_NS27_OR_NS43
    • MODACC_SYMBOL_CLOCK_ERROR_ESTIMATION_ENABLED
    • MODACC_PRE_FFT_ERROR_ESTIMATION_INTERVAL
    • MODACC_TIMING_TRACKING_ENABLED
    • MODACC_PHASE_TRACKING_ENABLED
  • Attributes

    • MODACC_RESULTS_MAXIMUM_PEAK_IQ_ORIGIN_OFFSET
    • MODACC_PRE_FFT_ERROR_ESTIMATION_INTERVAL
    • MODACC_SYMBOL_CLOCK_ERROR_ESTIMATION_ENABLED
    • MODACC_TIMING_TRACKING_ENABLED
    • MODACC_PHASE_TRACKING_ENABLED
    • MODACC_RESULTS_MAXIMUM_PEAK_FREQUENCY_ERROR

Why should this Pull Request be merged?

AB#2299243

What testing has been done?

Build passes.

@WesleyTangNationalInstruments WesleyTangNationalInstruments marked this pull request as ready for review February 15, 2023 18:24
@WesleyTangNationalInstruments WesleyTangNationalInstruments changed the title Updating nirfmxlte to 23.0 with generated files from scrapigen Update niRFmxLTE to 23.0 Feb 15, 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.

Everything looks good besides the callout for the breaking change. Want to see what Alex thinks and the resolution to that before approving.

@WesleyTangNationalInstruments WesleyTangNationalInstruments deleted the users/wetang/UpdateRFmxLTETo23_0 branch February 22, 2023 16:58
@reckenro reckenro added the source-breaking Change to proto file that will break client if they update label Mar 13, 2023
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.

3 participants