-
Notifications
You must be signed in to change notification settings - Fork 647
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
do_tune_freq_results_message() produces confusing verbose #27
Comments
Hi @guruofquality - We added those as extra status messages in an attempt to provide useful information to users. We have had a few users comment that they are confusing, though, so we will revisit how / when they are used. I'm going to pull this in as an internal bug so that we can track it properly. Thanks! Cheers, |
Closed
This was referenced Nov 12, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
For an example, I have added some extra prints to the following function to see what is going on:
https://github.com/EttusResearch/uhd/blob/master/host/lib/usrp/multi_usrp.cpp#L72
First problem, the prints from "The RF LO does not support the requested frequency" does not show any precision about the frequencies in question. So the message just looks kind of confusing.
Second problem. It seems that this message is triggered because rf_lo_tune_success is false, which is caused by a tune error of a fraction of a Hz. I thought that the expected behaviour of automatic tuning was to tune the frontend, and then compensate for the error with the DSP. So I wouldnt expect that a small frontend precision error would cause this print.
Anyway, nothing is behaving incorrectly. We just did not expect the verbose, and found it confusing.
The text was updated successfully, but these errors were encountered: