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.

Sign up for GitHub

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

Jump to bottom

misc. bugfixes #267

Merged
merged 7 commits into from
Jun 6, 2024
Merged

misc. bugfixes #267

merged 7 commits into from
Jun 6, 2024

Conversation

meeg
Copy link
Collaborator

@meeg meeg commented Jun 6, 2024

  • mux readouts suffer from "IQ offsets" that vary depending on the filter-bank channel, which was not accounted for by the workaround in workaround for readouts with IQ offsets #254. This would mostly only bite you if your readout frequency was precisely at a channel's center frequency, but now that case is also covered.
  • if you used pulse() with a list of channels, and set t='auto', your pulse times would be weird (the t parameter would get overwritten by the computed pulse time for the first channel, and then the other channels would use that value)
  • the QickConfig printout was just broken for firmwares with constant-IQ generators, since support for tproc mux #209.

@meeg meeg merged commit aea95e3 into openquantumhardware:main Jun 6, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@meeg @qickbot