Skip to content

Conversation

chriselrod
Copy link
Collaborator

No description provided.

@chriselrod chriselrod enabled auto-merge (squash) October 17, 2021 01:43
@codecov
Copy link

codecov bot commented Oct 17, 2021

Codecov Report

Merging #115 (84214a2) into master (f6110b9) will increase coverage by 0.98%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #115      +/-   ##
==========================================
+ Coverage   84.96%   85.94%   +0.98%     
==========================================
  Files          11       11              
  Lines         758      754       -4     
==========================================
+ Hits          644      648       +4     
+ Misses        114      106       -8     
Impacted Files Coverage Δ
src/complex_matmul.jl 100.00% <100.00%> (ø)
src/matmul.jl 91.24% <100.00%> (+0.54%) ⬆️
src/funcptrs.jl 100.00% <0.00%> (+2.98%) ⬆️
src/global_constants.jl 50.00% <0.00%> (+8.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 f6110b9...84214a2. Read the comment docs.

@chriselrod
Copy link
Collaborator Author

chriselrod commented Oct 18, 2021

@DilumAluthge I think we should split the coverage tests along eltype.

Not sure what's going on, but looking at the log, it's now been 5 hours and it finally finished the Float32 tests -- about half way done.

@DilumAluthge
Copy link
Member

@DilumAluthge I think we should split the coverage tests along eltype.

Makes sense to me!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants