Qiskit / qiskit-fermions / 30087484454

98%

Build Type

push

github

Committed by web-flow

Commit Message

refactor(mappers): rename concrete JW to `fermion_jordan_wigner` (#209)

The Jordan-Wigner mapper was the only one named after the transform rather
than its source operator type. Because JW is also definable for other operator
types (e.g. `MajoranaOperator`, mapping to the same qubit target), the bare
name would collide once such variants exist — and Rust and C cannot overload.

Rename the fermion-specific implementation to encode its source type
(`fermion_jordan_wigner` in Rust/Python; `qf_ferm_op_jordan_wigner` in C, which
also restores the `qf_ferm_op_*` prefix convention the bare name broke). The
`jordan_wigner` module keeps its name so future `<type>_jordan_wigner`
implementations live alongside.

Add a pure-Python `jordan_wigner(operator, num_qubits)` dispatcher in
`mappers.library` that delegates on the concrete operator type (today:
`FermionOperator`; else `TypeError`). Its `(OperatorTrait, int) ->
SparseObservable` signature matches the synthesis `MapperFunction`, so the
transpiler preset consumes it unchanged and gains future types for free.

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>

Coverage Stats

17 of 17 new or added lines in 3 files covered. (100.0%)

5 existing lines in 2 files now uncovered.

10082 of 10290 relevant lines covered (97.98%)

322.03 hits per line