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

PrawnBlaster binary programming #112

Merged

Conversation

carterturn
Copy link
Contributor

The labscript-devices side of PrawnBlaster PR #25.

Adds a fast binary mode for reprogramming the entire sequence, as well as heuristics for deciding when to use binary mode instead of incremental programming.

Verification testing has been performed, but performance testing is incomplete.

@dihm
Copy link
Contributor

dihm commented Feb 7, 2024

Just a note here: the docs build is failing because you are branched off master one commit behind the current HEAD (which fixes this issue). It's not a blocker, but if you make docs changes we'll want to rebase so they can get built correctly.

@philipstarkey
Copy link
Member

Could you add something that falls back to the original behaviour for old firmware versions? You should be able to query the firmware version from the PrawnBlaster itself (over the serial connection). Anything less than v1.1.0 won't support the binary mode (which will be the earliest firmware version we'll be targeting with labscript-suite/PrawnBlaster#25)

@carterturn
Copy link
Contributor Author

There is a better way to do the version number comparisons that I will implement soon.

@dihm dihm merged commit fa8b457 into labscript-suite:master May 23, 2024
1 check passed
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

Successfully merging this pull request may close these issues.

None yet

3 participants