Closed richfitz closed 3 years ago
Merging #81 (a3c06b8) into master (fca0b73) will not change coverage. The diff coverage is
100.00%
.:exclamation: Current head a3c06b8 differs from pull request most recent head 76d0af7. Consider uploading reports for the commit 76d0af7 to get more accurate results
@@ Coverage Diff @@
## master #81 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 7 7
Lines 860 921 +61
=========================================
+ Hits 860 921 +61
Impacted Files | Coverage Δ | |
---|---|---|
R/generate_dust.R | 100.00% <100.00%> (ø) |
|
R/generate_dust_equation.R | 100.00% <100.00%> (ø) |
|
R/generate_dust_sexp.R | 100.00% <100.00%> (ø) |
|
R/odin_dust.R | 100.00% <100.00%> (ø) |
|
R/utils.R | 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 fca0b73...76d0af7. Read the comment docs.
This PR totally rewrites the gpu code so that rather than generating code like the CPU version where we unpack everything then run each equation in turn, we unpack locally to each expression. The hope here is that reduces the register pressure (and indeed with sircovid it does)
Fixes #75