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

Added simple universal UncompressedVideoEncoder. #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bobahml
Copy link

@bobahml bobahml commented Jun 26, 2015

A class for more convenient work with uncompressed data.

@baSSiLL
Copy link
Owner

baSSiLL commented Jul 4, 2015

My intention was that input format for all encoders is fixed to very popular BGR32, for simplicity of usage. However, I admit that sometimes it may be convenient to supply data in another format if you already have such bitmaps. What are your use cases for this new encoder?
Anyway, I think it's better to merge this functionality into existing UncompressedVideoEncoder rather than create another very similar encoder.

@bobahml
Copy link
Author

bobahml commented Jul 6, 2015

I agree.
The new class is really better integrate with the existing ones.
My use case is simple - I get the uncompressed stream from the video camera (Basler): data come in the format of 8-bit grayscale and directly without compressing the stream writing to the output video file.
For me it is important that compression (or converting) is not performed.

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.

2 participants