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

Conversion failed when using -level 0 #103

Closed
retokromer opened this issue Jun 8, 2018 · 6 comments
Closed

Conversion failed when using -level 0 #103

retokromer opened this issue Jun 8, 2018 · 6 comments
Assignees
Labels
bug Something isn't working

Comments

@retokromer
Copy link
Collaborator

When using the option -level 0 on regular RGB48 content, I receive an error:

bildschirmfoto 2018-06-08 um 17 44 24

@retokromer retokromer changed the title crash with using -level 0 crash when using -level 0 Jun 8, 2018
@JeromeMartinez
Copy link
Member

I see, I force "defaults" for all levels, but e.g. slicecrc is available only from v3. Will not force unsupported features when a level not supporting the feature is requested.

@JeromeMartinez JeromeMartinez added the bug Something isn't working label Jun 8, 2018
@JeromeMartinez JeromeMartinez self-assigned this Jun 8, 2018
@JeromeMartinez
Copy link
Member

Workaround: -level 0 -slicecrc 0 -slices 1

@retokromer
Copy link
Collaborator Author

Hmm… -slicecrc 0 ;-)

@JeromeMartinez
Copy link
Member

You try -level 0 so... normal ;-)

@JeromeMartinez
Copy link
Member

Thinking more about the issue, using levels 0 & 1 remove features, and these removed features (slices) are useful for RAWcooked. So I move on providing a message " -level 0" does not permit slices, is it intended? if so, add " -slices 1" to the command instead of removing automatically the support of slices when " -level 0" is provided.

Note: hesitating about doing same for " -slicecrc", as actually FFmpeg can add CRC in version 0/1 of FFV1 (and RAWcooked, as well as MediaConch, read them), even if it is not standard.

@retokromer
Copy link
Collaborator Author

As said, I’m in favour of requiring the version 3 or above of FFV1 for RAWcooked, but this is not blocking at all.

@JeromeMartinez JeromeMartinez changed the title crash when using -level 0 Conversion failed when using -level 0 Jun 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants