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.

Sign up for GitHub

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

Jump to bottom

Update example in docstring of HybridGibbs #511

Closed

1 of 2 tasks

chaozg opened this issue Sep 10, 2024 · 0 comments · Fixed by #521

Closed

1 of 2 tasks

Update example in docstring of HybridGibbs #511

chaozg opened this issue Sep 10, 2024 · 0 comments · Fixed by #521

Assignees

Labels

Contributor

chaozg commented Sep 10, 2024 •

edited

Loading

CUQIpy/cuqi/experimental/mcmc/_gibbs.py

Line 92 in 98a74d5

samples = sampler.sample(Ns=1000, Nb=200)

should be updated to something like

sampler.warmup(200)
sampler.sample(1000)
samples = sampler.get_samples()

Definition of Done

Correct use if sampler API implemented in example
Change is reviewed and approved.

chaozg mentioned this issue

Improve (hopefully) how HybridGibbs uses samplers state and history #503

Merged

amal-ghamdi assigned nabriis

nabriis mentioned this issue

Update example in HybridGibbs #521

Merged

nabriis added the documentation label

nabriis closed this as completed in #521

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