GitHub PushEvent at 2014-08-12 18:13:49
Repository: stb-tester
Actor: wmanley
Payload: {"shas":[["e641b4c3174cb0b5bc71c658c296a1def06c856f","aa743a0aaec8f7d7a1f01442503957f4d7a2d634@rothlis.net","stbt run: Dump pipeline-error debug from the right pipeline\n\nIf you define the environment variable `GST_DEBUG_DUMP_DOT_DIR` and a\nGStreamer error occurs, `stbt run` will dump a Graphviz dot file with\ndebugging information (see 7b3eaf33). Previously we always dumped\ndebugging information for the *source* pipeline, even if the error came\nfrom the *sink* pipeline.","David Röthlisberger",false],["b40b830b40aafdb69a714cf4d45d318f61bb6719","aa743a0aaec8f7d7a1f01442503957f4d7a2d634@rothlis.net","stbt batch run: Classify blackmagic failure correctly\n\nThe classification for errors due to the Blackmagic Intensity Pro\nvideo-capture card hasn't worked correctly since we ported stbt to\nGStreamer 1.0 in commit a83333a2.\n\nAlso the \"blackmagic card in use by another process\" detection logic\nhasn't worked since the blackmagic device moved from /dev/blackmagic0 to\n/dev/blackmagic/dv0 -- presumably this changed with the recently\nreleased blackmagic 10.0 driver. We look at both locations, thus\nproviding an upgrade path to users (upgrade stb-tester first, then\nupgrade the blackmagic driver -- if you care enough about getting this\nerror classification right).\n\nThanks to Lewis Haley for the bug report and for the detailed solution.\nFixes #144.","David Röthlisberger",false],["7ec5a7e4fbe867488a4faa2928d3dac60d7f8cc3","aa743a0aaec8f7d7a1f01442503957f4d7a2d634@rothlis.net","Merge pull request #148 from drothlis/blackmagic-classification\n\nstbt batch run: Classify blackmagic failure correctly","David Röthlisberger",false],["0984374d44cac0ab9fd3418c124a6c33483d4801","37d41699bdee4fcb969ca499eb0f8b82c60d59cc@stb-tester.com","Bug fix: Ensure that frames are still dropped even after a pipeline restart\n\nIf stbt's image analysis is slower than real time frames can get queued\nup in the pipeline meaning that the frames we are operating on may be out\nof date.  To minimise this latency we drop frames from a queue in the\nGStreamer pipeline.\n\nThis commit fixes a bug introduced in b8d39d4 where this leaky behaviour\nwas not being applied after the pipeline was restarted.","William Manley",false],["2cc88a3b370193810652a48b07fd7c7770deaf25","70ef5ae0bbb1aaaec4bbbd9af6e4df4e6a56b055@youview.com","stbt.py, stbt-tv: Move the leaky queue right after `decodebin`\n\nThis commit resolves high latency of video frames on slow computers by\nensuring that an appropriate number of frames are dropped.\n\nThe latency is too high if the same queue is after `videoconvert` even\nthough the property `leaky=downstream` is set. The idea that the leaky\nqueue should come before `videoconvert` is the outcome of\ntrial-and-error.\n\nIt is in-keeping with the idea in the pipeline that we shouldn't be\nthrowing away frames we've done significant processing on.\n\nNote that the slow-down is due to `decodebin` and not stbt's video\nprocessing algorithm. Using `stbt tv` (i.e. a simple video capture\npipeline) without a leaky queue after `decodebin` also results in high\nlatency.\n\nThe \"slow computer\" used for testing had 2.4 GHz Core 2 Duo CPU.","Mate Szendro",false],["3904e87273ae2b06e9ea4f97b6c09508db4de1ed","aa743a0aaec8f7d7a1f01442503957f4d7a2d634@rothlis.net","stbt screenshot: Skip first frame\n\nSome V4L video-capture hardware delivers a single black frame before the\nreal video stream.","David Röthlisberger",false],["9b918604eaad48cdafd1f287a6be909e251607fa","aa743a0aaec8f7d7a1f01442503957f4d7a2d634@rothlis.net","Merge pull request #152 from drothlis/stbt-screenshot-skip-first-frame\n\nstbt screenshot: Skip first frame","David Röthlisberger",false],["1d50cf073335608e6eb90b61c1712d5bec828e21","64b2b6d12bfe4baae7dad3d018f8cbf6b0e7a044@williammanley.net","Merge pull request #151 from szemate/ensure-that-frames-are-dropped\n\nFix high latency of video frames on slow computers","William Manley",false],["07ccb49dc6c14ce06cb9c2028bddd6130f867521","aa743a0aaec8f7d7a1f01442503957f4d7a2d634@rothlis.net","README.rst: Update byline to match stb-tester.com\n\nThis is the first step in clearer documentation & promotional material\nabout stb-tester's true capabilities -- it isn't restricted to set-top\nboxes.","David Röthlisberger",false],["a9b8360af5cc9bb9331c690fe85452e3525608c4","aa743a0aaec8f7d7a1f01442503957f4d7a2d634@rothlis.net","make check: Enforce pep8 `import` order\n\nPEP8 says: [1]\n\n> Imports should be grouped in the following order:\n>\n> 1. standard library imports\n> 2. related third party imports\n> 3. local application/library specific imports\n>\n> You should put a blank line between each group of imports.\n\nIf we're going to follow this convention, let's make the computer check\nit to avoid wasting reviewer cycles.\n\nUnfortunately neither the `pep8` tool nor `pylint` have checks to\nenforce this. Instead, we use `isort`[2] from PyPI (pip install isort).\n\nIf you have trouble getting a file to pass isort's check, run\n`isort` (without `--check-only`) on the file to fix it in-place.\nTo keep this as convenient as possible, don't customise isort's\nformatting rules in our pylint.sh.\n\n[1] http://legacy.python.org/dev/peps/pep-0008/#imports\n[2] https://pypi.python.org/pypi/isort/","David Röthlisberger",false],["13b335854bf5df728895c9b0afddac46621b3edb","aa743a0aaec8f7d7a1f01442503957f4d7a2d634@rothlis.net","make check: check pep8 import order only if `isort` is installed\n\n`isort` isn't installable with yum/apt-get on Fedora, Debian or Ubuntu;\nsome developers find it a bit annoying to be forced to use pip, so make\nthis optional. The important thing is that Travis will check it.","David Röthlisberger",false],["b0119c1142a8cd93670ce387abb36d2570ba0fdf","aa743a0aaec8f7d7a1f01442503957f4d7a2d634@rothlis.net","make check: Run pylint before unit & integration tests\n\nStatic analysis is faster than our dynamic tests, so if static analysis\nis going to fail the build we want to fail as quickly as possible.","David Röthlisberger",false],["0cd184a3193acfa1cfe6ba2c6657c82df9d1f1ef","aa743a0aaec8f7d7a1f01442503957f4d7a2d634@rothlis.net","make check: Log details when isort fails\n\nisort (3.8.0) passes on my PC, but fails on Travis (isort version\n???). We need some diagnostic information.\n\nNote that `isort --diff --check-only` doesn't work; `--check-only` takes\npriority and `--diff` is ignored.","David Röthlisberger",false],["c3158285af31482b7bee5be304b83cdb0c8dd020","aa743a0aaec8f7d7a1f01442503957f4d7a2d634@rothlis.net","Add .isort.cfg to fix false positives on Travis\n\nWhen run on Travis, `isort` thinks that `binascii` is a 3rd-party\nmodule (it's part of the python standard library), and that `cv2` is a\nstandard library (it's a third-party library, from `apt-get install\npython-opencv`). I don't know why.\n\nI've raised https://github.com/timothycrosley/isort/issues/157","David Röthlisberger",false],["ee92040929099ad31870a522254cd3bfbd79f750","aa743a0aaec8f7d7a1f01442503957f4d7a2d634@rothlis.net","Merge pull request #154 from drothlis/make-check--import-order\n\nmake check: Enforce pep8 `import` order","David Röthlisberger",false],["3315a33734ae873099564e74f0eb29ec10f5d579","37d41699bdee4fcb969ca499eb0f8b82c60d59cc@stb-tester.com","OCR: Add script for measuring OCR effectiveness\n\nvalidate-ocr.py can be run on a corpus of test images reporting how good a\njob stbt has done of reading the text.  Thus it can be used to measure\nimprovements to the OCR algorithm and, more importantly, detect any\nregressions introduced.\n\nThis tool is designed such that it can be run on corpuses outside the\nstb-tester git tree to allow corpuses containing screen captures from many\nset-top boxes without bloating the main stb-tester repo or risking\nupsetting the owners of the creators of the various set-top box UIs.\n\nIt is not currently run in-tree as a part of make check.\n\nThanks to david@rothlis.net and lewis.haley@youview.com for review and\nfixups.","William Manley",false],["d296cb411857ee2ef0f598590d03b4989eb6294b","37d41699bdee4fcb969ca499eb0f8b82c60d59cc@stb-tester.com","Region: Add convenience methods\n\nSometimes it's convenient to think in terms of width and height, other\ntimes in terms of extents.  The addition of `right`, `bottom` and\n`from_extents` to `Region` allow either model to be used as convenient.\n\nThese and the `_bounding_box` helper function will be useful for more\nadvanced uses of OCR which requires manipulating regions.","William Manley",false],["50d862a83e8d7a5dea00e5a03c857160e7fb17e2","37d41699bdee4fcb969ca499eb0f8b82c60d59cc@stb-tester.com","OCR: Support tesseract writing to files with any extension\n\nThe second argument to the tesseract command line tool is \"output base\"\nwhich is a filename to which an extension will be appended.  Unfortunately\nthis extension varies depending on the arguments passed to tesseract and\neven varies between different versions of tesseract[^1].\n\nUsing a temporary directory means that we don't need to know the extension\nin advance and we can read the output independent of the extension that\ntesseract appends.\n\n[^1]: In hOCR mode tesseract 3.02 will write to base.html whereas 3.03 will\n      write to base.hocr.","William Manley",false],["1dffd5a8aeabf5484534b49fd6683a2ea6dec087","37d41699bdee4fcb969ca499eb0f8b82c60d59cc@stb-tester.com","Refactor: _tesseract: Require that a frame to be passed in\n\nThis removes a dependency on `Display` from `_tesseract` making it a pure\nfunction depending entirely on it's arguments and producing no\nside-effects[^1].  This makes uses of it clearer.\n\n[^1]: Or close enough.","William Manley",false],["fcdcb0fc90f67204f18c6e2a561a8e0c0d1fccb8","37d41699bdee4fcb969ca499eb0f8b82c60d59cc@stb-tester.com","Refactor: _tesseract: Allow passing a `Gst.Sample` as frame\n\nThis was originally written to allow getting the timestamp of frames being\npassed to `_tesseract` allowing returning richer information.  This has not\nyet been done but being able to pass a `GstSample` is still a nice thing to\nbe able to do.  It makes _tesseract more like the other image analysis\nfunctions and has the pleasant side-effect of reducing copying.","William Manley",false]],"size":167,"ref":"refs/heads/smart-tv-support","head":"4ce1fd6d528ad9734b301d2f31c498b7f59c1721"}