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

Argument to pass warmup sample size. #448

Closed
Vaibhavdixit02 opened this issue Jun 11, 2018 · 4 comments
Closed

Argument to pass warmup sample size. #448

Vaibhavdixit02 opened this issue Jun 11, 2018 · 4 comments

Comments

@Vaibhavdixit02
Copy link

I couldn't find any documentation on passing warmup sample size to the samplers, from #265 I understand you use 1k as the default warmup sample size?
It would be a good addition to have a argument that would allow the user to pass in the required warm up sample size.

@xukai92
Copy link
Member

xukai92 commented Sep 3, 2018

Sorry about our bad documentation. There are actually ways to pass it. Either

  • Constructing samplers with additional arguments by HMCDA(5000, 2000, ...) (the second integer is considered as warmup size), or
  • Passing sa::Stan.Adapt to sample by sample(...; adapt_conf=sa)

@trappmartin
Copy link
Member

Thanks for the clarification.

But I guess this will only work for Hamiltonian subtyped methods. Is this correct?

If so we could maybe find a common interface for this purpose. If not we should maybe think about a more convenient interface or kw argument?

@Vaibhavdixit02
Copy link
Author

Thanks for the clarification @xukai92. I think having kwarg for this would be a nice feature to have.

@yebai
Copy link
Member

yebai commented Sep 18, 2019

Closed by #793

@yebai yebai closed this as completed Sep 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants