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

acquisition: populate gnss_synchro.fs with acquisition sample rate #680

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

Conversation

vladisslav2011
Copy link
Contributor

@vladisslav2011 vladisslav2011 commented Oct 28, 2022

This makes it possible to use GNSS-SDR-Monitor to monitor acquisition
process too.
Set AcquisitionMonitor.udp_port to the same value as Monitor.udp_port to
see acquisition results.
The acquisitions will be shown with a red telemetry mark and no tracking
data.
gnss-sdr-monitor

@jwmelto
Copy link
Contributor

jwmelto commented Oct 28, 2022

I'm confused what this change does, besides fill in an uninitialized variable (which is a good thing).

@vladisslav2011
Copy link
Contributor Author

The GNSS-SDR-Monitor checks for initialization of fs member to confirm data validity https://github.com/vladisslav2011/gnss-sdr-monitor/blob/master/src/channel_table_model.cpp#L289
Initializing fs makes this check pass and the GNSS-SDR-Monitor starts showing both acquisition results and early tracking (before ephemeris reception and decoding).

@jwmelto
Copy link
Contributor

jwmelto commented Nov 3, 2022

Looks good to me

This makes it possible to use GNSS-SDR-Monitor to monitor acquisition
process too.
Set AcquisitionMonitor.udp_port to the same value as Monitor.udp_port to
see acquisition results.
The acquisitions will be shown with a red telemetry mark and no tracking
data.

Signed-off-by: Vladslav P <vladisslav2011@gmail.com>
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.

None yet

2 participants