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

Updates to make notebook runnable and fix deprecation warnings #461

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

hsm207
Copy link
Contributor

@hsm207 hsm207 commented May 11, 2019

Commit d39fa93 makes this notebook runnable. The rest are deprecation fixes.

Fixes SyntaxError: positional argument follows keyword argument
dtype is an argument to tf.cast
tfp.mcmc.make_simple_step_size_update_policy() requires num_adaptation_steps as an argument. I am not sure if the correct value is None but it gives the same results as the PyMC3 version of this notebook.
Fixes:
 to_int32 (from tensorflow.python.ops.math_ops) is deprecated and will be removed in a future version.
Instructions for updating:
Use tf.cast instead.
Forgot to change tf.to_int32 to tf.cast
The original implementation will throw a shape error when N>1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant