{"action":"created","issue":{"url":"https://api.github.com/repos/tiny-dnn/tiny-dnn/issues/536","repository_url":"https://api.github.com/repos/tiny-dnn/tiny-dnn","labels_url":"https://api.github.com/repos/tiny-dnn/tiny-dnn/issues/536/labels{/name}","comments_url":"https://api.github.com/repos/tiny-dnn/tiny-dnn/issues/536/comments","events_url":"https://api.github.com/repos/tiny-dnn/tiny-dnn/issues/536/events","html_url":"https://github.com/tiny-dnn/tiny-dnn/pull/536","id":203814015,"number":536,"title":"use ThreadPool","user":{"login":"beru","id":1131125,"avatar_url":"https://avatars.githubusercontent.com/u/1131125?v=3","gravatar_id":"","url":"https://api.github.com/users/beru","html_url":"https://github.com/beru","followers_url":"https://api.github.com/users/beru/followers","following_url":"https://api.github.com/users/beru/following{/other_user}","gists_url":"https://api.github.com/users/beru/gists{/gist_id}","starred_url":"https://api.github.com/users/beru/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/beru/subscriptions","organizations_url":"https://api.github.com/users/beru/orgs","repos_url":"https://api.github.com/users/beru/repos","events_url":"https://api.github.com/users/beru/events{/privacy}","received_events_url":"https://api.github.com/users/beru/received_events","type":"User","site_admin":false},"labels":[{"id":485196099,"url":"https://api.github.com/repos/tiny-dnn/tiny-dnn/labels/PR:%20Good%20to%20Merge","name":"PR: Good to Merge","color":"0e8a16","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":34,"created_at":"2017-01-28T15:51:16Z","updated_at":"2017-02-05T10:18:45Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/tiny-dnn/tiny-dnn/pulls/536","html_url":"https://github.com/tiny-dnn/tiny-dnn/pull/536","diff_url":"https://github.com/tiny-dnn/tiny-dnn/pull/536.diff","patch_url":"https://github.com/tiny-dnn/tiny-dnn/pull/536.patch"},"body":"Hi, this PR changes `parallel_for` function to use ThreadPool library written by Jakob Progsch, Václav Zeman. https://github.com/progschj/ThreadPool\r\n\r\nThis will improve execution speed on Linux.\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/tiny-dnn/tiny-dnn/issues/comments/277509269","html_url":"https://github.com/tiny-dnn/tiny-dnn/pull/536#issuecomment-277509269","issue_url":"https://api.github.com/repos/tiny-dnn/tiny-dnn/issues/536","id":277509269,"user":{"login":"Randl","id":3028543,"avatar_url":"https://avatars.githubusercontent.com/u/3028543?v=3","gravatar_id":"","url":"https://api.github.com/users/Randl","html_url":"https://github.com/Randl","followers_url":"https://api.github.com/users/Randl/followers","following_url":"https://api.github.com/users/Randl/following{/other_user}","gists_url":"https://api.github.com/users/Randl/gists{/gist_id}","starred_url":"https://api.github.com/users/Randl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Randl/subscriptions","organizations_url":"https://api.github.com/users/Randl/orgs","repos_url":"https://api.github.com/users/Randl/repos","events_url":"https://api.github.com/users/Randl/events{/privacy}","received_events_url":"https://api.github.com/users/Randl/received_events","type":"User","site_admin":false},"created_at":"2017-02-05T10:18:45Z","updated_at":"2017-02-05T10:18:45Z","body":"> Another concern is if putting third-party library files into tiny_dnn folder is really not allowed, adding a cmake option and preprocessor macros are needed and we need to increase the amount of Continuous Integration tests. That's something I don't want to choose..\r\n\r\nThat's exactly what I meant - making it hard dependency, i.e. not have an option to compile tiny-dnn without it. Anyways we are supposed to be hardly dependent on some thread pool implementation, so does it matter if it's written by us or someone else, as far as it's header only, supports all the platforms and is provided by our repository?"}}
{"actor":{"display_login":"Randl"}}