{"action":"created","issue":{"url":"https://api.github.com/repos/CUFCTFACE/face-recognition/issues/16","repository_url":"https://api.github.com/repos/CUFCTFACE/face-recognition","labels_url":"https://api.github.com/repos/CUFCTFACE/face-recognition/issues/16/labels{/name}","comments_url":"https://api.github.com/repos/CUFCTFACE/face-recognition/issues/16/comments","events_url":"https://api.github.com/repos/CUFCTFACE/face-recognition/issues/16/events","html_url":"https://github.com/CUFCTFACE/face-recognition/issues/16","id":184951887,"number":16,"title":"Improve accuracy of ICA C implementation","user":{"login":"ctargon","id":14915984,"avatar_url":"https://avatars.githubusercontent.com/u/14915984?v=3","gravatar_id":"","url":"https://api.github.com/users/ctargon","html_url":"https://github.com/ctargon","followers_url":"https://api.github.com/users/ctargon/followers","following_url":"https://api.github.com/users/ctargon/following{/other_user}","gists_url":"https://api.github.com/users/ctargon/gists{/gist_id}","starred_url":"https://api.github.com/users/ctargon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ctargon/subscriptions","organizations_url":"https://api.github.com/users/ctargon/orgs","repos_url":"https://api.github.com/users/ctargon/repos","events_url":"https://api.github.com/users/ctargon/events{/privacy}","received_events_url":"https://api.github.com/users/ctargon/received_events","type":"User","site_admin":false},"labels":[{"id":196945601,"url":"https://api.github.com/repos/CUFCTFACE/face-recognition/labels/enhancement","name":"enhancement","color":"84b6eb","default":true},{"id":196945602,"url":"https://api.github.com/repos/CUFCTFACE/face-recognition/labels/help%20wanted","name":"help wanted","color":"159818","default":true}],"state":"open","locked":false,"assignee":{"login":"ctargon","id":14915984,"avatar_url":"https://avatars.githubusercontent.com/u/14915984?v=3","gravatar_id":"","url":"https://api.github.com/users/ctargon","html_url":"https://github.com/ctargon","followers_url":"https://api.github.com/users/ctargon/followers","following_url":"https://api.github.com/users/ctargon/following{/other_user}","gists_url":"https://api.github.com/users/ctargon/gists{/gist_id}","starred_url":"https://api.github.com/users/ctargon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ctargon/subscriptions","organizations_url":"https://api.github.com/users/ctargon/orgs","repos_url":"https://api.github.com/users/ctargon/repos","events_url":"https://api.github.com/users/ctargon/events{/privacy}","received_events_url":"https://api.github.com/users/ctargon/received_events","type":"User","site_admin":false},"assignees":[{"login":"ctargon","id":14915984,"avatar_url":"https://avatars.githubusercontent.com/u/14915984?v=3","gravatar_id":"","url":"https://api.github.com/users/ctargon","html_url":"https://github.com/ctargon","followers_url":"https://api.github.com/users/ctargon/followers","following_url":"https://api.github.com/users/ctargon/following{/other_user}","gists_url":"https://api.github.com/users/ctargon/gists{/gist_id}","starred_url":"https://api.github.com/users/ctargon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ctargon/subscriptions","organizations_url":"https://api.github.com/users/ctargon/orgs","repos_url":"https://api.github.com/users/ctargon/repos","events_url":"https://api.github.com/users/ctargon/events{/privacy}","received_events_url":"https://api.github.com/users/ctargon/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/CUFCTFACE/face-recognition/milestones/3","html_url":"https://github.com/CUFCTFACE/face-recognition/milestones/ICA","labels_url":"https://api.github.com/repos/CUFCTFACE/face-recognition/milestones/3/labels","id":2002373,"number":3,"title":"ICA","description":null,"creator":{"login":"arlindohall","id":5963070,"avatar_url":"https://avatars.githubusercontent.com/u/5963070?v=3","gravatar_id":"","url":"https://api.github.com/users/arlindohall","html_url":"https://github.com/arlindohall","followers_url":"https://api.github.com/users/arlindohall/followers","following_url":"https://api.github.com/users/arlindohall/following{/other_user}","gists_url":"https://api.github.com/users/arlindohall/gists{/gist_id}","starred_url":"https://api.github.com/users/arlindohall/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/arlindohall/subscriptions","organizations_url":"https://api.github.com/users/arlindohall/orgs","repos_url":"https://api.github.com/users/arlindohall/repos","events_url":"https://api.github.com/users/arlindohall/events{/privacy}","received_events_url":"https://api.github.com/users/arlindohall/received_events","type":"User","site_admin":false},"open_issues":1,"closed_issues":1,"state":"open","created_at":"2016-09-13T20:16:46Z","updated_at":"2016-10-24T20:54:52Z","due_on":null,"closed_at":null},"comments":1,"created_at":"2016-10-24T20:53:38Z","updated_at":"2016-10-27T21:38:44Z","closed_at":null,"body":"ICA C implementation is compiling, running, and producing accuracies between ~25%-70%.  There are plenty of optimizations that can be incorporated and perhaps a missing line or two from the Matlab. Could use another pair of eyes to go through it and see if I missed anything."},"comment":{"url":"https://api.github.com/repos/CUFCTFACE/face-recognition/issues/comments/256777557","html_url":"https://github.com/CUFCTFACE/face-recognition/issues/16#issuecomment-256777557","issue_url":"https://api.github.com/repos/CUFCTFACE/face-recognition/issues/16","id":256777557,"user":{"login":"bentsherman","id":7884825,"avatar_url":"https://avatars.githubusercontent.com/u/7884825?v=3","gravatar_id":"","url":"https://api.github.com/users/bentsherman","html_url":"https://github.com/bentsherman","followers_url":"https://api.github.com/users/bentsherman/followers","following_url":"https://api.github.com/users/bentsherman/following{/other_user}","gists_url":"https://api.github.com/users/bentsherman/gists{/gist_id}","starred_url":"https://api.github.com/users/bentsherman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bentsherman/subscriptions","organizations_url":"https://api.github.com/users/bentsherman/orgs","repos_url":"https://api.github.com/users/bentsherman/repos","events_url":"https://api.github.com/users/bentsherman/events{/privacy}","received_events_url":"https://api.github.com/users/bentsherman/received_events","type":"User","site_admin":false},"created_at":"2016-10-27T21:38:44Z","updated_at":"2016-10-27T21:38:44Z","body":"Some notes on what Colin and I did today:\r\n\r\n- We implemented the formula for `icasig` in the MATLAB code because the C code didn't implement the whole formula; however, we did not see an increase in accuracy in the test run that we used (1-5 removed from each class, 27.5% in both cases\r\n- The MATLAB ICA code is actually calling `remmean` twice: once in `run_ica`, in which it removes the mean image, and once again in `fastica`, in which it removes the mean \"row\" rather than the mean image because `mixedsig = X'`\r\n\r\nSome things that I plan to do once I have time:\r\n- move matrix functions from `ica.c` to the matrix library\r\n- remove memory leaks\r\n- clean up the code, uniform naming conventions, etc\r\n- combine `temp_PCA` with the main PCA function\r\n\r\nOnce I have done my clean-up, we should run cross-validation so that we can see how the accuracy of the C code changes as the training set shrinks.\r\n\r\nMeanwhile, it might be useful for some people to play around with the parameters in the MATLAB ICA code that were not included in the C code and document their findings so that we know which parameters are worth keeping."}}
{"actor":{"display_login":"bentsherman"}}