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

Bandwidth setting AUTO does not "set the bandwidth to the largest value below the sample rate" #15

Open
kirat68 opened this issue Nov 5, 2021 · 5 comments

Comments

@kirat68
Copy link

kirat68 commented Nov 5, 2021

Hello Franco

Would be nice if you could check this issue. Setting Bandwidth to AUTO takes the lowest value of the allowed bandwidths and NOT "the largest value below the sample rate".

Test settings:
Sample rate = 5E6
Bandwidth = Auto

Result: Bandwidth = 2E5
Expected: Bandwidth = 5E6

Otherwise working very good. Thank you very much !!!

@fventuri
Copy link
Owner

fventuri commented Nov 6, 2021

@kirat68 - thanks for pointing out this problem.

It should now be fixed in the master branch.

Please also note that 4e6 is not one of the bandwidth supported by SDRplay RSPs; you can find the list of supported values in the SDRplay API Specification guide at page 12: 200k, 300k, 600k, 1536k, 5M, 6M, 7M, and 8M.

Franco

@kirat68
Copy link
Author

kirat68 commented Nov 6, 2021

Yes yes just realized that 4E6 was not the correct value and corrected my comment before seeing yours :-)

@kirat68
Copy link
Author

kirat68 commented Nov 6, 2021

Expected value should be in this case 5E6. Specification should be:
"set the bandwidth to the largest value below *or equal to the sample rate"
Am I right ?

@fventuri
Copy link
Owner

fventuri commented Nov 6, 2021

@kirat68 - please note that in the same commit yesterday (c74162f), I changed that wording to "compatible with".

Franco

@kirat68
Copy link
Author

kirat68 commented Nov 9, 2021

Perfect !!!

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

No branches or pull requests

2 participants