Skip to content

Commit 65f0b1e

Browse files
pipmelarryshamalama
andcommitted
Update docs/source/contributing/developer_guide.md
Co-authored-by: larryshamalama <[email protected]>
1 parent 424ece0 commit 65f0b1e

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

docs/source/contributing/developer_guide.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -543,7 +543,7 @@ The base class for most MCMC sampler (except SMC) is in [ArrayStep](https://gith
543543
You can see that the ``step.step()`` is mapping the ``point`` into an array, and call ``self.astep()``, which is an array in, array out function.
544544
A PyMC model compiles a conditional logp/dlogp function that replace the input RVs with a shared 1D tensor (flatten and stack view of the original RVs).
545545
And the transition kernel (i.e., ``.astep()``) takes an array as input and outputs an array.
546-
See for example in the [MH sampler](https://github.com/pymc-devs/pymc/blob/89f6fcf751774fb50016561dc448a87fba7ed3aa/pymc/step_methods/metropolis.py#L235-L289).
546+
For example, see the [MH sampler](https://github.com/pymc-devs/pymc/blob/89f6fcf751774fb50016561dc448a87fba7ed3aa/pymc/step_methods/metropolis.py#L235-L289).
547547

548548
This is of course very different compared to the transition kernel in e.g. TFP, which is a tenor in tensor out function.
549549
Moreover, transition kernels in TFP do not flatten the tensors, see eg docstring of [tensorflow\_probability/python/mcmc/random\_walk\_metropolis.py](https://github.com/tensorflow/probability/blob/master/tensorflow_probability/python/mcmc/random_walk_metropolis.py):

0 commit comments

Comments
 (0)