search

StanJulia / StanSample.jl

WIP: Wrapper package for the sample method in Stan's cmdstan executable.
MIT License
18 stars 4 forks source link

Add summary and printsummary to SampleModel constructor #41

Closed Vaibhavdixit02 closed 3 years ago

goedman commented 3 years ago

Hi Vaibhav (is that indeed your correct first name?),

Thanks for your PR! It should be merged in the the next hour or so.

I always used a separate call to sdf = read_summary(model) but your addition is fine. I'm also considering to start using AxisKeys.jl for these purposes.

I would have no problem adding you to the StanJulia members if that would help your work. I'm certainly planning to take a much closer loop at the possibilities of the Symbolics.jl's build_function() option for Stan. Very exciting stuff!

Rob

Vaibhavdixit02 commented 3 years ago

Hi Vaibhav (is that indeed your correct first name?),

Yes it is.

Thanks for the quick merge and release, restarted the CI on the DiffEqBayes PR, hopefully we should see green now 🎉

I would have no problem adding you to the StanJulia members if that would help your work. I'm certainly planning to take a much closer loop at the possibilities of the Symbolics.jl's build_function() option for Stan.

I would be happy to help