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

Remove 15 kHz from selectable flux ramp rates #712

Open
agustiner opened this issue Mar 9, 2022 · 0 comments
Open

Remove 15 kHz from selectable flux ramp rates #712

agustiner opened this issue Mar 9, 2022 · 0 comments
Labels
bug Something isn't working

Comments

@agustiner
Copy link
Member

agustiner commented Mar 9, 2022

15 kHz does not divide 25.6 MHz and 122.88 MHz (?), which creates very sad looking flux ramp signals. All other frequencies, 1, 2, 3, 4, 5, 8, 10, 12 kHz are valid flux ramp rates from my testing so far. Right now users can apparently select any frequency desired with integer value.

Screen Shot 2022-03-09 at 1 38 08 PM

Easy fix, remove 15 kHz, smarter fix, check the given flux ramp rate divides 25.6 Mhz and 122.88 MHz and then send it to the pyrogue layer.

@agustiner agustiner added the bug Something isn't working label Mar 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant