Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

start posterior vignette #719

Merged
merged 3 commits into from
Jun 26, 2023
Merged

Conversation

gravesti
Copy link
Contributor

Submission Checklist

  • Run unit tests
  • Declare copyright holder and agree to license (see below)

Summary

This is to start some work on #590

Specifically this point

  • use the default $summary() function and show how to run non-default summaries with it (similar to what is shown in custom summary )

Copyright and Licensing

Please list the copyright holder for the work you are submitting
(this will be you or your assignee, such as a university or company):
Isaac Gravestock

By submitting this pull request, the copyright holder is agreeing to
license the submitted work under the following licenses:

@jgabry
Copy link
Member

jgabry commented Nov 18, 2022

Thanks for starting this!

@codecov-commenter
Copy link

codecov-commenter commented Nov 18, 2022

Codecov Report

Merging #719 (af819c2) into master (5f61efe) will decrease coverage by 0.03%.
The diff coverage is n/a.

❗ Current head af819c2 differs from pull request most recent head 9b6b88a. Consider uploading reports for the commit 9b6b88a to get more accurate results

@@            Coverage Diff             @@
##           master     #719      +/-   ##
==========================================
- Coverage   88.11%   88.08%   -0.03%     
==========================================
  Files          12       12              
  Lines        4130     4130              
==========================================
- Hits         3639     3638       -1     
- Misses        491      492       +1     

see 1 file with indirect coverage changes

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

@danielinteractive
Copy link

Hi @jgabry just checking whether you have any comments? (From our side we would like to help moving cmdstanr on CRAN)

@rok-cesnovar rok-cesnovar self-requested a review June 16, 2023 09:00
* master: (22 commits)
  Temporarily skip CI failures
  Temporarily skip CI failures
  Fix bug in check_syntax and format with include_paths on compiled model
  Add tests for check_syntax and format on compiled model
  Bump r-lib/actions from 2.4.0 to 2.6.3
  Fix functor name
  Windows oldrel requires rtools42
  Bump actions ver
  Auto rtools ver
  Fix expose_functions under 2.32
  skips
  append to make local
  one more hpp reading fix
  fix hpp last line warning
  Update install.R
  Update install.R
  Add stanc M1 make patch, suppress boost warnings
  Fix Rtools42 minor version
  Update workflow rtools
  Update utilities for RTools43
  ...
Copy link
Member

@rok-cesnovar rok-cesnovar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great to me, thanks! Will rebuild for the website once we bump the version.

@rok-cesnovar rok-cesnovar merged commit 83bbc35 into stan-dev:master Jun 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants