-
Notifications
You must be signed in to change notification settings - Fork 201
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
analyzer: support large data #651
Comments
So a self-synchronizing variable-length encoding? Something like this:
where |
It would be nice if DMA (input and output), Analyzer, and DRTIO all three could use the same (or at least very similar) serialization formats. @whitequark |
There cannot be a lot in common, except for trivialities (e.g. the order of those few fields that are common), and more importantly for the way the variable length is communicated. |
@jordens yes, |
And the analyzer is the only one that requires the variable length encoding to be self-synchronizing, so it's not clear that even that should be shared. |
Well, it is definitely less error-prone to have one variable-length encoder than two, on both sides. |
The analyzer record format doesn't have enough data bits for SAWG events.
Fixing this requires defining a new record format that ideally:
Then it needs to be implemented in gateware and in
artiq_coreanalyzer
.The text was updated successfully, but these errors were encountered: