JuliaIO / VideoIO.jl / 1034 / 1
34%
master: 34%

DEFAULT BRANCH: master
Ran
Files 105
Run time 6s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 32.305% (+10.8%) from 21.537%
1034.1

push

travis-ci

web-flow 
Add iterator interface to VideoReader objects (#261)

* Add iterator interface to VideoReader objects

I have added an iterator interface to the `VideoReader` type, making collecting
all frames of a video as simple as `f = openvideo(fname); imgs = collect(f)`.
The iterator returns a new array for each frame, as I think mutating the same
array on subsequent calls to `iterate` would be surprising and lead to strange
behavior when calling functions such as `collect`. Note that the VideoReader
iterator is mutable, like Channel or Stateful iterators in Julia, and will
resume iteration where it was last stopped, instead of at the beginning. I have
added some text in the documentation explaining this behavior, and also added
some tests.

* Do not collect all frames into memory to test iterator interface

Instead of collecting all frames into memory with `collect` to test the
`VideoReader` interface, I am instead testing the iterator interface with a
simple for loop. This should reduce the memory requirement for CI, and avoid
memory-related test errors on low memory CI VMs.

* Simplify test of mutability of `VideoReader` iterator

Instead of looping over the iterator to test that there are zero elements
remaining, it is simpler to just see if `iterate` returns `nothing`.

* Test that iterator interface for `VideoReader` does not mutate frames

I have added a test to verify that frames returned by the `VideoReader`
iterator interface have distinct storage.

471 of 1458 relevant lines covered (32.3%)

65383.98 hits per line

Source Files on job 1034.1