MPEG2 Profile Options Integer order wrong

classic Classic list List threaded Threaded
9 messages Options
Reply | Threaded
Open this post in threaded view
|

MPEG2 Profile Options Integer order wrong

Bill Crockett
https://ffmpeg.org/ffmpeg-all.html#mpeg2

Who maintains the documentation on the above and how can I contact to ask if it is wrong or correct?

16.15 mpeg2

MPEG-2 video encoder.

16.15.1 Options

profile integer

    Select the mpeg2 profile to encode:

    ‘422’
    ‘main’
    ‘ss’

        Spatially Scalable
    ‘snr’

        SNR Scalable
    ‘high’
    ‘simple’

There are no numbers listed, but my guess is the order as is written and would be "main" = 1 and "high" = 4

This is wrong and "high should be 1 and "main" should be 4.
_______________________________________________
ffmpeg-user mailing list
[hidden email]
https://ffmpeg.org/mailman/listinfo/ffmpeg-user

To unsubscribe, visit link above, or email
[hidden email] with subject "unsubscribe".
Reply | Threaded
Open this post in threaded view
|

Re: MPEG2 Profile Options Integer order wrong

Limin Wang
On Wed, Jun 17, 2020 at 09:31:47AM +1000, Bill Crockett wrote:

> https://ffmpeg.org/ffmpeg-all.html#mpeg2
>
> Who maintains the documentation on the above and how can I contact to ask if it is wrong or correct?
>
> 16.15 mpeg2
>
> MPEG-2 video encoder.
>
> 16.15.1 Options
>
> profile integer
>
>     Select the mpeg2 profile to encode:
>
>     ‘422’
>     ‘main’
>     ‘ss’
>
>         Spatially Scalable
>     ‘snr’
>
>         SNR Scalable
>     ‘high’
>     ‘simple’
>
> There are no numbers listed, but my guess is the order as is written and would be "main" = 1 and "high" = 4

Yes, it's const string, you can use -profile:v high or -profile:v main to get
expect result.

>
> This is wrong and "high should be 1 and "main" should be 4.

If you guess the number by the order, I'll submit a patch to fix it to avoid such
confusion, it's not expected.

> _______________________________________________
> ffmpeg-user mailing list
> [hidden email]
> https://ffmpeg.org/mailman/listinfo/ffmpeg-user
>
> To unsubscribe, visit link above, or email
> [hidden email] with subject "unsubscribe".

--
Thanks,
Limin Wang
_______________________________________________
ffmpeg-user mailing list
[hidden email]
https://ffmpeg.org/mailman/listinfo/ffmpeg-user

To unsubscribe, visit link above, or email
[hidden email] with subject "unsubscribe".
Reply | Threaded
Open this post in threaded view
|

Re: MPEG2 Profile Options Integer order wrong

Bill Crockett
----- Original Message -----
From: "Limin Wang" <[hidden email]>
To: <[hidden email]>
Sent: Wednesday, June 17, 2020 10:24 AM
Subject: Re: [FFmpeg-user] MPEG2 Profile Options Integer order wrong


> On Wed, Jun 17, 2020 at 09:31:47AM +1000, Bill Crockett wrote:
>> https://ffmpeg.org/ffmpeg-all.html#mpeg2
>>
>> Who maintains the documentation on the above and how can I contact to ask
>> if it is wrong or correct?
>>
>> 16.15 mpeg2
>>
>> MPEG-2 video encoder.
>>
>> 16.15.1 Options
>>
>> profile integer
>>
>>     Select the mpeg2 profile to encode:
>>
>>     ‘422’
>>     ‘main’
>>     ‘ss’
>>
>>         Spatially Scalable
>>     ‘snr’
>>
>>         SNR Scalable
>>     ‘high’
>>     ‘simple’
>>
>> There are no numbers listed, but my guess is the order as is written and
>> would be "main" = 1 and "high" = 4
>
> Yes, it's const string, you can use -profile:v high or -profile:v main to
> get
> expect result.
>
>>
>> This is wrong and "high should be 1 and "main" should be 4.
>
> If you guess the number by the order, I'll submit a patch to fix it to
> avoid such
> confusion, it's not expected.
>
> Thanks,
> Limin Wang

