{"action":"created","issue":{"url":"https://api.github.com/repos/opengm/opengm/issues/361","labels_url":"https://api.github.com/repos/opengm/opengm/issues/361/labels{/name}","comments_url":"https://api.github.com/repos/opengm/opengm/issues/361/comments","events_url":"https://api.github.com/repos/opengm/opengm/issues/361/events","html_url":"https://github.com/opengm/opengm/pull/361","id":100277904,"number":361,"title":"Add utility class CanonicalView","user":{"login":"fgrsnau","id":11267323,"avatar_url":"https://avatars.githubusercontent.com/u/11267323?v=3","gravatar_id":"","url":"https://api.github.com/users/fgrsnau","html_url":"https://github.com/fgrsnau","followers_url":"https://api.github.com/users/fgrsnau/followers","following_url":"https://api.github.com/users/fgrsnau/following{/other_user}","gists_url":"https://api.github.com/users/fgrsnau/gists{/gist_id}","starred_url":"https://api.github.com/users/fgrsnau/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fgrsnau/subscriptions","organizations_url":"https://api.github.com/users/fgrsnau/orgs","repos_url":"https://api.github.com/users/fgrsnau/repos","events_url":"https://api.github.com/users/fgrsnau/events{/privacy}","received_events_url":"https://api.github.com/users/fgrsnau/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-08-11T10:15:00Z","updated_at":"2015-08-13T13:40:50Z","closed_at":"2015-08-12T12:48:57Z","pull_request":{"url":"https://api.github.com/repos/opengm/opengm/pulls/361","html_url":"https://github.com/opengm/opengm/pull/361","diff_url":"https://github.com/opengm/opengm/pull/361.diff","patch_url":"https://github.com/opengm/opengm/pull/361.patch"},"body":"This class wraps an arbitrary GraphicalModel and acts as a view on the\r\nmodel. The original model is changed with respect to the following\r\naspects:\r\n\r\n  - all variables are associated with exactly one unary factor (multiple\r\n    unary factors get squashed into one, a non-existent unary factor is\r\n    mapped to a zero-constant factor)\r\n\r\n  - there is at most one factor for a given set of variables (multiple\r\n    factors attached to the clique are squashed into one factor)\r\n\r\nAny comments are appreciated. :-)\r\n\r\nI’m using this wrapper in combination with the partial optimality framework (the reduced model somehow contains overlapping unary factors).\r\n\r\nFurthermore, some inference algorithms don’t work if there are no unary potentials (for example CombiLP)."},"comment":{"url":"https://api.github.com/repos/opengm/opengm/issues/comments/130675951","html_url":"https://github.com/opengm/opengm/pull/361#issuecomment-130675951","issue_url":"https://api.github.com/repos/opengm/opengm/issues/361","id":130675951,"user":{"login":"fgrsnau","id":11267323,"avatar_url":"https://avatars.githubusercontent.com/u/11267323?v=3","gravatar_id":"","url":"https://api.github.com/users/fgrsnau","html_url":"https://github.com/fgrsnau","followers_url":"https://api.github.com/users/fgrsnau/followers","following_url":"https://api.github.com/users/fgrsnau/following{/other_user}","gists_url":"https://api.github.com/users/fgrsnau/gists{/gist_id}","starred_url":"https://api.github.com/users/fgrsnau/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fgrsnau/subscriptions","organizations_url":"https://api.github.com/users/fgrsnau/orgs","repos_url":"https://api.github.com/users/fgrsnau/repos","events_url":"https://api.github.com/users/fgrsnau/events{/privacy}","received_events_url":"https://api.github.com/users/fgrsnau/received_events","type":"User","site_admin":false},"created_at":"2015-08-13T13:40:50Z","updated_at":"2015-08-13T13:40:50Z","body":"I pushed my changes to the [same branch](https://github.com/fgrsnau/opengm/commits/utility/canonicalview).\r\n\r\nI don’t consider the branch ready for merging, because I haven’t tested the\r\nimplementation carefully. Maybe you can express your thoughts on it and tell if\r\nthe branch eliminates your aforementioned concerns.\r\n"}}