{"action":"created","issue":{"url":"https://api.github.com/repos/ccrisan/streameye/issues/6","repository_url":"https://api.github.com/repos/ccrisan/streameye","labels_url":"https://api.github.com/repos/ccrisan/streameye/issues/6/labels{/name}","comments_url":"https://api.github.com/repos/ccrisan/streameye/issues/6/comments","events_url":"https://api.github.com/repos/ccrisan/streameye/issues/6/events","html_url":"https://github.com/ccrisan/streameye/pull/6","id":178679619,"number":6,"title":"added new command line options related to preview output","user":{"login":"doughadfield","id":6051108,"avatar_url":"https://avatars.githubusercontent.com/u/6051108?v=3","gravatar_id":"","url":"https://api.github.com/users/doughadfield","html_url":"https://github.com/doughadfield","followers_url":"https://api.github.com/users/doughadfield/followers","following_url":"https://api.github.com/users/doughadfield/following{/other_user}","gists_url":"https://api.github.com/users/doughadfield/gists{/gist_id}","starred_url":"https://api.github.com/users/doughadfield/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/doughadfield/subscriptions","organizations_url":"https://api.github.com/users/doughadfield/orgs","repos_url":"https://api.github.com/users/doughadfield/repos","events_url":"https://api.github.com/users/doughadfield/events{/privacy}","received_events_url":"https://api.github.com/users/doughadfield/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-09-22T18:04:11Z","updated_at":"2016-09-24T11:07:21Z","closed_at":"2016-09-22T18:09:06Z","pull_request":{"url":"https://api.github.com/repos/ccrisan/streameye/pulls/6","html_url":"https://github.com/ccrisan/streameye/pull/6","diff_url":"https://github.com/ccrisan/streameye/pull/6.diff","patch_url":"https://github.com/ccrisan/streameye/pull/6.patch"},"body":"modified raspimjpeg.py file:\r\nAdded -p (--preview) to enable HDMI port video output\r\nAdded --s_height and --s_width to separately specify resolution of video\r\nstream\r\nif not set, these default to the same as the video device settings (-w,\r\n-h)\r\nNew streams_iter() function to speed up frame processing in network\r\nstream\r\nRequires addition to \"Video Device\" web form to add \"preview\" switch (boolean) to set -p flag, and resolution drop-down in \"Video Streaming\" form to set s_width and s_height\r\n\r\nTo do: work out a scheme for separately setting frame rate for network video stream"},"comment":{"url":"https://api.github.com/repos/ccrisan/streameye/issues/comments/249359171","html_url":"https://github.com/ccrisan/streameye/pull/6#issuecomment-249359171","issue_url":"https://api.github.com/repos/ccrisan/streameye/issues/6","id":249359171,"user":{"login":"ccrisan","id":76639,"avatar_url":"https://avatars.githubusercontent.com/u/76639?v=3","gravatar_id":"","url":"https://api.github.com/users/ccrisan","html_url":"https://github.com/ccrisan","followers_url":"https://api.github.com/users/ccrisan/followers","following_url":"https://api.github.com/users/ccrisan/following{/other_user}","gists_url":"https://api.github.com/users/ccrisan/gists{/gist_id}","starred_url":"https://api.github.com/users/ccrisan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ccrisan/subscriptions","organizations_url":"https://api.github.com/users/ccrisan/orgs","repos_url":"https://api.github.com/users/ccrisan/repos","events_url":"https://api.github.com/users/ccrisan/events{/privacy}","received_events_url":"https://api.github.com/users/ccrisan/received_events","type":"User","site_admin":false},"created_at":"2016-09-24T11:07:21Z","updated_at":"2016-09-24T11:07:21Z","body":"1. Why have you changed \"capture\" to \"camera\" in the context of width and height arguments? It still is the capturing resolution even after your changes. And when you say \"camera width/height\" you usually think of the maximum supported camera resolution.\r\n\r\n2. I really don't like the names of \"s_width/s_height\" arguments. Let's go with `-x`, `--streaming-width` and `-y`, `--streaming-height`. Command line arguments rarely make use of underscore.\r\n\r\n3. The preview functionality is perfect.\r\n\r\n4. We should see if we can actually go beyond 1920x1080 and in what circumstances, as people have asked for higher resolutions knowing that the camera actually does support up to 2k something. It may only work in stills mode."}}
{"actor":{"display_login":"ccrisan"}}