Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Register
  • Sign in
  • slapos slapos
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Merge requests 129
    • Merge requests 129
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • nexedinexedi
  • slaposslapos
  • Merge requests
  • !1478

software/ors-amarisoft: Enable check-rx-saturated promise for Lopcomm Radio Unit as well

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Kirill Smelkov requested to merge kirr/slapos:y/rxsat-all into master Nov 21, 2023
  • Overview 2
  • Commits 1
  • Pipelines 0
  • Changes 2

The code had check-rx-saturated under if not ru == "lopcomm", but checking RX signal for saturation is implemented on eNB side, not by RU itself, and is RU-independent because eNB only receives raw samples from RU and decides on its own whether those samples are higher than saturation margin or not.

As an extra proof that checking for RX saturation is meaningful for Lopcomm too here is how samples.rx data can look like in the case of 2 Lopcomm Radio Units:

root@callbox-005:/srv/slapgrid/slappart16/var/log# cat amarisoft-stats.json.log |jq .data.samples.rx
[
  {
    "rms": -382.3080749511719,
    "max": -382.3080749511719,
    "sat": 0,
    "count": 1221181440,
    "rms_dbm": -382.3080749511719
  },
  {
    "rms": -382.3080749511719,
    "max": -382.3080749511719,
    "sat": 0,
    "count": 1221179392,
    "rms_dbm": -382.3080749511719
  }
]
...

i.e. all the data, that check-rx-saturated promise works on, is there.

/cc @jhuge, @lu.xu, @xavier_thompson, @Daetalus

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: y/rxsat-all
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7