Thanks for that.

Before you told me this, I did try the words "high" or "main", but I put in
quotes either single or double and got errors.

https://ffmpeg.org/ffmpeg-all.html#Options

Does MPEG2 still have the -level:v options?  This is all that is said:

level integer (encoding,audio,video)
Possible values:

‘unknown’

In the past, I found you could use 2, 4, 5, 6. or 8

_______________________________________________
ffmpeg-user mailing list
[hidden email]
https://ffmpeg.org/mailman/listinfo/ffmpeg-user

To unsubscribe, visit link above, or email
[hidden email] with subject "unsubscribe".
Reply | Threaded
Open this post in threaded view
|

Re: MPEG2 Profile Options Integer order wrong

Limin Wang
On Wed, Jun 17, 2020 at 12:28:39PM +1000, Bill Crockett wrote:

> ----- Original Message ----- From: "Limin Wang"
> <[hidden email]>
> To: <[hidden email]>
> Sent: Wednesday, June 17, 2020 10:24 AM
> Subject: Re: [FFmpeg-user] MPEG2 Profile Options Integer order wrong
>
>
> >On Wed, Jun 17, 2020 at 09:31:47AM +1000, Bill Crockett wrote:
> >>https://ffmpeg.org/ffmpeg-all.html#mpeg2
> >>
> >>Who maintains the documentation on the above and how can I
> >>contact to ask if it is wrong or correct?
> >>
> >>16.15 mpeg2
> >>
> >>MPEG-2 video encoder.
> >>
> >>16.15.1 Options
> >>
> >>profile integer
> >>
> >>    Select the mpeg2 profile to encode:
> >>
> >>    ‘422’
> >>    ‘main’
> >>    ‘ss’
> >>
> >>        Spatially Scalable
> >>    ‘snr’
> >>
> >>        SNR Scalable
> >>    ‘high’
> >>    ‘simple’
> >>
> >>There are no numbers listed, but my guess is the order as is
> >>written and would be "main" = 1 and "high" = 4
> >
> >Yes, it's const string, you can use -profile:v high or -profile:v
> >main to get
> >expect result.
> >
> >>
> >>This is wrong and "high should be 1 and "main" should be 4.
> >
> >If you guess the number by the order, I'll submit a patch to fix
> >it to avoid such
> >confusion, it's not expected.
> >
> >Thanks,
> >Limin Wang
>
> Thanks for that.
>
> Before you told me this, I did try the words "high" or "main", but I
> put in quotes either single or double and got errors.
>
> https://ffmpeg.org/ffmpeg-all.html#Options
>
> Does MPEG2 still have the -level:v options?  This is all that is said:

You are talking about profile, please do not switch to level, they're different
option.

level didn't support with const option, so you had to use integer for it.

Below is examples for reference:
Main profile and Main level:
-profile:v main -level:v 8

Main profile and high level:
-profile:v main -level:v 4

422 profile and main level:
-profile:v 422 -level:v 5

422 profile and high level:
-profile:v 422 -level:v 2

>
> level integer (encoding,audio,video)
> Possible values:
>
> ‘unknown’
>
> In the past, I found you could use 2, 4, 5, 6. or 8
>
> _______________________________________________
> ffmpeg-user mailing list
> [hidden email]
> https://ffmpeg.org/mailman/listinfo/ffmpeg-user
>
> To unsubscribe, visit link above, or email
> [hidden email] with subject "unsubscribe".

--
Thanks,
Limin Wang
_______________________________________________
ffmpeg-user mailing list
[hidden email]
https://ffmpeg.org/mailman/listinfo/ffmpeg-user

To unsubscribe, visit link above, or email
[hidden email] with subject "unsubscribe".
Reply | Threaded
Open this post in threaded view
|

