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

Major update from UPV #13

Open
wants to merge 1 commit into
base: development
Choose a base branch
from
Open

Conversation

jsroldan
Copy link

@jsroldan jsroldan commented Oct 1, 2024

Added new dinamic graph for BLER and SNR.
Added control pannel to modify the receiver parameters. Added brief description to options and values.

Added new dinamic graph for BLER and SNR.
Added control pannel to modify the receiver parameters.
Added brief description to options and values.
@dsilhavy dsilhavy added this to the 0.2.0 milestone Oct 8, 2024
@dsilhavy dsilhavy self-requested a review October 8, 2024 07:11
@dsilhavy
Copy link
Contributor

dsilhavy commented Oct 8, 2024

@alessandro-lucco: Fyi, this PR includes the changes presented by @jsroldan in the last call. If you find the time, it would be nice if you can run some tests on your side as well. We are currently testing against sample recordings: https://5g-mag.github.io/Getting-Started/pages/lte-based-5g-broadcast/additional/sample-files.html

@@ -1,6 +1,6 @@
{
"name": "rt-wui",
"version": "0.0.11",
"version": "0.1.1",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Increase version number to 0.2.0 in package.json and run npm install again. This should update the package-lock.json as well.

function plot_graph(canvas_id, status_rx, snr, snr_avg, bler) {
const canvas = document.getElementById(canvas_id);
const ctx = canvas.getContext("2d");
//canvas.width = canvas.parentElement.clientWidth;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove commented lines

ctx.lineWidth = 1;
ctx.strokeStyle = "#00CC00";
ctx.fillStyle = "#00CC00";
// ctx.clearRect( 0, 0, canvas.width, canvas.height );
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove commented lines

}
ctx.strokeStyle = "#999999";
ctx.fillStyle = "#999999";
//ctx.moveTo(0, y);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove commented lines

ctx.fillRect(canvas.width- right_margin,
y - 1, // Instantaneous snr
2,2);
//ctx.stroke();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove commented lines


if (!window.started && d["state"].localeCompare("Not running") ) {
window.started = 1;
//clear_all();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove commented lines

});

load_ce_values("sdr-carriers");

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove empty lines

exports.status = async(function*(req, res) {
proxy_call(req, res, "/modem-api/status");
});
exports.ce_values = async(function*(req, res) {
proxy_call(req, res, "/modem-api/ce_values");
});


Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove empty lines

router.get('/api/modem/status', modem_api.status);
router.get('/api/modem/ce_values', modem_api.ce_values);


Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove empty lines

@@ -12,7 +12,8 @@
<body class="" data-rp-api="<%= global.rp_api %>">
<div class="wrapper">
<%- include('_header.ejs') %>
<div class="container">
<!--div class="container"><!-->
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove commented lines

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

Successfully merging this pull request may close these issues.

2 participants