{"action":"created","issue":{"url":"https://api.github.com/repos/bodylabs/smithers/issues/2","labels_url":"https://api.github.com/repos/bodylabs/smithers/issues/2/labels{/name}","comments_url":"https://api.github.com/repos/bodylabs/smithers/issues/2/comments","events_url":"https://api.github.com/repos/bodylabs/smithers/issues/2/events","html_url":"https://github.com/bodylabs/smithers/pull/2","id":77644944,"number":2,"title":"Asynchronous captures to have longer kinect sequences","user":{"login":"raffienficiaud","id":6357107,"avatar_url":"https://avatars.githubusercontent.com/u/6357107?v=3","gravatar_id":"","url":"https://api.github.com/users/raffienficiaud","html_url":"https://github.com/raffienficiaud","followers_url":"https://api.github.com/users/raffienficiaud/followers","following_url":"https://api.github.com/users/raffienficiaud/following{/other_user}","gists_url":"https://api.github.com/users/raffienficiaud/gists{/gist_id}","starred_url":"https://api.github.com/users/raffienficiaud/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/raffienficiaud/subscriptions","organizations_url":"https://api.github.com/users/raffienficiaud/orgs","repos_url":"https://api.github.com/users/raffienficiaud/repos","events_url":"https://api.github.com/users/raffienficiaud/events{/privacy}","received_events_url":"https://api.github.com/users/raffienficiaud/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-05-18T12:48:17Z","updated_at":"2015-07-07T03:09:15Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/bodylabs/smithers/pulls/2","html_url":"https://github.com/bodylabs/smithers/pull/2","diff_url":"https://github.com/bodylabs/smithers/pull/2.diff","patch_url":"https://github.com/bodylabs/smithers/pull/2.patch"},"body":"This is part 1/2 of the PR. \r\nWe changes the code in order to have an asynchronous capture scheme and to enable longer capture sequences. The captured frames are now stored in a memory pool from the capture thread, and threads from a thread pool are saving these frames to disk during the capture. \r\n\r\nPart of the processing is also delayed to the saving threads to have 30fps in the capture thread. "},"comment":{"url":"https://api.github.com/repos/bodylabs/smithers/issues/comments/119057476","html_url":"https://github.com/bodylabs/smithers/pull/2#issuecomment-119057476","issue_url":"https://api.github.com/repos/bodylabs/smithers/issues/2","id":119057476,"user":{"login":"chenliu0831","id":1504463,"avatar_url":"https://avatars.githubusercontent.com/u/1504463?v=3","gravatar_id":"","url":"https://api.github.com/users/chenliu0831","html_url":"https://github.com/chenliu0831","followers_url":"https://api.github.com/users/chenliu0831/followers","following_url":"https://api.github.com/users/chenliu0831/following{/other_user}","gists_url":"https://api.github.com/users/chenliu0831/gists{/gist_id}","starred_url":"https://api.github.com/users/chenliu0831/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chenliu0831/subscriptions","organizations_url":"https://api.github.com/users/chenliu0831/orgs","repos_url":"https://api.github.com/users/chenliu0831/repos","events_url":"https://api.github.com/users/chenliu0831/events{/privacy}","received_events_url":"https://api.github.com/users/chenliu0831/received_events","type":"User","site_admin":false},"created_at":"2015-07-07T03:09:15Z","updated_at":"2015-07-07T03:09:15Z","body":"@raffienficiaud Just took a shallow pass. Actually in the not-yet-open-sourced part of this framework, we were doing some similar P-C queue thing but just in the visualization part. This PR is great as it seems to also extend it to the serialization part. \r\n\r\nThat being said, it looks like we were using some higher level primitives of .Net like `BlockingCollection` and `Task` to manage workers and  `ConcurrentQueue` for reuse free items. Here it seems more low-level but I would like to do a more detailed comparison and probably combine the goodies. If possible please take a look at those primitives and I'll try to make our implementation of this part open-sourced asap. \r\n\r\nThanks! More detailed review will be upcoming. "}}