srlanalytics / bdfm

Bayesian dynamic factor model estimation and predictive statistics including nowcasting and forecasting
MIT License
5 stars 6 forks source link

C++ files split up into smaller files with needed headers for each #64

Closed SethOttoQuant closed 5 years ago

SethOttoQuant commented 5 years ago

This should clean up the C++ code quite a bit. I also got rid of a few old functions we're not using.

codecov-io commented 5 years ago

Codecov Report

Merging #64 into master will increase coverage by 2.11%. The diff coverage is 88.3%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #64      +/-   ##
==========================================
+ Coverage   60.24%   62.35%   +2.11%     
==========================================
  Files           9       11       +2     
  Lines        1567     1517      -50     
==========================================
+ Hits          944      946       +2     
+ Misses        623      571      -52
Impacted Files Coverage Δ
src/BDFM.cpp 57.27% <ø> (-14.29%) :arrow_down:
src/toolbox.cpp 87.85% <87.85%> (ø)
src/utils.cpp 89.68% <89.68%> (ø)

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 77250a9...023677c. Read the comment docs.

christophsax commented 5 years ago

Nice!