Closed zmorrell closed 2 years ago
Merging #19 (3c7f1ee) into main (37c5ba8) will increase coverage by
2.26%
. The diff coverage is90.00%
.
@@ Coverage Diff @@
## main #19 +/- ##
==========================================
+ Coverage 94.58% 96.84% +2.26%
==========================================
Files 5 5
Lines 443 444 +1
==========================================
+ Hits 419 430 +11
+ Misses 24 14 -10
Impacted Files | Coverage Δ | |
---|---|---|
src/simulate.jl | 94.59% <88.88%> (+7.45%) |
:arrow_up: |
src/simulate_de.jl | 100.00% <100.00%> (ø) |
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 37c5ba8...3c7f1ee. Read the comment docs.
LGTM, happy to merge this.
@ccoffrin I think that this fits the functionality we described for the hamiltonian builder. I made sure to include tests. I also fixed a bug I noticed when someone would call sum_z without any weights - now it assumes default weights as it should.