Daniel Turek

Results 80 comments of Daniel Turek

@perrydv @paciorek I'm moving in that direction (described in my most recent comment), until I hear some reason against.

@perrydv Thanks for the feedback and comments. For right now, I'm not going to try to update the `target` nodes variable, in part because the sampler setup code is executed...

@paciorek Thanks for the feedback and the suggestion.

@perrydv @paciorek I’ve worked through our discussion of the posterior predictive revamp. Big picture, I’m satisfied with where this is at. I welcome any degree of comments or code review....

@paciorek Thanks for looking into this. Confirming that this infinite recursion case was recently fixed, and this current example fails on the most recent release, but passes on current `devel`...

@paciorek @claudiawehrhahn Yes, as I recall how this is handled, the "dimension" of `out` will be determined solely based upon the object that's created in the `setup` function. In this...

GitHub makes it much too easy to accidentally "assign yourself" to an issue, by clicking the wrong button...

@MHollanders Give this code a try, in the attached file. Needed to push the dynamic indexing into a nimbleFunction, but it should work the same as your code. [impute-dynamic_DT.txt](https://github.com/nimble-dev/nimble/files/9282137/impute-dynamic_DT.txt)

@MHollanders Good to hear, you won't regret it.