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

[change] Added the name of the metric to the label #311 #621

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Priyanshu-Batham
Copy link

Previously the signal charts didn't have a metric name so the following changes were made.
Signal Strength -> Signal Strength (RSSI)
Signal Power -> Signal Power (RSRP)
Signal Quality -> Signal Quality (RSRQ)
Signal to Noise Ratio -> Signal to Noise Ratio (SNR)

Fixes #311

Previously the signal charts didn't have a metric name so the following changes were made.
Signal Strength -> Signal Strength (RSSI)
Signal Power -> Signal Power (RSRP)
Signal Quality -> Signal Quality (RSRQ)
Signal to Noise Ratio -> Signal to Noise Ratio (SNR)

Fixes openwisp#311
@Priyanshu-Batham
Copy link
Author

@pandafy This time I have run the run-qa-checks with all checks successful and hoping no problems occur.

@pandafy pandafy self-requested a review December 2, 2024 11:59
@pandafy
Copy link
Member

pandafy commented Dec 19, 2024

@Priyanshu-Batham the QA checks are still failing. Do you want to continue working on this PR?

@Priyanshu-Batham
Copy link
Author

@pandafy yes sir, am currently out of station and will be back on Sunday. I'll fix it ASAP!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Needs review
Development

Successfully merging this pull request may close these issues.

Signal charts: add the name of the metric to the label
2 participants