-
-
Notifications
You must be signed in to change notification settings - Fork 22
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
Significantly less information decoded compared to other libraries #296
Comments
Is this also using https://github.com/rsadsb/dump1090_rs ? |
EDIT: To clarify, I used the fa dump1090 for this comparison for both view1090 and adsb_deku radar. However I also tried dump1090_rs and the result is similar. When comparing I always used the same dump implementation for both clients. This behavior is observed irrespective of the range filter in adsb_deku btw. |
Ok, just making sure it's an issue with adsb_deku. |
Have you checked the logs? do you see any You can do this by adding |
The only thing I can think of is that you are getting TisB message that I wasn't adding to the positions of the aircraft, pushed an MR to do that here: #301. This is untested. |
I think I figured this out. I was having the same issue, even after using linked branch. After looking at the logs it looks like some airplane positions are failing the range check.
You may notice the latitude is more than +-90 which is messing up the range check. We can do the math and figure out that the reported lat is "correct" it's just not in the expected range:
Might be something do to with lat/long parsing? I'm not sure this should be a reported value. |
@YurBoiRene I don't have a way of testing this, but causing mod to be positive might be the fix? b980c92 (pushed it to that MR) |
I moved locations so I also cannot test your exact changes but I made a similar fix that fixed it. We can probably make some tests. |
I just did some testing and this does look a LOT better here. Even with a very compromised antenna I now get an almost complete picture around me. It's really a massive improvement. |
Thanks for the data, this should be fixed in https://github.com/rsadsb/adsb_deku/releases/tag/v2024.09.02 🎊 |
Hi,
I am noticing a significant difference in the amount of information that is being decoded. For a lot of aircraft the lat/lon is never populated whereas when using view1090 from dump1090_fa, I can see a lot more. I have played around with the distance filter and set the lat/lon as expected, but here are some comparisons:
view1090:
adsb_deku radar output:
Is this expected behavior? I compared adsb_deku with several different implementations, including open source and certified commercial ones, and the results from adsb_deku are consistently worse despite using the same antenna and SDR. I also tried various different antennae, with and without filter and using a hackrf instead of an RTLSDR dongle, but I only see a fraction of the traffic being decoded.
The text was updated successfully, but these errors were encountered: