Closed VarLad closed 2 years ago
Pasting a png file here
Merging #11 (2985f63) into main (912fa6c) will increase coverage by
6.77%
. The diff coverage is85.91%
.
@@ Coverage Diff @@
## main #11 +/- ##
==========================================
+ Coverage 81.25% 88.02% +6.77%
==========================================
Files 4 6 +2
Lines 48 167 +119
==========================================
+ Hits 39 147 +108
- Misses 9 20 +11
Impacted Files | Coverage Δ | |
---|---|---|
src/loss.jl | 77.63% <77.63%> (ø) |
|
src/QuantumNLDiffEq.jl | 94.91% <94.64%> (-0.54%) |
:arrow_down: |
src/calculate_diff_evalue.jl | 100.00% <100.00%> (+16.66%) |
:arrow_up: |
src/calculate_evalue.jl | 100.00% <100.00%> (ø) |
|
src/new_circuit.jl | 100.00% <100.00%> (+40.00%) |
:arrow_up: |
src/phi.jl | 100.00% <0.00%> (+50.00%) |
:arrow_up: |
:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more
Looks great!
Make an issue about improving that derivative handling, but it's more than fine enough for now.
This PR includes support for
Other than that, better tests which aim to catch a wrong function call, incorrect function definition and other such problems through all the features.