Alexey Radul

Results 163 comments of Alexey Radul

Sent next round of feedback on #89. My inclination for now is to still try to get that merged first, so no history surgery becomes necessary.

Yes, I think at this stage the thing to do is to have the test suite be developer-facing, and possibly add user-facing self-tests in the future. When it comes into...

There is a second question here, prompted by the observation that the code that writes out the average value dumps checks the `h` array for NaNs (again), rather than `hav`....

@nicolaspi thanks for the contribution! I am no longer an active maintainer of TFP, so I'm not really in a position to review your PR in detail (@jburnim please suggest...

Don't know about time lines, exactly, but work is proceeding.

Actually, @dustinvtran, can you assign me this github issue (or give me sufficient permissions to assign it to myself)?

As an update, there is now a draft barebones NUTS implementation in TensorFlow Probability, at https://github.com/tensorflow/probability/tree/master/experimental/no_u_turn_sampler. This implementation is expected to be compatible with Edward2. See also the associated TFP...

TFP now has a batch NUTS (path length adaptation, haven't tested step size adaptation yet but don't anticipate excessive problems) here: https://github.com/tensorflow/probability/blob/master/tensorflow_probability/python/experimental/mcmc/nuts.py.

That interface looks ideal to me.