ChrisJones687 commented 3 years ago

add r wrapper for pops_model and change c++ function to pops_model_cpp that isn't exposed. Everything works exactly the same but this makes updating documentation easier and in line with best practices.

codecov[bot] commented 3 years ago

Codecov Report

Merging #94 (d02462e) into master (4600da2) will increase coverage by 0.49%. The diff coverage is 100.00%.

:exclamation: Current head d02462e differs from pull request most recent head cef1d83. Consider uploading reports for the commit cef1d83 to get more accurate results

@@            Coverage Diff             @@
##           master      #94      +/-   ##
==========================================
+ Coverage   75.17%   75.67%   +0.49%     
==========================================
  Files          39       40       +1     
  Lines        4589     4658      +69     
==========================================
+ Hits         3450     3525      +75     
+ Misses       1139     1133       -6

Impacted Files	Coverage Δ
R/pops_model.R	`100.00% <100.00%> (ø)`
src/pops.cpp	`94.54% <100.00%> (ø)`
R/calibrate.R	`92.32% <0.00%> (+1.02%)`	: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 4600da2...cef1d83. Read the comment docs.

ChrisJones687 commented 3 years ago

fixes #88 last part and makes documentation easier to update

ncsu-landscape-dynamics / rpops

create r wrapper function for pops_model #94

Codecov Report