{"action":"created","issue":{"url":"https://api.github.com/repos/dhowden/tag/issues/7","labels_url":"https://api.github.com/repos/dhowden/tag/issues/7/labels{/name}","comments_url":"https://api.github.com/repos/dhowden/tag/issues/7/comments","events_url":"https://api.github.com/repos/dhowden/tag/issues/7/events","html_url":"https://github.com/dhowden/tag/pull/7","id":78300592,"number":7,"title":"Bitrate calculation for mp3","user":{"login":"xhenner","id":713356,"avatar_url":"https://avatars.githubusercontent.com/u/713356?v=3","gravatar_id":"","url":"https://api.github.com/users/xhenner","html_url":"https://github.com/xhenner","followers_url":"https://api.github.com/users/xhenner/followers","following_url":"https://api.github.com/users/xhenner/following{/other_user}","gists_url":"https://api.github.com/users/xhenner/gists{/gist_id}","starred_url":"https://api.github.com/users/xhenner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/xhenner/subscriptions","organizations_url":"https://api.github.com/users/xhenner/orgs","repos_url":"https://api.github.com/users/xhenner/repos","events_url":"https://api.github.com/users/xhenner/events{/privacy}","received_events_url":"https://api.github.com/users/xhenner/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-05-20T00:06:07Z","updated_at":"2015-06-09T22:59:17Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/dhowden/tag/pulls/7","html_url":"https://github.com/dhowden/tag/pull/7","diff_url":"https://github.com/dhowden/tag/pull/7.diff","patch_url":"https://github.com/dhowden/tag/pull/7.patch"},"body":"This patch reads the first mp3 frames, with support for the XING header to speed things up\r\nIt adds some sanity check (for example a flac file with an id3 tag will be rejected)\r\nI tested it against  20k mp3.\r\nWithout the patch, it takes 19s to scan the whole library\r\nWith the patch, it takes 52s\r\n\r\nI still have some checks to do. I want to compare the result with other tool I know are working fine\r\n\r\nNext step : adding the support for mp4 and flac files ; should be easier"},"comment":{"url":"https://api.github.com/repos/dhowden/tag/issues/comments/110527591","html_url":"https://github.com/dhowden/tag/pull/7#issuecomment-110527591","issue_url":"https://api.github.com/repos/dhowden/tag/issues/7","id":110527591,"user":{"login":"xhenner","id":713356,"avatar_url":"https://avatars.githubusercontent.com/u/713356?v=3","gravatar_id":"","url":"https://api.github.com/users/xhenner","html_url":"https://github.com/xhenner","followers_url":"https://api.github.com/users/xhenner/followers","following_url":"https://api.github.com/users/xhenner/following{/other_user}","gists_url":"https://api.github.com/users/xhenner/gists{/gist_id}","starred_url":"https://api.github.com/users/xhenner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/xhenner/subscriptions","organizations_url":"https://api.github.com/users/xhenner/orgs","repos_url":"https://api.github.com/users/xhenner/repos","events_url":"https://api.github.com/users/xhenner/events{/privacy}","received_events_url":"https://api.github.com/users/xhenner/received_events","type":"User","site_admin":false},"created_at":"2015-06-09T22:59:17Z","updated_at":"2015-06-09T22:59:17Z","body":"I don't really get the subpackage idea\r\nFor mp3, it make sense, but for mp4, the information is in the moov atom. Using a sub-package will make thing more complicated\r\n\r\nSame thing for the flac format"}}