Warning message issue

[Marcus Brubaker]

Hey all,

So I'm getting a lot of these:

Warning (non-fatal):     sampling statement (~) contains a transformed parameter or local variable.

     You must increment lp__ with the log absolute determinant of the Jacobian of the transform.

     Sampling Statement left-hand-side expression:

          row(x_pred,i) ~ normal_log(...)

There are two issues here.  One is that lp__ is deprecated, so the message is somewhat misleading.

The other is that this message isn't correct here, in fact I'm not sure we're really going to be able to provide a warning like this that won't chronically misfire.  I would propose disabling it for now, or providing a flag to stanc which silences it.

Cheers,

Marcus

[Daniel Lee]

What branch are you using?

--
You received this message because you are subscribed to the Google Groups "stan development mailing list" group.
To unsubscribe from this group and stop receiving emails from it, send an email to stan-dev+u...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

[Marcus Brubaker]

This is on develop.

[Bob Carpenter]

We can do that.

We're going around in circles on these warning messages.

Our users seem to interpret "warning" as "ERROR", and
I'm not sure how to correct that.

The message is technically correct here, it's just that
the log Jacobian is 0.

We should definitely change the language, though, if
we keep it, to mention increment_log_prob().

- Bob

On 8/16/13 1:31 PM, Marcus Brubaker wrote:
> This is on develop.
>
>
> On Fri, Aug 16, 2013 at 1:26 PM, Daniel Lee <bea...@alum.mit.edu <mailto:bea...@alum.mit.edu>> wrote:
>
> What branch are you using?
>
>

> On Fri, Aug 16, 2013 at 1:22 PM, Marcus Brubaker <marcus.a...@gmail.com <mailto:marcus.a...@gmail.com>>

> wrote:
>
> Hey all,
>
> So I'm getting a lot of these:
>
> Warning (non-fatal): sampling statement (~) contains a transformed parameter or local variable.
> You must increment lp__ with the log absolute determinant of the Jacobian of the transform.
> Sampling Statement left-hand-side expression:
> row(x_pred,i) ~ normal_log(...)
>
> There are two issues here. One is that lp__ is deprecated, so the message is somewhat misleading.
>
> The other is that this message isn't correct here, in fact I'm not sure we're really going to be able to provide
> a warning like this that won't chronically misfire. I would propose disabling it for now, or providing a flag
> to stanc which silences it.
>
> Cheers,
> Marcus
>
> --
> You received this message because you are subscribed to the Google Groups "stan development mailing list" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to

> stan-dev+u...@googlegroups.com <mailto:stan-dev%2Bunsu...@googlegroups.com>.

> For more options, visit https://groups.google.com/groups/opt_out.
>
>
> --
> You received this message because you are subscribed to the Google Groups "stan development mailing list" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to

> stan-dev+u...@googlegroups.com <mailto:stan-dev%2Bunsu...@googlegroups.com>.