Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update playvideo to work with v0.4 and latest ImageView #71

Closed
wants to merge 4 commits into from

Conversation

bnels
Copy link

@bnels bnels commented Oct 16, 2015

Fixes problem where read! could not find an appropriate method.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 52436e8 on bnels:master into * on kmsquire:master*.

@kmsquire
Copy link
Collaborator

Sorry not to respond to this sooner (I've been mostly MIA on this package).

playvideo seems to work (for me) on the current master, without this change. If you're interested in testing this, I'd love to hear if it doesn't work for you still.

Note that, if all goes as planned, the parts of this package which depend on Images.jl will be split out into a separate package at some point soon. (Of course, that assumes I have time to work on this...)

Closing, but please post back here if this is still an issue, and I'll reopen.

@kmsquire kmsquire closed this May 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants