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

Use Stan 2.26 functions #1077

Closed
hsbadr opened this issue Jan 26, 2021 · 7 comments
Closed

Use Stan 2.26 functions #1077

hsbadr opened this issue Jan 26, 2021 · 7 comments
Milestone

Comments

@hsbadr
Copy link
Contributor

hsbadr commented Jan 26, 2021

Now that the development version of rstan supports Stan version 2.26, it may be more efficient to use the new functions, such as log_inv_logit_diff:

# TODO: replace with log_inv_logit_diff once rstan >= 2.25

Also, since version 2.25 won't exist, as mentioned by @rok-cesnovar in #1074 (comment), it'd be cleaner to use version 2.26.

@paul-buerkner
Copy link
Owner

paul-buerkner commented Jan 26, 2021 via email

@hsbadr
Copy link
Contributor Author

hsbadr commented Jan 26, 2021

We will need to wait until that rstan version is on CRAN.

Yes, just preparing for it. I'll open a PR and leave it until the new version is published on CRAN.

@hsbadr
Copy link
Contributor Author

hsbadr commented Jan 26, 2021

We will need to wait until that rstan version is on CRAN.

@paul-buerkner Please help in replacing the functions in #1078. We can still make this conditional:

if (rstan::stan_version() >= 2.26) {
  # Use the new functions
} else {
  # Keep the current code.
}

@paul-buerkner
Copy link
Owner

paul-buerkner commented Jan 26, 2021 via email

@hsbadr
Copy link
Contributor Author

hsbadr commented Jan 26, 2021

I am sorry but I have so much stuff on my hands right now that won't be able to do much on this at the moment. But I will come back to it when I can.

No problem. Take your time! None of this is urgent. The performance may or may not change, but it'd be great to follow the Stan development anyway.

@paul-buerkner paul-buerkner added this to the 2.21.0 milestone Nov 15, 2023
@paul-buerkner
Copy link
Owner

Most of this is now implemented. Only categorical_logit_glm is missing, but that will be added too for the 2.21 release.

@paul-buerkner
Copy link
Owner

Completed now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants