stan-dev / posterior

The posterior R package
https://mc-stan.org/posterior/
Other
167 stars 24 forks source link

Add examples of converting base R 3-D array and mcmc/mcmc.list objects to README.md #301

Closed avehtari closed 1 year ago

avehtari commented 1 year ago

Add examples of converting base R 3-D array and mcmc/mcmc.list objects

As it took me a moment to figure out how to convert R 3-D array to draws so that variables are named (as the approach used for matrix doesn't work) it is probably helpful for others to see the example, too.

I didn't even know that mcmc/mcmc.list object conversion is supported, and it would be useful to mention it at least once somewhere, so that it is possible to point others to tha information.

codecov-commenter commented 1 year ago

Codecov Report

Merging #301 (1245251) into master (e23467b) will not change coverage. The diff coverage is n/a.

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

@@           Coverage Diff           @@
##           master     #301   +/-   ##
=======================================
  Coverage   95.66%   95.66%           
=======================================
  Files          46       46           
  Lines        3645     3645           
=======================================
  Hits         3487     3487           
  Misses        158      158           

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

github-actions[bot] commented 1 year ago

This is how benchmark results would change (along with a 95% confidence interval in relative change) if 1245251f74747941ff4a3b95e19d7cb09e8bcd5f is merged into master:

paul-buerkner commented 1 year ago

We do not require coda and rjags in posterior so I assume we will fail some checks by adding these packages to the readme. Did you run the usual CMD checks with this already? if not, I will do it when working on the new posterior release in the next few days.

paul-buerkner commented 1 year ago

I think you edited the readme.md file directly, but all changes should only go to the readme.Rmd file which will then be used to render the corresponding .md file. Anyway, I have made the necessary changes and will merge now. Thank you for this addition!

github-actions[bot] commented 1 year ago

This is how benchmark results would change (along with a 95% confidence interval in relative change) if 9aa5a8d88cbbe6feefbba6852325dc374d1fa166 is merged into master: