{"action":"created","issue":{"url":"https://api.github.com/repos/guysoft/OctoPi/issues/113","labels_url":"https://api.github.com/repos/guysoft/OctoPi/issues/113/labels{/name}","comments_url":"https://api.github.com/repos/guysoft/OctoPi/issues/113/comments","events_url":"https://api.github.com/repos/guysoft/OctoPi/issues/113/events","html_url":"https://github.com/guysoft/OctoPi/pull/113","id":88433749,"number":113,"title":"Fixes webcamDaemon for Logitech C170","user":{"login":"foosel","id":83657,"avatar_url":"https://avatars.githubusercontent.com/u/83657?v=3","gravatar_id":"","url":"https://api.github.com/users/foosel","html_url":"https://github.com/foosel","followers_url":"https://api.github.com/users/foosel/followers","following_url":"https://api.github.com/users/foosel/following{/other_user}","gists_url":"https://api.github.com/users/foosel/gists{/gist_id}","starred_url":"https://api.github.com/users/foosel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/foosel/subscriptions","organizations_url":"https://api.github.com/users/foosel/orgs","repos_url":"https://api.github.com/users/foosel/repos","events_url":"https://api.github.com/users/foosel/events{/privacy}","received_events_url":"https://api.github.com/users/foosel/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-06-15T13:51:19Z","updated_at":"2015-06-15T20:59:52Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/guysoft/OctoPi/pulls/113","html_url":"https://github.com/guysoft/OctoPi/pull/113","diff_url":"https://github.com/guysoft/OctoPi/pull/113.diff","patch_url":"https://github.com/guysoft/OctoPi/pull/113.patch"},"body":"For whatever reason using the -f parameter with that specific camera\r\nmodel causes it to just not output any image. mjpgstreamer will start\r\nup just fine and not log an error or anything, the camera will appear\r\nto start up too, but when connecting to the webcam stream nothing will\r\nbe displayed at all. Just removing the -f parameter in this case fixes\r\nit completely.\r\n\r\nThis patch adds a new check to the webcamDaemon when starting up a\r\nUSB webcam which checks the webcams USB vid and pid against a list\r\nof webcams known not to work with -f (right now that's just the C170,\r\nbut more can be added easily by the user through the new\r\nadditional_brokenfps_usb_devices option in /boot/octopi.txt) and if\r\na match is found strips the -f parameter from the camera options.\r\n\r\nThis is the only way I found I could make a C170 work. It would\r\nprobably be cleaner to try to debug what's going wrong in mjpgstreamer\r\nthere, but since that might as well be a problem with the chipset\r\ndriver of the C170, this is a workaround that should at least\r\nyield a picture at a default frame rate."},"comment":{"url":"https://api.github.com/repos/guysoft/OctoPi/issues/comments/112210353","html_url":"https://github.com/guysoft/OctoPi/pull/113#issuecomment-112210353","issue_url":"https://api.github.com/repos/guysoft/OctoPi/issues/113","id":112210353,"user":{"login":"guysoft","id":325670,"avatar_url":"https://avatars.githubusercontent.com/u/325670?v=3","gravatar_id":"","url":"https://api.github.com/users/guysoft","html_url":"https://github.com/guysoft","followers_url":"https://api.github.com/users/guysoft/followers","following_url":"https://api.github.com/users/guysoft/following{/other_user}","gists_url":"https://api.github.com/users/guysoft/gists{/gist_id}","starred_url":"https://api.github.com/users/guysoft/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/guysoft/subscriptions","organizations_url":"https://api.github.com/users/guysoft/orgs","repos_url":"https://api.github.com/users/guysoft/repos","events_url":"https://api.github.com/users/guysoft/events{/privacy}","received_events_url":"https://api.github.com/users/guysoft/received_events","type":"User","site_admin":false},"created_at":"2015-06-15T20:59:52Z","updated_at":"2015-06-15T20:59:52Z","body":"Merged your mega-commit first, lets let that build a nightly, then will merge this."}}