-
Notifications
You must be signed in to change notification settings - Fork 0
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
Document design decisions #7
Comments
Just noting that we've already got https://github.com/epiforecasts/dist.spec/blob/main/inst/design.md |
Yes, I had that in mind as a starting point. I think we should make it a vignette to properly expose it, though. I'm also not exactly sure how the other packages would adopt it into their current infrastructure or how much breaking changes they'd need to make to adopt this package. It would be good to access that by engaging with the developers of the listed packages. |
Noting for future reference I don't think we need to target |
I think |
I'm yet to review |
I mean only 1 person is interested and they say its great (its me) |
The TLDR is we ship stan code inside the R code and provides some tools to get at it and put it in peoples models either at run time or by developers. Its not ideal but its better than copy and pasting |
Closing this issue the suggested document already exists, although I think it's better stored as a vignette. |
Let's take a step back and have a call to brainstorm and document the design decisions we want to stick to here. This will also capture wider interoperability issues (related: #6 and epiforecasts/EpiNow2#823 (comment))
The text was updated successfully, but these errors were encountered: