-
Notifications
You must be signed in to change notification settings - Fork 101
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
Fix differential equation tutorial #296
Conversation
tutorials/10-bayesian-differential-equations/10_bayesian-differential-equations.jmd
Outdated
Show resolved
Hide resolved
tutorials/10-bayesian-differential-equations/10_bayesian-differential-equations.jmd
Outdated
Show resolved
Hide resolved
tutorials/10-bayesian-differential-equations/10_bayesian-differential-equations.jmd
Outdated
Show resolved
Hide resolved
tutorials/10-bayesian-differential-equations/10_bayesian-differential-equations.jmd
Outdated
Show resolved
Hide resolved
tutorials/10-bayesian-differential-equations/10_bayesian-differential-equations.jmd
Outdated
Show resolved
Hide resolved
tutorials/10-bayesian-differential-equations/10_bayesian-differential-equations.jmd
Outdated
Show resolved
Hide resolved
@staticfloat I think there might be an issue with coppermind? https://buildkite.com/julialang/turingtutorials/builds/76#9b6b2925-254f-43b2-8401-9a7224be6376/205-207 |
@staticfloat It seems it's a problem specifically with aarch64 (or the workers) as it still works with x86_64: https://buildkite.com/julialang/turingtutorials/builds/78#b12d7990-0ba6-422b-9a30-df85278cc46d Probably bash does not support all features in the failing builds? The offending line seems to be https://github.com/staticfloat/coppermind-buildkite-plugin/blob/bc1c8b1f473651490bcaed15dfe16488d447a0e4/hooks/pre-command#L8. |
Ah no, it's just the exit code. So I assume |
Will be fixed by #298. |
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
|
||
# This next command runs 3 independent chains without using multithreading. | ||
# Sample 3 independent chains with forward-mode automatic differentiation (the default). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe add a link to the docs on how to change AD backends here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's shown how to change it (and change it back) below, is that sufficiently clear? Or should we add a link here? Or maybe rather below?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for the clarification, that should be sufficient.
There's one remaining issue, namely that |
I've wanted to do this for quite some time but unfortunately my computer broke while working on an update...
So here are some fixes for the differential equation tutorial.
Most notably,
MvNormal
deprecationsPreview: https://julialang-buildkite-artifacts.s3.amazonaws.com/turingtutorials/e3c8bc002f5223b50eb6e9c7902473c040159c0aa4d5942d979166fa05bcb422/html/10-bayesian-differential-equations/10_bayesian-differential-equations.html