Open samuel-esp opened 2 years ago
rayegun
commented
2 years ago Sorry I lost track of this. If you're still interested in this would you mind doing a couple things for me?
If not I will of course finish these up and merge once I have the time to test them :).
One of the changes I just made requires a change to master (which I overlooked previously) which I will do in a moment.
codecov-commenter
commented
2 years ago Merging #57 (f3b3900) into master (90ab8e7) will increase coverage by
0.36%. The diff coverage isn/a.
@@ Coverage Diff @@
## master #57 +/- ##
==========================================
+ Coverage 24.55% 24.92% +0.36%
==========================================
Files 44 45 +1
Lines 4471 4546 +75
==========================================
+ Hits 1098 1133 +35
- Misses 3373 3413 +40 | Impacted Files | Coverage Δ | |
|---|---|---|
| src/options.jl | 27.02% <0.00%> (-6.31%) |
:arrow_down: |
| src/export.jl | 37.64% <0.00%> (-1.86%) |
:arrow_down: |
| src/chainrules/constructorrules.jl | 54.00% <0.00%> (-0.84%) |
:arrow_down: |
| src/types.jl | 100.00% <0.00%> (ø) |
|
| src/SuiteSparseGraphBLAS.jl | 87.50% <0.00%> (ø) |
|
| src/chainrules/maprules.jl | 15.38% <0.00%> (ø) |
|
| src/lib/LibGraphBLAS.jl | 14.88% <0.00%> (+0.15%) |
:arrow_up: |
| src/operations/map.jl | 64.06% <0.00%> (+2.15%) |
:arrow_up: |
| src/operators/oplist.jl | 10.04% <0.00%> (+2.28%) |
:arrow_up: |
| src/operations/broadcasts.jl | 39.06% <0.00%> (+4.36%) |
: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 90ab8e7...f3b3900. Read the comment docs.
samuel-esp
commented
2 years ago No worries! I have some tests ready on my other repository, just give some days that I'm currently busy with midterms and i'll work on this
new examples added, new files could still contain some print statements used to test the algorithms