Re: [FFmpeg-devel] libavcodec/hapdec : add support for HapAphaOnly decoding

2017-10-05 Thread Carl Eugen Hoyos
2017-09-28 11:14 GMT+02:00 Tom Butterworth : > On Wed, 27 Sep 2017 at 16:52 Carl Eugen Hoyos wrote: >> As long as FFmpeg does not have a native alpha-only >> pix_fmt (I am not sure this would have any user-visible >> advantage), single-layer 8bit formats should be decoded >> as gray8. > > This wo

Re: [FFmpeg-devel] libavcodec/hapdec : add support for HapAphaOnly decoding

2017-09-28 Thread Tom Butterworth
On Wed, 27 Sep 2017 at 16:52 Carl Eugen Hoyos wrote: > 2017-09-27 15:20 GMT+02:00 Martin Vignali : > > 2017-09-27 13:25 GMT+02:00 Tom Butterworth : > > > >> > >> > >> > On 27 Sep 2017, at 11:59, Martin Vignali > >> wrote: > >> > > >> >> > >> >>> +ctx->tex_fun = ctx->dxtc.rgtc1u_block; >

Re: [FFmpeg-devel] libavcodec/hapdec : add support for HapAphaOnly decoding

2017-09-27 Thread Carl Eugen Hoyos
2017-09-27 15:20 GMT+02:00 Martin Vignali : > 2017-09-27 13:25 GMT+02:00 Tom Butterworth : > >> >> >> > On 27 Sep 2017, at 11:59, Martin Vignali >> wrote: >> > >> >> >> >>> +ctx->tex_fun = ctx->dxtc.rgtc1u_block; >> >>> +avctx->pix_fmt = AV_PIX_FMT_RGB0; >> >> >> >> The rgtc1u_bloc

Re: [FFmpeg-devel] libavcodec/hapdec : add support for HapAphaOnly decoding

2017-09-27 Thread Martin Vignali
2017-09-27 15:33 GMT+02:00 Tom Butterworth : > > > > On 27 Sep 2017, at 14:20, Martin Vignali > wrote: > > > > 2017-09-27 13:25 GMT+02:00 Tom Butterworth bangno...@gmail.com>>: > > > >> > >> > >>> On 27 Sep 2017, at 11:59, Martin Vignali > >> wrote: > >>> > > > +ctx->tex_fun =

Re: [FFmpeg-devel] libavcodec/hapdec : add support for HapAphaOnly decoding

2017-09-27 Thread Tom Butterworth
> On 27 Sep 2017, at 14:20, Martin Vignali wrote: > > 2017-09-27 13:25 GMT+02:00 Tom Butterworth >: > >> >> >>> On 27 Sep 2017, at 11:59, Martin Vignali >> wrote: >>> > +ctx->tex_fun = ctx->dxtc.rgtc1u_block; > +avctx->pix_fmt = AV

Re: [FFmpeg-devel] libavcodec/hapdec : add support for HapAphaOnly decoding

2017-09-27 Thread Martin Vignali
2017-09-27 13:25 GMT+02:00 Tom Butterworth : > > > > On 27 Sep 2017, at 11:59, Martin Vignali > wrote: > > > >> > >>> +ctx->tex_fun = ctx->dxtc.rgtc1u_block; > >>> +avctx->pix_fmt = AV_PIX_FMT_RGB0; > >> > >> The rgtc1u_block function places the single channel value in every > cha

Re: [FFmpeg-devel] libavcodec/hapdec : add support for HapAphaOnly decoding

2017-09-27 Thread Tom Butterworth
> On 27 Sep 2017, at 11:59, Martin Vignali wrote: > >> >>> +ctx->tex_fun = ctx->dxtc.rgtc1u_block; >>> +avctx->pix_fmt = AV_PIX_FMT_RGB0; >> >> The rgtc1u_block function places the single channel value in every channel >> except the alpha channel >> The pixel format you have c

Re: [FFmpeg-devel] libavcodec/hapdec : add support for HapAphaOnly decoding

2017-09-27 Thread Martin Vignali
> > > +ctx->tex_fun = ctx->dxtc.rgtc1u_block; > > +avctx->pix_fmt = AV_PIX_FMT_RGB0; > > The rgtc1u_block function places the single channel value in every channel > except the alpha channel > The pixel format you have chosen is one which disregards the alpha channel > > The output

