{"action":"opened","issue":{"url":"https://api.github.com/repos/ntucllab/striatum/issues/66","repository_url":"https://api.github.com/repos/ntucllab/striatum","labels_url":"https://api.github.com/repos/ntucllab/striatum/issues/66/labels{/name}","comments_url":"https://api.github.com/repos/ntucllab/striatum/issues/66/comments","events_url":"https://api.github.com/repos/ntucllab/striatum/issues/66/events","html_url":"https://github.com/ntucllab/striatum/issues/66","id":176302414,"number":66,"title":"weights in Exp4.P are incorrect","user":{"login":"ianlini","id":4056670,"avatar_url":"https://avatars.githubusercontent.com/u/4056670?v=3","gravatar_id":"","url":"https://api.github.com/users/ianlini","html_url":"https://github.com/ianlini","followers_url":"https://api.github.com/users/ianlini/followers","following_url":"https://api.github.com/users/ianlini/following{/other_user}","gists_url":"https://api.github.com/users/ianlini/gists{/gist_id}","starred_url":"https://api.github.com/users/ianlini/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ianlini/subscriptions","organizations_url":"https://api.github.com/users/ianlini/orgs","repos_url":"https://api.github.com/users/ianlini/repos","events_url":"https://api.github.com/users/ianlini/events{/privacy}","received_events_url":"https://api.github.com/users/ianlini/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ntucllab/striatum/labels/bug","name":"bug","color":"ee0701"}],"state":"open","locked":false,"assignee":{"login":"taweihuang","id":19475153,"avatar_url":"https://avatars.githubusercontent.com/u/19475153?v=3","gravatar_id":"","url":"https://api.github.com/users/taweihuang","html_url":"https://github.com/taweihuang","followers_url":"https://api.github.com/users/taweihuang/followers","following_url":"https://api.github.com/users/taweihuang/following{/other_user}","gists_url":"https://api.github.com/users/taweihuang/gists{/gist_id}","starred_url":"https://api.github.com/users/taweihuang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/taweihuang/subscriptions","organizations_url":"https://api.github.com/users/taweihuang/orgs","repos_url":"https://api.github.com/users/taweihuang/repos","events_url":"https://api.github.com/users/taweihuang/events{/privacy}","received_events_url":"https://api.github.com/users/taweihuang/received_events","type":"User","site_admin":false},"assignees":[{"login":"taweihuang","id":19475153,"avatar_url":"https://avatars.githubusercontent.com/u/19475153?v=3","gravatar_id":"","url":"https://api.github.com/users/taweihuang","html_url":"https://github.com/taweihuang","followers_url":"https://api.github.com/users/taweihuang/followers","following_url":"https://api.github.com/users/taweihuang/following{/other_user}","gists_url":"https://api.github.com/users/taweihuang/gists{/gist_id}","starred_url":"https://api.github.com/users/taweihuang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/taweihuang/subscriptions","organizations_url":"https://api.github.com/users/taweihuang/orgs","repos_url":"https://api.github.com/users/taweihuang/repos","events_url":"https://api.github.com/users/taweihuang/events{/privacy}","received_events_url":"https://api.github.com/users/taweihuang/received_events","type":"User","site_admin":false}],"milestone":null,"comments":0,"created_at":"2016-09-12T06:24:02Z","updated_at":"2016-09-12T06:24:02Z","closed_at":null,"body":"https://github.com/ntucllab/striatum/blob/master/striatum/bandit/exp4p.py#L90 \r\nI found that in the second round (the model has been rewarded), w will be something like this: \r\n`{0: array([ 2.36154256,  2.03240672,  2.00984322,  2.01720784,  2.220576  ]), 1: array([ 2.36064009,  2.03172242,  2.00917387,  2.01653361,  2.21976697])}` \r\nwhich I think it's not the same as the original algorithm (every expert has only one weight value). \r\nThis doesn't generate error because the `np.sum` here:\r\nhttps://github.com/ntucllab/striatum/blob/master/striatum/bandit/exp4p.py#L98\r\nincidentally sum all the value, so there won't be any array with wrong dimension. \r\nIs this a bug?"}}
{"actor":{"display_login":"ianlini"}}