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

Fix wrong satellite miss detection #3

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

lars18th
Copy link
Contributor

In some NIT tables the orbital position has a negative value. This patch solves it and it adds more useful information in the log message "WRONG SATELLITE".

- Improve the log message "WRONG SATELLITE" including the two values comparted.
- Adds to the check that one value will not be the other with an opposite sign (detected in some NIT tables).
@lars18th
Copy link
Contributor Author

This is related to wirbel-at-vdr-portal/w_scan_cpp#22 (comment)

@wirbel-at-vdr-portal
Copy link
Owner

No, this one is a wrong change.

Instead, it should be fixed at the source of the problem.
Send me a full log with four time '-v' for the problematic satellite. Will be a few megs.
I need to the read the nit tables send on this sat from the log.

@wirbel-at-vdr-portal
Copy link
Owner

wirbel-at-vdr-portal commented Feb 11, 2024

May be you could check

scanfilter.cpp:840

-- if (!sd->getWestEastFlag())
-- transponder->OrbitalPos *= -1;
++ transponder->West = west_flag;

@lars18th
Copy link
Contributor Author

Hi @wirbel-at-vdr-portal ,

May be you could check

Yes. I was thinking about something similar today. So please be on hold. I'll update this PR.

Remove the check of diferent sign
Pass the west_flag
@lars18th
Copy link
Contributor Author

Hi @wirbel-at-vdr-portal ,

I've updated the PR. With your suggestion I don't have detected any trouble scanning 30W. I suggest to test it.

Regards.

@wirbel-at-vdr-portal
Copy link
Owner

So, the problem as such is gone. We're close to merge it, solution is nice. Just finishing it.

However, i would like to debug print (only) the wrong satellite's position, something like S30W0 or similar. Just in case, this one pops up later again and i don't understand it reading megabytes of looong log files.

Any idea, or you want me to step in (may take some time unfortunally, beeing busy for a few weeks).

@lars18th
Copy link
Contributor Author

Hi @wirbel-at-vdr-portal ,

Please take as much time as you need to test and merge this correction. As the solution is uncomplicated, it's preferable to wait until everything is properly tested. I always prefer a good, well-tested software rather than making quick changes.

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