checking bayes nets: arrows go from relational attributes to rnodes, not the other way

[oschulte]

No description provided.

[oschulte]

Examples 👍

• BIF_MovieLens_std
• in unielwin, given RA = T, Capability should be n/a with almost probability 1

[woodsouths]

the old xml file of toyexample

and the new xml file of toyexample

the new version misses a edge from RA to Salary. @oschulte

[oschulte]

Thanks runnan. This is because we are currently failing to add these edges by hand (from Rnodes to 2Nodes). Need to check why that part of the SQL script is not being called. Otherwise the edges are the same which is reassuring.

[oschulte]

@woodsouths @rmar3a So the problem seems to be that in model-manager.sql to build the SchemaEdges table, we need to refer to lattice_membership.member, not rnid. I hope this will fix Runnan's issue. As for the general logic, here's the behaviour I think we need.

1. If link analysis is off, then insert the Schema_Edges pairs into Path_Bayes_Nets for the final result after structure learning and before parameter learning. The reason is that in this case, the Bayes net contains no rnodes during learning. This is similar to the current query except we insert into Required_Edges not Path_Bayes_Nets. We can check the Java code for this case.

2. If link analysis is on, then insert the Schema_Edges pairs into Path_Required_Edges before learning. This will make them available for learning. Again We can check the Java code for this case.

[rmar3a]

So I did a bit of investigating to see how/why we dropped the values in the rnid column and it happened/is explained in this commit:

https://github.com/sfu-cl-lab/FactorBase/commit/dfe5e8ca821b1d092b487be1f696316099fd3e11

Also this issue is related to this pull request I made before:

https://github.com/sfu-cl-lab/FactorBase/pull/68

I will attempt to fix both of this issues in a future pull request.

[oschulte]

Sounds good. I think we can resolve this in 15 min. At least for the schema edges.

[oschulte]

So if link analysis is off, schema edges should be added after structure learning and before parameter learning.

[woodsouths]

Eliminate where clause from the creating the schemaEdges table

When link analysis is on:

• Insert edges from schemaEdges into Path_Required_Edges before structure learning;

When link analysis is off:

• Insert edges from schemaEdges into Path_BayesNets after structure learning (together with Progagate ContextEdges);