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 packages with stanfunctions under rstan 2.33+ #117

Merged
merged 4 commits into from Sep 12, 2023

Conversation

andrjohns
Copy link
Collaborator

This PR resolves a similar issue as this cmdstanr PR: stan-dev/cmdstanr#847

After 2.33 stanc3 will emit plain types for standalone functions instead of auto, so we no longer need to pre-process the hpp to do this ourselves.

This PR is needed for rstan 2.33 compatibility with packages like lgpr and rmdcev which export standalone functions during installation

@andrjohns andrjohns merged commit 4efe8ff into master Sep 12, 2023
6 checks passed
@andrjohns andrjohns deleted the stanfunctions-2.33+ branch September 12, 2023 13:53
@andrjohns
Copy link
Collaborator Author

@jgabry When you've got the time would you mind pushing a new release to CRAN? We'll need this change release for lgpr and rmdcev to build with the next rstan/StanHeaders

@jgabry
Copy link
Member

jgabry commented Jan 16, 2024 via email

@andrjohns
Copy link
Collaborator Author

No worries, there's no rush!

@jgabry
Copy link
Member

jgabry commented Jan 16, 2024

I can get the ball rolling on this while catching up on other stuff. I'll start running reverse dependence checks.

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.

None yet

2 participants