[QHC-846] Solve Layout problems & Update qibo version transpiler

[GuillermoAbadLopez]

TODOs in this and next PRs:

Solve more buggs about final layout/compute it yourself? 🔥🔥🔥

• Check if Update qibo version solves it?
• Mirar reported layout para el caso que no cambia
• Make that reported layout works always

Improve readability of layout 🔥🔥🔥:

• Make more clear what are keys/values in the dict
• State clearly the final ordering of qubits (@Ameer)
• Maybe log both init_layout (placement) & final_layout (placement+routing)

[linear]

QHC-846 Solving bugs/problems from `Routing` from demo:

[pullpo-for-slack]

🔗Pullpo.io Slack channel

[pullpo-for-slack]

AI Analysis

Review:

The changes in the test files seem to focus on improving fixture organization and ensuring input validation for the QDAC-II class, which enhances the test coverage and reliability. The modifications in circuit_transpiler.py and qdevil_qdac2.py to use integer keys and define ramping rate ranges respectively show a clear effort to enhance code clarity and maintainability. Overall, these changes appear to improve the codebase by enforcing stricter validation and consistency.

Detailed file changes

(dropdown):

In tests/instruments/qdevil/test_qdevil_qdac2.py:

• Added a new fixture qdac_out_range for a dummy QDAC-II instance with specific parameters.
• Updated the get_square_waveform fixture to fix spacing in the return statement.
• Added test methods to check input ranges for runcard and set_parameter methods for the QDAC.

In circuit_transpiler.py:

• Updated the return type hint from dict[str, int] to dict[int, int].
• Changed the final_layout dictionary keys from strings to integers.

In qdevil_qdac2.py:

• Added class attributes _MIN_RAMPING_RATE and _MAX_RAMPING_RATE to define the ramping rate range.
• Added input validation to check if the ramp rate is within the defined range (0.01 V/s to 2e7 V/s) before setting it.

In tests/digital/test_circuit_router.py:

• Changed the test_layout and test_bad_layout dictionaries to use integer keys instead of strings.
• Updated the final_layout assertions in test_route_doesnt_affect_already_routed_circuit and test_route_affects_non_routed_circuit functions to use integer keys instead of strings.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.29%. Comparing base (4573d20) to head (2073614).

Additional details and impacted files

@@                         Coverage Diff                          @@
##           adding_optimize_transpiled_gates     #869      +/-   ##
====================================================================
- Coverage                             97.30%   97.29%   -0.02%     
====================================================================
  Files                                   231      231              
  Lines                                  8274     8266       -8     
====================================================================
- Hits                                   8051     8042       -9     
- Misses                                  223      224       +1     

Flag	Coverage Δ
unittests	97.29% <100.00%> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.