Re: MPEG2 Profile Options Integer order wrong

Moritz Barsnick
In reply to this post by Bill Crockett
Hi Bill,

On Wed, Jun 17, 2020 at 12:28:39 +1000, Bill Crockett wrote:
> Before you told me this, I did try the words "high" or "main", but I put in
> quotes either single or double and got errors.

It would be extremely helpful if you showed us the *actual* command
line used, and the (complete, uncut) console output, which is helpful
to debug any kind of issues. We can usually point out your issue from
this info. "Got errors" is very terse.

Cheers,
Moritz
_______________________________________________
ffmpeg-user mailing list
[hidden email]
https://ffmpeg.org/mailman/listinfo/ffmpeg-user

To unsubscribe, visit link above, or email
[hidden email] with subject "unsubscribe".
Reply | Threaded
Open this post in threaded view
|

Re: MPEG2 Profile Options Integer order wrong

Bill Crockett
In reply to this post by Bill Crockett
On Wed Jun 17 15:28:43 EEST 2020 Moritz Barsnick wrote:

> Hi Bill,

> On Wed, Jun 17, 2020 at 12:28:39 +1000, Bill Crockett wrote:
>> Before you told me this, I did try the words "high" or "main", but I put
>> in
>> quotes either single or double and got errors.

> It would be extremely helpful if you showed us the *actual* command
> line used, and the (complete, uncut) console output, which is helpful
> to debug any kind of issues. We can usually point out your issue from
> this info. "Got errors" is very terse.

> Cheers,
> Moritz

Hi Moritz,

Sorry it sounded terse.  There was no need for me to add the command line or
console output.  I took the documentation here literally:

https://ffmpeg.org/ffmpeg-all.html#Options-33

profile integer

    Select the mpeg2 profile to encode:

    ‘422’
    ‘main’
    ‘ss’

        Spatially Scalable
    ‘snr’

        SNR Scalable
    ‘high’
    ‘simple’

The Profile options have quotes around them.  Probably to high-light them in
the document.  As Limin stated, you don't put in quotes.  It was also
confusing because it asks for an integer, but should be string.

Thanks,
Bill

_______________________________________________
ffmpeg-user mailing list
[hidden email]
https://ffmpeg.org/mailman/listinfo/ffmpeg-user

To unsubscribe, visit link above, or email
[hidden email] with subject "unsubscribe".
Reply | Threaded
Open this post in threaded view
|

Re: MPEG2 Profile Options Integer order wrong

Carl Eugen Hoyos-2
In reply to this post by Bill Crockett
Am Mi., 17. Juni 2020 um 01:32 Uhr schrieb Bill Crockett <[hidden email]>:

>
> https://ffmpeg.org/ffmpeg-all.html#mpeg2
>
> Who maintains the documentation on the above and how can I contact to ask if it is wrong or correct?
>
> 16.15 mpeg2
>
> MPEG-2 video encoder.
>
> 16.15.1 Options
>
> profile integer
>
>     Select the mpeg2 profile to encode:
>
>     ‘422’
>     ‘main’
>     ‘ss’
>
>         Spatially Scalable
>     ‘snr’
>
>         SNR Scalable
>     ‘high’
>     ‘simple’
>
> There are no numbers listed, but my guess is the order as is written and would be "main" = 1 and "high" = 4
>
> This is wrong and "high should be 1 and "main" should be 4.

Indeed.

I will wait a few more days and fix it then unless somebody beats me.

Carl Eugen
_______________________________________________
ffmpeg-user mailing list
[hidden email]
https://ffmpeg.org/mailman/listinfo/ffmpeg-user

To unsubscribe, visit link above, or email
[hidden email] with subject "unsubscribe".
Reply | Threaded
Open this post in threaded view
|

Re: MPEG2 Profile Options Integer order wrong

Marton Balint


On Fri, 19 Jun 2020, Carl Eugen Hoyos wrote:

