qblox backend: configuration update for QCM#292
qblox backend: configuration update for QCM#292RJQblox wants to merge 1 commit intooqc-community:mainfrom
Conversation
|
I have read the Contributor License Agreement and I hereby accept the Terms. You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot. |
helmaazouz-oqc
left a comment
There was a problem hiding this comment.
Thank you @RJQblox for the suggested changes ! When it comes to configuration, QBlox have fine-grained APIs for numerous settings, we've grouped NCO related ones exactly to avoid fine-grained methods. But you still have several subtypes of QBloxConfigHelper for the purposes you mentioned, I encourage you to override nco there with the sequence of changes you like (excluding prop delay)
This fix is needed since the NCO propagation delay is only available for readout modules (QRM), but not control modules (QCM). Both the
src/qat/backend/qbloxandsrc/purr/cat/backends/qbloxfolder have been updated, is this correct?