Re: [FFmpeg-devel] libavcodec/hapdec : add support for HapAphaOnly decoding

2017-09-27 Thread Tom Butterworth
Hey Martin > +ctx->tex_fun = ctx->dxtc.rgtc1u_block; > +avctx->pix_fmt = AV_PIX_FMT_RGB0; The rgtc1u_block function places the single channel value in every channel except the alpha channel The pixel format you have chosen is one which disregards the alpha channel The output of

Re: [FFmpeg-devel] libavcodec/hapdec : add support for HapAphaOnly decoding

2017-09-25 Thread Carl Eugen Hoyos
2017-09-25 21:25 GMT+02:00 Martin Vignali : > 2017-09-25 20:19 GMT+02:00 Carl Eugen Hoyos : > >> 2017-09-25 9:48 GMT+02:00 Martin Vignali : >> >> >> >> Sorry if I misunderstand: >> >> If the format only contains one layer, why is the pix_fmt >> >> rgb0 and not gray8? >> >> > I follow the existing t

Re: [FFmpeg-devel] libavcodec/hapdec : add support for HapAphaOnly decoding

2017-09-25 Thread Martin Vignali
2017-09-25 20:19 GMT+02:00 Carl Eugen Hoyos : > 2017-09-25 9:48 GMT+02:00 Martin Vignali : > >> > >> Sorry if I misunderstand: > >> If the format only contains one layer, why is the pix_fmt > >> rgb0 and not gray8? > > > I follow the existing texture dsp design, who output RGB0 > > or RGBA dependi

Re: [FFmpeg-devel] libavcodec/hapdec : add support for HapAphaOnly decoding

2017-09-25 Thread Carl Eugen Hoyos
2017-09-25 9:48 GMT+02:00 Martin Vignali : >> >> Sorry if I misunderstand: >> If the format only contains one layer, why is the pix_fmt >> rgb0 and not gray8? > I follow the existing texture dsp design, who output RGB0 > or RGBA depending of the texture. > But you're right, the format is gray only

Re: [FFmpeg-devel] libavcodec/hapdec : add support for HapAphaOnly decoding

2017-09-25 Thread Martin Vignali
> > Sorry if I misunderstand: > If the format only contains one layer, why is the pix_fmt > rgb0 and not gray8? > > Carl Eugen > > > Hello, I follow the existing texture dsp design, who output RGB0 or RGBA depending of the texture. But you're right, the format is gray only, Martin ___

Re: [FFmpeg-devel] libavcodec/hapdec : add support for HapAphaOnly decoding

2017-09-24 Thread Carl Eugen Hoyos
2017-09-23 21:49 GMT+02:00 Martin Vignali : > Following this doc : > https://github.com/Vidvox/hap/blob/master/documentation/HapVideoDRAFT.md > > Hap Alpha Only, is only an rgtc1 texture with or without > snappy compression Sorry if I misunderstand: If the format only contains one layer, why is t

Re: [FFmpeg-devel] libavcodec/hapdec : add support for HapAphaOnly decoding

2017-09-24 Thread Michael Niedermayer
On Sat, Sep 23, 2017 at 09:52:50PM +0200, Martin Vignali wrote: > Sorry, i made a mistake in the previous patchs > > Correct patchs in attach will apply thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Those who would give up essential Liberty, to purch

[FFmpeg-devel] libavcodec/hapdec : add support for HapAphaOnly decoding

2017-09-23 Thread Martin Vignali
Hello, Following this doc : https://github.com/Vidvox/hap/blob/master/documentation/HapVideoDRAFT.md Hap Alpha Only, is only an rgtc1 texture with or without snappy compression Samples can found here (these samples have been create using the second texture of an hapqa file (generate with Hap Qu

Re: [FFmpeg-devel] libavcodec/hapdec : add support for HapAphaOnly decoding

2017-09-23 Thread Martin Vignali
Sorry, i made a mistake in the previous patchs Correct patchs in attach Martin 0001-libavformat-add-mov-dataformat-tag-for-HapAlphaOnly-.patch Description: Binary data 0002-libavcodec-hapdec-add-support-HapAlphaOnly.patch Description: Binary data __