peachnuts / ha Goto Github PK

Take care of the note here.

The best solution would be to write another "iterative_mapping_algorithm" function with the appropriate defaults functions as parameters. I still need to think about this.

The solidpy dependency is no longer used.

If the input quantum circuit does not contain exactly the number of qubits of the hardware, we might end up having some index errors raised by the mapping algorithm.
A solution is to expand the quantum circuit at the beginning of the mapping procedure.

The package name "qubit_mapping_optimizer" is too long and impractical.
Propositions:

• ha_mapping
• hawaremap
• haware

Note that Python package names should be lower-case, so "HA_mapping" is not a candidate.

The name is too long and not descriptive. I propose

• ha_mapping

We should change the license at the start of each Python file to include Siyuan name.
Change

# Copyright TOTAL / CERFACS / LIRMM (02/2020)
# Contributor: Adrien Suau (<[email protected]>
#                           <[email protected]>)

to

# Copyright TOTAL / CERFACS / LIRMM (02/2020)
# Contributors: Adrien Suau (<[email protected]>
#                            <[email protected]>)
#               Siyuan Niu  (<[email protected]>)

Warning: the month in the first line of the text snippet might change across files.

The default behaviour when we call the iterative_mapping_algorithm is to use SABRE algorithm with Bridge gates.
The default behaviour should be our algorithm with the right coefficients.