crp2a / BayLum

Chronological Bayesian models integrated for Optically Stimulated Luminescence (OSL) Dating
https://crp2a.github.io/BayLum/
GNU General Public License v3.0
9 stars 1 forks source link

object.size.reduce #27

Closed imgoej closed 1 year ago

imgoej commented 1 year ago

The MCMC-list provided by the output of runjags can easily become huge in terms of bytes. It is therefore not prudent to have that list appear twice in the return output of these functions (once as "Sampling" and once as "mcmc" inside "runjags_object").

I therefore set "mcmc" of "runjags_object" to a string pointing to where is "Sampling" is. If input is a BayLum.list (output of these functions and therefore not a generated DataFile), "mcmc" inside "runjags_object" takes on "Sampling" as its value.

codecov[bot] commented 1 year ago

Codecov Report

Merging #27 (29fb5b0) into master (140a715) will decrease coverage by 0.04%. The diff coverage is 17.64%.

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

@@            Coverage Diff             @@
##           master      #27      +/-   ##
==========================================
- Coverage   68.55%   68.51%   -0.04%     
==========================================
  Files          18       18              
  Lines        2194     2198       +4     
==========================================
+ Hits         1504     1506       +2     
- Misses        690      692       +2     
Impacted Files Coverage Δ
R/Age_OSLC14.R 62.68% <13.04%> (-0.07%) :arrow_down:
R/AgeS_Computation.R 81.84% <27.27%> (-0.19%) :arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

RLumSK commented 1 year ago

@imgoej I forgot, please also create a NEWS entry in NEWS.Rmd and then render it with RLumBuild::module_knit_README().

RLumSK commented 1 year ago

@imgoej Thanks!