Closed gwwatkin closed 2 years ago
Merging #248 (b4bbbc1) into dev (a9fd1f2) will increase coverage by
0.44%
. The diff coverage is88.09%
.
@@ Coverage Diff @@
## dev #248 +/- ##
==========================================
+ Coverage 67.88% 68.32% +0.44%
==========================================
Files 22 23 +1
Lines 1893 1926 +33
==========================================
+ Hits 1285 1316 +31
- Misses 608 610 +2
Impacted Files | Coverage Δ | |
---|---|---|
...c/pipeline/lattice_surgery_compilation_pipeline.py | 0.00% <0.00%> (ø) |
|
...cc/resource_estimation/llops_resource_estimator.py | 100.00% <100.00%> (ø) |
|
...cc/resource_estimation/slice_resource_estimator.py | 100.00% <100.00%> (ø) |
|
src/lsqecc/utils.py | 96.15% <100.00%> (+0.50%) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update a9fd1f2...b4bbbc1. Read the comment docs.
pre-commit.ci autofix
Changes to make the resource estimator work from Logical Lattice Ops (similar to topological assembly).
The resource estimation from slices was incorrect. This is a more accurate use of Qentiana. Been using it in the benchmarks and getting reasonable results. Also went over it with @alexandrupaler.