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.

Sign up for GitHub

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

Jump to bottom

RTLSDR_PPM value not picked up #55

Closed
klara31 opened this issue Oct 28, 2020 · 4 comments
Closed

RTLSDR_PPM value not picked up #55

klara31 opened this issue Oct 28, 2020 · 4 comments
Assignees
@mikenye

Comments

@klara31
Copy link

klara31 commented Oct 28, 2020 •
edited

I have set the RTLSDR_PPM value in de docker.yml file. If I check the code, in etc/services.d/dump1090/run there is a line: RTLSDR_PPM="$(piaware-config -show rtlsdr-ppm)"
But RTLSDR_PPM is not set to piaware-config. Hence the value is always 0.

Or am I missing something?
@mikenye mikenye self-assigned this Oct 28, 2020
@mikenye
Copy link
Member

mikenye commented Oct 29, 2020

You're correct. Oops. I'll fix this now. Sorry for the inconvenience.

mikenye added a commit that referenced this issue Oct 29, 2020
@mikenye
attempted fix for issue #55
09d67b8
mikenye added a commit that referenced this issue Oct 29, 2020
@mikenye
attempted fix for issue #55 (#56)
38dfcf5
@mikenye
Copy link
Member

mikenye commented Oct 29, 2020

@klara31 I've implemented an attempted fix. Can you pull the :latest image and let me know if that solves your problem?

Thanks.

@klara31
Copy link
Author

klara31 commented Oct 29, 2020

Yes, PPM value now is used correctly. Thanks again!

@klara31 klara31 closed this as completed Oct 29, 2020
@mikenye
Copy link
Member

mikenye commented Oct 29, 2020

You're most welcome. Thanks for reporting the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mikenye mikenye

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@klara31 @mikenye