Feature/auto wiring

[deanpoulos]

(Work in progress)

Overview

Added a feature for auto-assigning channels for a collection of Quantum elements given a specific QM instrument setup.

graph TD
    A(Instruments) --> D["allocate_wiring()"]
    C(Connectivity) --> D["allocate_wiring()"]
    subgraph Wirer
        D["allocate_wiring()"]
    end

Loading

Features

Supports the following:

• assignment of channels to any combination of MW-FEM/LF-FEM/Octave/OPX+.
• any mapping of N resonator lines to M qubits
• any mapping of N fems to OPX1000 chassis slots
• overflowing of assignment to multiple slots, chassis, modules, octaves, etc.
• any combination of resonator, flux line, coupler line for each qubit.
• hardcoding of con/slot/port for particular lines
• basic visualization of a single OPX1000

Safety

It has some simple validation:

• can't mix OPX+ and OPX1000
• can't define two FEMs on the same slot on the same chassis

Examples

    instruments = Instruments()
    instruments.add_lf_fem(con=1, slots=1)
    instruments.add_mw_fem(con=1, slots=2)

    qubits = [1, 2, 3, 4, 5, 6]
    connectivity = Connectivity()
    connectivity.add_resonator_line(qubits=qubits)
    connectivity.add_qubit_drive_lines(qubits=qubits)
    connectivity.add_qubit_flux_lines(qubits=qubits)

    allocate_wiring(connectivity, instruments)

    if visualize:
        visualize_chassis(connectivity.elements, instruments.available_channels)

[github-actions]

Unit Test Results

412 tests   403 ✔️  45s ⏱️
    1 suites      9 💤
    1 files        0 ❌

Results for commit 8e8945d.

♻️ This comment has been updated with latest results.

[deanpoulos]

@TheoLaudatQM this is ready to for a final review and merge!

(the black formatting is failing, but I can't reproduce it locally).

[yomach]

@TheoLaudatQM this is ready to for a final review and merge!

(the black formatting is failing, but I can't reproduce it locally).

Do poetry install to install the same version of black.

[TheoLaudatQM]

It is truly amazing, great job Dean!!

[TheoLaudatQM]

To me it is ready to merge, but I am not sure how to fix the linting issues raised by the test... (black is okay)

[yomach]

To me it is ready to merge, but I am not sure how to fix the linting issues raised by the test... (black is okay)

@TheoLaudatQM @deanpoulos - If you're stuck, schedule 30 minutes with me.

[TheoLaudatQM]

Essentially flake8 doesn't want the init.py files so I could just delete them but maybe they are here for a reason so let's wait for @deanpoulos

[TheoLaudatQM]

[celebrate] Theo Laudat reacted to your message:

…

________________________________ From: Dean Poulos ***@***.***> Sent: Tuesday, October 22, 2024 2:05:47 PM To: qua-platform/py-qua-tools ***@***.***> Cc: Theo Laudat ***@***.***>; Mention ***@***.***> Subject: Re: [qua-platform/py-qua-tools] Feature/auto wiring (PR #229) Merged #229<#229> into main. — Reply to this email directly, view it on GitHub<#229 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AXR3HVSE5EB7IW3NSSFELFTZ4ZLTXAVCNFSM6AAAAABNEBGCBKVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJUG44DMMRRGUYDANQ>. You are receiving this because you were mentioned.Message ID: ***@***.***>