Closed moble closed 3 years ago
Merging #3 (8da5c70) into main (1181c67) will decrease coverage by
0.80%
. The diff coverage is90.00%
.:exclamation: Current head 8da5c70 differs from pull request most recent head f9b1847. Consider uploading reports for the commit f9b1847 to get more accurate results
@@ Coverage Diff @@
## main #3 +/- ##
==========================================
- Coverage 75.30% 74.50% -0.81%
==========================================
Files 10 10
Lines 648 710 +62
==========================================
+ Hits 488 529 +41
- Misses 160 181 +21
Impacted Files | Coverage Δ | |
---|---|---|
src/map2salm.jl | 92.85% <90.00%> (-7.15%) |
:arrow_down: |
src/wigner_matrices/evaluate.jl | 1.00% <0.00%> (-0.17%) |
:arrow_down: |
src/weights.jl | 100.00% <0.00%> (ø) |
|
src/wigner_matrices/Hrecursions.jl | 98.78% <0.00%> (+0.09%) |
:arrow_up: |
src/complex_powers.jl | 97.22% <0.00%> (+0.16%) |
:arrow_up: |
src/wigner_matrices/indexing.jl | 96.15% <0.00%> (+0.32%) |
:arrow_up: |
src/associated_legendre/calculator.jl | 60.37% <0.00%> (+0.37%) |
:arrow_up: |
src/indexing.jl | 90.00% <0.00%> (+1.11%) |
:arrow_up: |
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 1181c67...f9b1847. Read the comment docs.
This drastically reduces the size of the
G
variable (which essentially doubled the memory required by map2salm), and enables threading in that function. Also adds an option to show the progress bar.