{"action":"created","issue":{"url":"https://api.github.com/repos/sfu-cl-lab/FactorBase/issues/81","repository_url":"https://api.github.com/repos/sfu-cl-lab/FactorBase","labels_url":"https://api.github.com/repos/sfu-cl-lab/FactorBase/issues/81/labels{/name}","comments_url":"https://api.github.com/repos/sfu-cl-lab/FactorBase/issues/81/comments","events_url":"https://api.github.com/repos/sfu-cl-lab/FactorBase/issues/81/events","html_url":"https://github.com/sfu-cl-lab/FactorBase/issues/81","id":353139820,"node_id":"MDU6SXNzdWUzNTMxMzk4MjA=","number":81,"title":"checking bayes nets: arrows go from relational attributes to rnodes, not the other way","user":{"login":"oschulte","id":16602497,"node_id":"MDQ6VXNlcjE2NjAyNDk3","avatar_url":"https://avatars1.githubusercontent.com/u/16602497?v=4","gravatar_id":"","url":"https://api.github.com/users/oschulte","html_url":"https://github.com/oschulte","followers_url":"https://api.github.com/users/oschulte/followers","following_url":"https://api.github.com/users/oschulte/following{/other_user}","gists_url":"https://api.github.com/users/oschulte/gists{/gist_id}","starred_url":"https://api.github.com/users/oschulte/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/oschulte/subscriptions","organizations_url":"https://api.github.com/users/oschulte/orgs","repos_url":"https://api.github.com/users/oschulte/repos","events_url":"https://api.github.com/users/oschulte/events{/privacy}","received_events_url":"https://api.github.com/users/oschulte/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"woodsouths","id":23412069,"node_id":"MDQ6VXNlcjIzNDEyMDY5","avatar_url":"https://avatars2.githubusercontent.com/u/23412069?v=4","gravatar_id":"","url":"https://api.github.com/users/woodsouths","html_url":"https://github.com/woodsouths","followers_url":"https://api.github.com/users/woodsouths/followers","following_url":"https://api.github.com/users/woodsouths/following{/other_user}","gists_url":"https://api.github.com/users/woodsouths/gists{/gist_id}","starred_url":"https://api.github.com/users/woodsouths/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/woodsouths/subscriptions","organizations_url":"https://api.github.com/users/woodsouths/orgs","repos_url":"https://api.github.com/users/woodsouths/repos","events_url":"https://api.github.com/users/woodsouths/events{/privacy}","received_events_url":"https://api.github.com/users/woodsouths/received_events","type":"User","site_admin":false},"assignees":[{"login":"woodsouths","id":23412069,"node_id":"MDQ6VXNlcjIzNDEyMDY5","avatar_url":"https://avatars2.githubusercontent.com/u/23412069?v=4","gravatar_id":"","url":"https://api.github.com/users/woodsouths","html_url":"https://github.com/woodsouths","followers_url":"https://api.github.com/users/woodsouths/followers","following_url":"https://api.github.com/users/woodsouths/following{/other_user}","gists_url":"https://api.github.com/users/woodsouths/gists{/gist_id}","starred_url":"https://api.github.com/users/woodsouths/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/woodsouths/subscriptions","organizations_url":"https://api.github.com/users/woodsouths/orgs","repos_url":"https://api.github.com/users/woodsouths/repos","events_url":"https://api.github.com/users/woodsouths/events{/privacy}","received_events_url":"https://api.github.com/users/woodsouths/received_events","type":"User","site_admin":false}],"milestone":null,"comments":3,"created_at":"2018-08-22T21:53:05Z","updated_at":"2018-08-28T01:02:04Z","closed_at":null,"author_association":"COLLABORATOR","body":""},"comment":{"url":"https://api.github.com/repos/sfu-cl-lab/FactorBase/issues/comments/416416525","html_url":"https://github.com/sfu-cl-lab/FactorBase/issues/81#issuecomment-416416525","issue_url":"https://api.github.com/repos/sfu-cl-lab/FactorBase/issues/81","id":416416525,"node_id":"MDEyOklzc3VlQ29tbWVudDQxNjQxNjUyNQ==","user":{"login":"oschulte","id":16602497,"node_id":"MDQ6VXNlcjE2NjAyNDk3","avatar_url":"https://avatars1.githubusercontent.com/u/16602497?v=4","gravatar_id":"","url":"https://api.github.com/users/oschulte","html_url":"https://github.com/oschulte","followers_url":"https://api.github.com/users/oschulte/followers","following_url":"https://api.github.com/users/oschulte/following{/other_user}","gists_url":"https://api.github.com/users/oschulte/gists{/gist_id}","starred_url":"https://api.github.com/users/oschulte/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/oschulte/subscriptions","organizations_url":"https://api.github.com/users/oschulte/orgs","repos_url":"https://api.github.com/users/oschulte/repos","events_url":"https://api.github.com/users/oschulte/events{/privacy}","received_events_url":"https://api.github.com/users/oschulte/received_events","type":"User","site_admin":false},"created_at":"2018-08-28T01:02:04Z","updated_at":"2018-08-28T01:02:04Z","author_association":"COLLABORATOR","body":"@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.\r\n\r\n1. 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.\r\n\r\n2. 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."}}
{"actor":{"display_login":"oschulte"}}