> Am Mi., 17. Juni 2020 um 01:32 Uhr schrieb Bill Crockett <[hidden email]>:
>>
>> https://ffmpeg.org/ffmpeg-all.html#mpeg2
>>
>> Who maintains the documentation on the above and how can I contact to ask if it is wrong or correct?
>>
>> 16.15 mpeg2
>>
>> MPEG-2 video encoder.
>>
>> 16.15.1 Options
>>
>> profile integer
>>
>>     Select the mpeg2 profile to encode:
>>
>>     ‘422’
>>     ‘main’
>>     ‘ss’
>>
>>         Spatially Scalable
>>     ‘snr’
>>
>>         SNR Scalable
>>     ‘high’
>>     ‘simple’
>>
>> There are no numbers listed, but my guess is the order as is written and would be "main" = 1 and "high" = 4
>>
>> This is wrong and "high should be 1 and "main" should be 4.
>
> Indeed.
>
> I will wait a few more days and fix it then unless somebody beats me.

You can fix it if you want, but I don't know why anybody assumes that the
order of the options and their integer mapping actually matches? Nothing
says it does, and I bet there are other options where it does not. I think
what confused the user here is that the option is mentioned as an
"integer" option when in fact it is a string option from user point of
view. Maybe we shuld simply remove the <integer> part, so the user can use
the string equivavalent as generally advised.

Regards,
Marton
_______________________________________________
ffmpeg-user mailing list
[hidden email]
https://ffmpeg.org/mailman/listinfo/ffmpeg-user

To unsubscribe, visit link above, or email
[hidden email] with subject "unsubscribe".
Reply | Threaded
Open this post in threaded view
|

Re: MPEG2 Profile Options Integer order wrong

Limin Wang
On Fri, Jun 19, 2020 at 08:03:00PM +0200, Marton Balint wrote:

>
>
> On Fri, 19 Jun 2020, Carl Eugen Hoyos wrote:
>
> > Am Mi., 17. Juni 2020 um 01:32 Uhr schrieb Bill Crockett <[hidden email]>:
> > >
> > > https://ffmpeg.org/ffmpeg-all.html#mpeg2
> > >
> > > Who maintains the documentation on the above and how can I contact to ask if it is wrong or correct?
> > >
> > > 16.15 mpeg2
> > >
> > > MPEG-2 video encoder.
> > >
> > > 16.15.1 Options
> > >
> > > profile integer
> > >
> > >     Select the mpeg2 profile to encode:
> > >
> > >     ‘422’
> > >     ‘main’
> > >     ‘ss’
> > >
> > >         Spatially Scalable
> > >     ‘snr’
> > >
> > >         SNR Scalable
> > >     ‘high’
> > >     ‘simple’
> > >
> > > There are no numbers listed, but my guess is the order as is written and would be "main" = 1 and "high" = 4
> > >
> > > This is wrong and "high should be 1 and "main" should be 4.
> >
> > Indeed.
> >
> > I will wait a few more days and fix it then unless somebody beats me.
>
> You can fix it if you want, but I don't know why anybody assumes that the
> order of the options and their integer mapping actually matches? Nothing
> says it does, and I bet there are other options where it does not. I think
> what confused the user here is that the option is mentioned as an "integer"
> option when in fact it is a string option from user point of view. Maybe we
> shuld simply remove the <integer> part, so the user can use the string
> equivavalent as generally advised.

I'll submit a patch to fix the order.  Also the ffpeg -h full can't get the profile
option anyore or it'll show the value for every const string.

>
> Regards,
> Marton
> _______________________________________________
> ffmpeg-user mailing list
> [hidden email]
> https://ffmpeg.org/mailman/listinfo/ffmpeg-user
>
> To unsubscribe, visit link above, or email
> [hidden email] with subject "unsubscribe".

--
Thanks,
Limin Wang
_______________________________________________
ffmpeg-user mailing list
[hidden email]
https://ffmpeg.org/mailman/listinfo/ffmpeg-user

To unsubscribe, visit link above, or email
[hidden email] with subject "unsubscribe".