Normalization in condensed Chain factor

[wkretschmer]

In theory, the following two elements are equivalent:

val e1 = If(Flip(0.5), Select(0.4 -> 0, 0.6 -> 1), Select(0.6 -> 0, 0.4 -> 1))
val e2 = If(Flip(0.5), Select(0.4 -> 0, 0.6 -> 1), Select(1.2 -> 0, 0.8 -> 1))

The only difference is whether or not the Select in the second argument is normalized. However, running StructuredVE on the second element returns an incorrect result for the distribution. The reason why is that the condensed Chain factor is incorrect when subproblems are not normalized. There are a two ways I could resolve this:

1. Change factor creation to normalize everything. In other words, make every model without evidence have partition function 1.
2. Explicitly normalize the subproblems when computing the condensed factor.

[matanox]

I wonder why is a non-normalized Select allowed in the first place, but I'm just getting started. I mean why doesn't Select enforce normalization by rejecting a tuple that doesn't add up to 1, or auto-normalize upon instantiation.

[wkretschmer]

This is precisely the issue! The normalization is inconsistently applied. I agree that the best way to solve this might be to move some level of normalization to Select initialization, however my concern is that the lack of normalization might occur elsewhere in the codebase.

[bruttenberg]

This normalization issue is popping up in influence diagrams as well. @apfeffer do you have any objection if I change the Select factory to always normalize the factors? I don't see how this could be a problem

[apfeffer]

GO ahead. From: Brian Ruttenberg <[email protected]> Reply-To: p2t2/figaro <[email protected]> Date: Tuesday, October 17, 2017 at 4:45 PM To: p2t2/figaro <[email protected]> Cc: Avi Pfeffer <[email protected]>, Mention <[email protected]> Subject: Re: [p2t2/figaro] Normalization in condensed Chain factor (#715) This normalization issue is popping up in influence diagrams as well. @apfeffer<https://github.com/apfeffer> do you have any objection if I change the Select factory to always normalize the factors? I don't see how this could be a problem — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#715 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AFJkdwY2Xak9N2HOTIq2TVc1cUqWuJPIks5stRHngaJpZM4N1yBB>.