Open YeWang1576 opened 3 years ago
@kbenoit Thanks for the comments! I will work on the sparse version of the two-dimensional model and modify the dense version according to the suggestions as well.
Merging #32 (639f6b8) into master (34017dc) will decrease coverage by
5.55%
. The diff coverage is28.02%
.:exclamation: Current head 639f6b8 differs from pull request most recent head 989ef98. Consider uploading reports for the commit 989ef98 to get more accurate results
@@ Coverage Diff @@
## master #32 +/- ##
==========================================
- Coverage 64.27% 58.71% -5.56%
==========================================
Files 19 19
Lines 2536 2873 +337
==========================================
+ Hits 1630 1687 +57
- Misses 906 1186 +280
Impacted Files | Coverage Δ | |
---|---|---|
src/wordfish_dense.cpp | 40.91% <25.00%> (-59.09%) |
:arrow_down: |
R/textmodel_wordfish.R | 79.41% <65.51%> (-4.66%) |
:arrow_down: |
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 17f1c84...989ef98. Read the comment docs.
Add the one-dimensional and two-dimensional negative binomial model, as well as the two-dimensional Poisson model