codecov[bot]
commented
2 years ago Codecov Report
Merging #25 (c369713) into lockwo_updates (73535cc) will increase coverage by
0.10%. The diff coverage is93.91%.
@@ Coverage Diff @@
## lockwo_updates #25 +/- ##
==================================================
+ Coverage 92.51% 92.61% +0.10%
==================================================
Files 12 12
Lines 1243 1382 +139
==================================================
+ Hits 1150 1280 +130
- Misses 93 102 +9 | Impacted Files | Coverage Δ | |
|---|---|---|
| src/gates.jl | 86.66% <77.27%> (-1.53%) |
:arrow_down: |
| src/data.jl | 94.58% <96.29%> (+0.40%) |
:arrow_up: |
| src/constraints.jl | 97.05% <100.00%> (ø) |
|
| src/log.jl | 87.42% <100.00%> (+1.21%) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update 73535cc...c369713. Read the comment docs.
harshangrjn
Support is added for CRX, CRY, CRZ and CU3 gates. The naming follows with the other 2 qubit gates and the specificity of rotation with the single qubit gates. All tests pass locally. I added a few tests, but coverage will definitely be short of what is needed (I wanted to share it early on to enable contributions to tests/allow for any other changes). Although there shouldn't be any breaking changes, I am pulling it to the lockwo_updates branch.