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

Rev 1.0 awkward rounding voltage #23

Open
jfrazao opened this issue Feb 11, 2024 · 0 comments
Open

Rev 1.0 awkward rounding voltage #23

jfrazao opened this issue Feb 11, 2024 · 0 comments
Assignees
Labels
bug Something isn't working documentation Improvements or additions to documentation fix Pull request that fixes an issue proposal Request for a new feature

Comments

@jfrazao
Copy link
Collaborator

jfrazao commented Feb 11, 2024

Rev 1 hardware does a Awkward voltage quantization rounding (from bonsai to pulse pal) that doesn't follow any of the comum approaches (flooring or ceiling to the closest quantum, or to the biggest, or smallest, or even).
Example rounding bonsai to hw:
5.2v to 5.16
5.21 to 5.16
5.22 to 5.23

This issue may be related to how the scale is defined on this version of the HW.

pulse pal rev1 byte to voltage correspondence:
000 to -10.0v
127 to 0.0v
255 to +9.92v
256 to +10.0v

@jfrazao jfrazao added bug Something isn't working documentation Improvements or additions to documentation fix Pull request that fixes an issue proposal Request for a new feature labels Feb 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation fix Pull request that fixes an issue proposal Request for a new feature
Projects
None yet
Development

No branches or pull requests

3 participants