-
Notifications
You must be signed in to change notification settings - Fork 73
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
cmdstan format #211
Comments
On reflection, I do not think this is worth a special feature. Users can load the relevant output into memory and specify the existing self$draws()
try(self$sampler_diagnostics(), silent = TRUE)
try(self$init(), silent = TRUE) |
Related: @mike-lawrence and others, do you think there would be value in a
|
That sounds for sure useful! |
This weekend, I implemented something like #211 (comment) in a new package called |
|
Update: I am trying to continue what I started with |
Hey @wlandau. Upon seeing |
I think you do. You know
Targetopia packages aim to automatically build in options and techniques that would either be annoying to implement manually or too advanced for the majority of users. Examples:
At some point I plan to write general guidance for targetopia developers. |
Thanks for the detailed response, Will. This is very helpful for understanding how these add-on packages contribute to the |
I am glad you are interested. With some good |
That looks really nice! I's be happy to help! Let me know if you need any development in the torch side too. |
Thanks, glad to hear it! The |
@mattwarkentin, FYI: the R Targetopia now has its own website with a more detailed contributing guide. |
Prework
Proposal
We could consider accommodating
CmdStanFit
objects withtar_target(format = "cmdstanr")
. Related: stan-dev/cmdstanr#340.The text was updated successfully, but these errors were encountered: