Closed eliarbel closed 5 days ago
Hi @eliarbel, I have opened a PR with placeholders for all missing standard gates in rust so that we don't have clashing IDs when working on gates in parallel. Do you think it would be helpful? https://github.com/Qiskit/qiskit/pull/12646
Hi @eliarbel, I have opened a PR with placeholders for all missing standard gates in rust so that we don't have clashing IDs when working on gates in parallel. Do you think it would be helpful? #12646
Yeap, I've used it. Thanks @ElePT !
One or more of the following people are relevant to this code:
@Cryoris
@Qiskit/terra-core
@ajavadia
@kevinhartman
@mtreinish
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
crates/circuit/src/operations.rs | 15 | 16 | 93.75% | ||
<!-- | Total: | 17 | 18 | 94.44% | --> |
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
crates/qasm2/src/lex.rs | 4 | 92.88% | ||
<!-- | Total: | 4 | --> |
Totals | |
---|---|
Change from base Build 9650973845: | 0.01% |
Covered Lines: | 63795 |
Relevant Lines: | 71070 |
Can you update the
QuantumCircuit.dcx()
method to directly append the rust gate too? You can look at cx or h for an example of how to do this.
Good catch! done
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
crates/circuit/src/operations.rs | 15 | 16 | 93.75% | ||
<!-- | Total: | 18 | 19 | 94.74% | --> |
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
crates/qasm2/src/lex.rs | 4 | 91.6% | ||
crates/qasm2/src/parse.rs | 12 | 97.15% | ||
<!-- | Total: | 16 | --> |
Totals | |
---|---|
Change from base Build 9650973845: | -0.01% |
Covered Lines: | 63777 |
Relevant Lines: | 71069 |
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
crates/circuit/src/operations.rs | 15 | 16 | 93.75% | ||
<!-- | Total: | 18 | 19 | 94.74% | --> |
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
crates/qasm2/src/expr.rs | 1 | 94.02% | ||
crates/qasm2/src/lex.rs | 2 | 92.88% | ||
qiskit/synthesis/discrete_basis/solovay_kitaev.py | 4 | 94.74% | ||
qiskit/providers/fake_provider/generic_backend_v2.py | 10 | 94.71% | ||
crates/qasm2/src/parse.rs | 18 | 96.69% | ||
<!-- | Total: | 35 | --> |
Totals | |
---|---|
Change from base Build 9650973845: | -0.001% |
Covered Lines: | 63794 |
Relevant Lines: | 71077 |
Summary
As the name suggests.
Details and comments
Addresses #12566