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

Fix CRAN NOTEs and failures #587

Merged
merged 17 commits into from
Apr 7, 2023
Merged

Fix CRAN NOTEs and failures #587

merged 17 commits into from
Apr 7, 2023

Conversation

andrjohns
Copy link
Contributor

Addresses the CRAN notes and donttest example failure from #586, and updates the Makevars files for compatibility with rstan 2.26+

@andrjohns
Copy link
Contributor Author

Diff inflated due to my IDE stripping whitespace, can view with those filtered out here: https://github.com/stan-dev/rstanarm/pull/587/files?diff=unified&w=1

@jgabry
Copy link
Member

jgabry commented Apr 3, 2023

Thanks @andrjohns!!

@jgabry jgabry requested a review from bgoodri April 3, 2023 19:50
@andrjohns
Copy link
Contributor Author

FYI that I've just found one more issue that will need to be resolved for 2.26+ (functions in the functions block modifying target directly), will update this PR asap

@andrjohns
Copy link
Contributor Author

andrjohns commented Apr 4, 2023

That's all the changes from me!

stanc3 doesn't like functions which directly modify target, as well as functions which call other lpdf/lpmf functions without having the lpdf/lpmf suffix themselves. This was causing failures with the stan_functions test under 2.26+. I resolved those issues and expanded the GHA workflow to test against the rstan preview as well.

I've also verified locally that everything compiles and passes under 2.31 as well, so this PR can be considered the minimal changes needed for future compatibility.

Let me know if I've missed or can clarify anything @bgoodri

diff excluding whitespace changes: https://github.com/stan-dev/rstanarm/pull/587/files?diff=unified&w=1

@sambrilleman
Copy link
Collaborator

Wow epic stuff @andrjohns - thanks for keeping this thing alive!! 😅

@bgoodri bgoodri merged commit afd4eb8 into stan-dev:master Apr 7, 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.

4 participants