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

Couplers Breaking Changes #587

Closed
aorgazf opened this issue Oct 26, 2023 · 2 comments · Fixed by #591
Closed

Couplers Breaking Changes #587

aorgazf opened this issue Oct 26, 2023 · 2 comments · Fixed by #591
Assignees
Labels
bug Something isn't working

Comments

@aorgazf
Copy link
Member

aorgazf commented Oct 26, 2023

Dear all,
chevron routine throws the error shown below when trying to run it on qw5q_gold.
It appears the changes introduced by @Jacfomg to support QPU architectures with couplers were not tested on platforms that do not have couplers.

Traceback (most recent call last):
  File "/nfs/users/alvaro.orgaz/.envs/november/bin/qq", line 6, in <module>
    sys.exit(command())
  File "/nfs/users/alvaro.orgaz/.envs/november/lib/python3.10/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/nfs/users/alvaro.orgaz/.envs/november/lib/python3.10/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "/nfs/users/alvaro.orgaz/.envs/november/lib/python3.10/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/nfs/users/alvaro.orgaz/.envs/november/lib/python3.10/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/nfs/users/alvaro.orgaz/.envs/november/lib/python3.10/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/nfs/users/alvaro.orgaz/qibocal/src/qibocal/cli/_base.py", line 53, in auto
    autocalibrate(card, folder, force, update)
  File "/nfs/users/alvaro.orgaz/qibocal/src/qibocal/cli/autocalibration.py", line 61, in autocalibrate
    for task_uid in executor.run(mode=ExecutionMode.autocalibration):
  File "/nfs/users/alvaro.orgaz/qibocal/src/qibocal/auto/execute.py", line 136, in run
    completed = task.run(
  File "/nfs/users/alvaro.orgaz/qibocal/src/qibocal/auto/task.py", line 146, in run
    completed.data, completed.data_time = operation.acquisition(
  File "/nfs/users/alvaro.orgaz/qibocal/src/qibocal/auto/operation.py", line 41, in wrapper
    out = func(*args, **kwds)
  File "/nfs/users/alvaro.orgaz/qibocal/src/qibocal/protocols/characterization/two_qubit_interaction/chevron.py", line 128, in _aquisition
    cz.coupler_pulses(platform.pairs[tuple(sorted(ordered_pair))].coupler.name)
AttributeError: 'NoneType' object has no attribute 'name'
@aorgazf aorgazf added the bug Something isn't working label Oct 26, 2023
@Jacfomg
Copy link
Contributor

Jacfomg commented Oct 26, 2023

Already fixed on qiboteam/qibolab#623, it's waiting for review to merge.

@andrea-pasquale
Copy link
Contributor

Already fixed on qiboteam/qibolab#623, it's waiting for review to merge.

I tried this branch and it still does not work on qw5q_gold

@andrea-pasquale andrea-pasquale linked a pull request Oct 27, 2023 that will close this issue
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants