Signed-off-by: Paras Chadha
---
tests/fate/demux.mak | 2 +-
tests/fate/fits.mak | 6 +++---
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/tests/fate/demux.mak b/tests/fate/demux.mak
index 261b004..68d4b6a 100644
--- a/tests/fate/demux.mak
+++ b/tests/fate/demux.mak
@@ -36,7
Signed-off-by: Paras Chadha
---
tests/fate/fits.mak | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tests/fate/fits.mak b/tests/fate/fits.mak
index bc1b771..704f655 100644
--- a/tests/fate/fits.mak
+++ b/tests/fate/fits.mak
@@ -1,3 +1,4 @@
+tests/data/fits-multi.fits: $(TARGET_SAMPLES)/gif
Signed-off-by: Paras Chadha
---
libavcodec/fitsdec.c | 34 ++
1 file changed, 18 insertions(+), 16 deletions(-)
diff --git a/libavcodec/fitsdec.c b/libavcodec/fitsdec.c
index b075381..c72415f 100644
--- a/libavcodec/fitsdec.c
+++ b/libavcodec/fitsdec.c
@@ -205,9
On Fri, Sep 15, 2017 at 10:59 PM, James Almer wrote:
> On 9/12/2017 9:56 PM, James Almer wrote:
> > On 9/3/2017 12:05 PM, Paras Chadha wrote:
> >> Signed-off-by: Paras Chadha
> >> ---
> >> This fixes the failed fate tests on ppc64be:
> >> http://f
Signed-off-by: Paras Chadha
---
This fixes the failed fate tests on ppc64be:
http://fate.ffmpeg.org/report.cgi?time=20170903050743&slot=ppc64be-RHEL7.0-gcc-4.8.5-ibmcrl
libavcodec/fitsdec.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/libavcodec/fitsdec
Signed-off-by: Paras Chadha
---
This fixes the failed fate tests on msvc12:
http://fate.ffmpeg.org/report.cgi?time=20170901190511&slot=x86_32-msvc12-windows-native
http://fate.ffmpeg.org/report.cgi?time=20170901211336&slot=x86_64-msvc12-windows-native
Tests failed because snprintf was
Signed-off-by: Paras Chadha
---
This fixes the failed fate tests on mingw32:
http://fate.ffmpeg.org/report.cgi?time=20170902182311&slot=x86_32-mingw-w64-dll-windows-native
libavcodec/fitsdec.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/libavcodec/fitsdec
On Fri, Sep 1, 2017 at 5:26 AM, James Almer wrote:
> On 8/27/2017 2:45 AM, Paras Chadha wrote:
> > Signed-off-by: Paras Chadha
> > ---
> > All other FITS patches need to be applied before this one.
> >
> > tests/fate/demux.mak
Signed-off-by: Paras Chadha
---
All other FITS patches need to be applied before this one.
tests/fate/demux.mak| 3 +-
tests/fate/fits.mak | 57 ++---
tests/ref/fate/fitsdec-bitpix-32| 6
tests/ref/fate/fitsdec
Signed-off-by: Paras Chadha
---
Add FATE coverage
libavformat/Makefile | 1 +
libavformat/allformats.c | 1 +
libavformat/fitsdec.c | 231 ++
libavformat/version.h | 2 +-
tests/fate/demux.mak | 3 +
tests/ref/fate/fits
Signed-off-by: Paras Chadha
---
Add FATE coverage
Changelog| 1 +
doc/general.texi | 2 +-
libavcodec/Makefile | 1 +
libavcodec/allcodecs.c | 2 +-
libavcodec/fitsenc.c | 130
Signed-off-by: Paras Chadha
---
Add FATE coverage
libavformat/Makefile | 1 +
libavformat/allformats.c | 2 +-
libavformat/fitsenc.c| 183 +++
libavformat/img2enc.c| 2 +
tests/fate/avformat.mak | 1 +
tests/lavf-regression.sh
Signed-off-by: Paras Chadha
---
Add FATE coverage
Changelog | 1 +
doc/general.texi | 2 +
libavcodec/Makefile| 2 +
libavcodec/allcodecs.c | 1 +
libavcodec/avcodec.h | 1 +
libavcodec/codec_desc.c| 7
Signed-off-by: Paras Chadha
---
Added a new line in img2enc to add support for writing frames
into separate FITS files in a fashion similar to gif.
The following command can be used for that:
ffmpeg -i INPUT -c:v fits -f image2 "out%d.fits"
libavformat/Makefile | 1 +
l
Signed-off-by: Paras Chadha
---
Moved header writing code to muxer
Made the changes suggested
Changelog | 1 +
doc/general.texi | 2 +-
libavcodec/Makefile| 1 +
libavcodec/allcodecs.c | 2 +-
libavcodec/fitsenc.c | 130
Signed-off-by: Paras Chadha
---
Made the changes suggested.
Added a new file fits.c with common code
Changelog | 1 +
doc/general.texi| 2 +
libavcodec/Makefile | 2 +
libavcodec/allcodecs.c | 1 +
libavcodec/avcodec.h| 1 +
libavcodec/codec_desc.c | 7
On Sun, Aug 6, 2017 at 5:27 PM, Nicolas George wrote:
> Le duodi 12 thermidor, an CCXXV, Paras Chadha a écrit :
> > But i am returning the size from this function. If i make them unsigned,
> > how will i return errors which are negative integers.
> > One thing i can do
Signed-off-by: Paras Chadha
---
Made changes suggested.
Refactored Code
libavformat/Makefile | 1 +
libavformat/allformats.c | 1 +
libavformat/fitsdec.c| 231 +++
libavformat/version.h| 2 +-
4 files changed, 234 insertions(+), 1
On Fri, Jul 28, 2017 at 3:26 PM, Nicolas George wrote:
> Le decadi 10 thermidor, an CCXXV, Paras Chadha a écrit :
> > Signed-off-by: Paras Chadha
> > ---
> > Changelog | 1 +
> > doc/general.texi| 2 +
> > libavcodec/Makefile |
On Fri, Jul 28, 2017 at 3:26 PM, Nicolas George wrote:
> Le decadi 10 thermidor, an CCXXV, Paras Chadha a écrit :
> > Signed-off-by: Paras Chadha
> > ---
> >
> > Made all the changes suggested.
>
> Nice. There are a few nitpicks, but I like these versions much
On Fri, Jul 28, 2017 at 1:44 PM, Clément Bœsch wrote:
> On Fri, Jul 28, 2017 at 12:32:59AM +0530, Paras Chadha wrote:
> [...]
> > +static int fits_probe(AVProbeData *p)
> > +{
> > +const uint8_t *b = p->buf;
> > +
> > +if (AV_RB64(b) == 0x53494d5
On Fri, Jul 21, 2017 at 7:59 PM, Nicolas George wrote:
> Le tridi 3 thermidor, an CCXXV, Paras Chadha a écrit :
> > > > AVOutputFormat ff_image2pipe_muxer = {
> > > > .name = "image2pipe",
> > > > .long_name
On Mon, Jul 24, 2017 at 3:42 AM, Carl Eugen Hoyos
wrote:
> 2017-07-21 20:16 GMT+02:00 Reimar Döffinger :
> > On 21.07.2017, at 16:26, Paras Chadha wrote:
> >
> >> On Fri, Jul 21, 2017 at 3:27 AM, Carl Eugen Hoyos
> >> wrote:
> >>
>
Signed-off-by: Paras Chadha
---
Made all the changes suggested.
libavformat/Makefile | 1 +
libavformat/allformats.c | 1 +
libavformat/fitsdec.c| 236 +++
libavformat/version.h| 2 +-
4 files changed, 239 insertions(+), 1 deletion
Signed-off-by: Paras Chadha
---
Changelog | 1 +
doc/general.texi| 2 +
libavcodec/Makefile | 1 +
libavcodec/allcodecs.c | 1 +
libavcodec/avcodec.h| 1 +
libavcodec/codec_desc.c | 7 +
libavcodec/fits.h | 79 +++
libavcodec/fitsdec.c
On Fri, Jul 21, 2017 at 3:27 AM, Carl Eugen Hoyos
wrote:
> 2017-07-20 21:46 GMT+02:00 Paras Chadha :
>
> > +case AV_PIX_FMT_RGB24:
> > +case AV_PIX_FMT_RGBA:
> > +for (k = 0; k < naxis3; k++) {
> > +
On Fri, Jul 21, 2017 at 1:45 AM, James Almer wrote:
> On 7/20/2017 4:46 PM, Paras Chadha wrote:
> > diff --git a/libavformat/img2enc.c b/libavformat/img2enc.c
> > index 1297b1a..25283cc 100644
> > --- a/libavformat/img2enc.c
> > +++ b/libavformat/img2enc.c
> > @
Signed-off-by: Paras Chadha
---
Made the changes suggested.
doc/general.texi | 2 +-
libavcodec/Makefile| 1 +
libavcodec/allcodecs.c | 2 +-
libavcodec/fitsenc.c | 237 +
libavformat/img2enc.c | 1 +
5 files changed, 241
ure this is entirely the case here, for example the first
> > image would have a different header it looks like from the muxer
> > patch.
>
> According to this mail from Paras Chadha:
> https://ffmpeg.org/pipermail/ffmpeg-devel/2017-July/213180.html
> there is no global header. A
On Thu, Jul 20, 2017 at 11:19 PM, Nicolas George wrote:
> Hi.
>
> Le duodi 2 thermidor, an CCXXV, Paras Chadha a écrit :
> > +size = abs(header->bitpix) >> 3;
>
> > +size *= header->naxisn[i];
>
> Can you explain why PCOUNT and GCOUNT are no
Signed-off-by: Paras Chadha
---
Fixed an issue with RGB48 and RGBA64 images.
Also simplified the RGB decoding code.
Changelog | 1 +
doc/general.texi| 2 +
libavcodec/Makefile | 1 +
libavcodec/allcodecs.c | 1 +
libavcodec/avcodec.h| 1 +
libavcodec
On Sun, Jul 16, 2017 at 10:06 PM, Paul B Mahol wrote:
> Can you give link to file which holds more than one FITS image?
>
Yes, here is the file: https://fits.gsfc.nasa.gov/samples/EUVEngc4151imgx.
fits
> ___
> ffmpeg-devel mailing list
> ffmpeg-devel
On Sun, Jul 16, 2017 at 6:29 PM, Derek Buitenhuis <
derek.buitenh...@gmail.com> wrote:
> On 7/16/2017 1:31 PM, Paras Chadha wrote:
> > Signed-off-by: Paras Chadha
> > ---
> > doc/general.texi | 2 +
> > libavcodec/Makefile| 1 +
> > libavco
On Fri, Jul 14, 2017 at 11:15 PM, Nicolas George wrote:
> Le tridi 23 messidor, an CCXXV, Paras Chadha a écrit :
> > Hi, I have attached the dump of a FITS File containing 5 images and 4
> > binary table xtensions. The dump is taken using fdump utility. Please
> take
> >
Signed-off-by: Paras Chadha
---
doc/general.texi | 2 +
libavcodec/Makefile| 1 +
libavcodec/allcodecs.c | 1 +
libavcodec/fitsenc.c | 238 +
libavformat/img2enc.c | 1 +
5 files changed, 243 insertions(+)
create mode 100644
On Sat, Jul 15, 2017 at 12:13 PM, Reimar Döffinger wrote:
> On 14.07.2017, at 19:04, Paras Chadha wrote:
>
> > +if (bitpix == 16) {
> > +for (j = 0; j < avctx->width; j++) {
> > +bytestream2_put_b
Signed-off-by: Paras Chadha
---
doc/general.texi | 2 +
libavcodec/Makefile| 1 +
libavcodec/allcodecs.c | 1 +
libavcodec/fitsenc.c | 259 +
libavformat/img2enc.c | 1 +
5 files changed, 264 insertions(+)
create mode 100644
On Tue, Jul 11, 2017 at 3:41 PM, Nicolas George wrote:
> Le decadi 20 messidor, an CCXXV, Reimar Döffinger a écrit :
> > I don't think that's a correct description then.
> > First, the format is made to change and be extended, so what is
> > true now might not stay true.
> > But also the images c
On Tue, Jul 11, 2017 at 3:41 PM, Nicolas George wrote:
> Le decadi 20 messidor, an CCXXV, Reimar Döffinger a écrit :
> > I don't think that's a correct description then.
> > First, the format is made to change and be extended, so what is
> > true now might not stay true.
> > But also the images c
On Tue, Jul 4, 2017 at 10:20 PM, Nicolas George wrote:
> Le sextidi 16 messidor, an CCXXV, Paras Chadha a écrit :
> > There is no global header.
> >
> > Basically FITS files can have multiple images.
>
>
> Thanks for all the details. When there are several images
On Tue, Jul 4, 2017 at 2:51 PM, Nicolas George wrote:
> Le quartidi 14 messidor, an CCXXV, Paras Chadha a écrit :
> > Filled buf with 0 to prevent overfow
> > Also added checks for integer overflow
> >
> > Signed-off-by: Paras Chadha
> > ---
&g
On Tue, Jul 4, 2017 at 4:12 AM, Reimar Döffinger
wrote:
>
> > +static int64_t find_size(AVIOContext * pb, FITSContext * fits)
> > +{
> > +int bitpix, naxis, dim_no, i, naxisn[999], groups=0;
> > +int64_t header_size = 0, data_size=0, ret, pcount=0, gcount=1, d;
> > +char buf[81], c;
>
On Tue, Jul 4, 2017 at 2:57 PM, Nicolas George wrote:
> Le quartidi 14 messidor, an CCXXV, Paras Chadha a écrit :
> > Made all the changes suggested
> > Added a new function which reads a header FITS line safely. It also
> makes it more modular
> > Added an option for the
On Mon, Jul 3, 2017 at 3:56 AM, Moritz Barsnick wrote:
> On Sun, Jul 02, 2017 at 20:48:17 +0530, Paras Chadha wrote:
> > +int64_t header_size = 0, data_size=0, ret, pcount=0, gcount=1, d;
> [...]
> > +header_size += 80;
> [...]
> > +header_size += 80;
&
Filled buf with 0 to prevent overfow
Also added checks for integer overflow
Signed-off-by: Paras Chadha
---
libavformat/Makefile | 1 +
libavformat/allformats.c | 1 +
libavformat/fitsdec.c| 224 +++
libavformat/version.h| 2 +-
4
On Fri, Jun 30, 2017 at 6:38 PM, Nicolas George wrote:
> Hi. A few technical / cosmetic remarks below. I do not know the FITS
> format itself more than in passing.
>
> Le duodi 12 messidor, an CCXXV, Paras Chadha a écrit :
> > Made the changes suggested above
> >
> &g
Made all the changes suggested
Added a new function which reads a header FITS line safely. It also makes it
more modular
Added an option for the user to enter the value to be used in place of BLANK
pixels
Refactored code using macros to make it short
Signed-off-by: Paras Chadha
---
Changelog
Made the changes suggested above
Signed-off-by: Paras Chadha
---
Changelog | 1 +
doc/general.texi| 2 +
libavcodec/Makefile | 1 +
libavcodec/allcodecs.c | 1 +
libavcodec/avcodec.h| 1 +
libavcodec/codec_desc.c | 8 +
libavcodec/fitsdec.c| 580
Added code to export metadata as frame metadata
Signed-off-by: Paras Chadha
---
Changelog | 1 +
doc/general.texi| 2 +
libavcodec/Makefile | 1 +
libavcodec/allcodecs.c | 1 +
libavcodec/avcodec.h| 1 +
libavcodec/codec_desc.c | 8 +
libavcodec
On Tue, Jun 20, 2017 at 2:44 PM, Paul B Mahol wrote:
> On 6/19/17, Paras Chadha wrote:
> > Added support for XTENSION keyword
> >
> > Signed-off-by: Paras Chadha
> > ---
> > Changelog | 1 +
> > doc/general.texi|
Above changes done. Also fixed an issue with BLANK keyword.
Signed-off-by: Paras Chadha
---
Changelog | 1 +
doc/general.texi| 2 +
libavcodec/Makefile | 1 +
libavcodec/allcodecs.c | 1 +
libavcodec/avcodec.h| 1 +
libavcodec/codec_desc.c | 8
Added support for XTENSION keyword
Signed-off-by: Paras Chadha
---
Changelog | 1 +
doc/general.texi| 2 +
libavcodec/Makefile | 1 +
libavcodec/allcodecs.c | 1 +
libavcodec/avcodec.h| 1 +
libavcodec/codec_desc.c | 8 +
libavcodec/fitsdec.c| 518
Signed-off-by: Paras Chadha
---
Changelog| 2 +-
libavformat/Makefile | 1 +
libavformat/allformats.c | 1 +
libavformat/fitsdec.c| 207 +++
libavformat/version.h| 2 +-
5 files changed, 211 insertions(+), 2
On Tue, Jun 13, 2017 at 7:45 PM, Paul B Mahol wrote:
> On 6/13/17, Paras Chadha wrote:
> > Above mentioned changes have been done.
> >
> > Signed-off-by: Paras Chadha
> > ---
> > Changelog | 1 +
> > doc/general.texi|
Above mentioned changes have been done.
Signed-off-by: Paras Chadha
---
Changelog | 1 +
doc/general.texi| 2 +
libavcodec/Makefile | 1 +
libavcodec/allcodecs.c | 1 +
libavcodec/avcodec.h| 1 +
libavcodec/codec_desc.c | 8 +
libavcodec/fitsdec.c
Hi,
Any other changes to be made to get this patch applied?
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
On Fri, Jun 9, 2017 at 1:26 AM, Moritz Barsnick wrote:
> On Thu, Jun 08, 2017 at 22:34:15 +0530, Paras Chadha wrote:
> > +t64 = (((uint64_t) ptr8[0]) << 56) | (((uint64_t) ptr8[1])
> << 48) | (((uint64_t) ptr8[2]) << 40) | (((uint64_t) ptr8[3]) <&l
On Fri, Jun 9, 2017 at 4:19 PM, Michael Niedermayer
wrote:
> On Thu, Jun 08, 2017 at 10:34:15PM +0530, Paras Chadha wrote:
> > It supports all 2-d images alongwith bzero, bscale and blank keywords.
> > RGBA images are supported as NAXIS3 = 3 or 4 i.e. Planes in RGBA order.
>
All the changes mentioned above have been done.
Signed-off-by: Paras Chadha
---
Changelog | 1 +
doc/general.texi| 2 +
libavcodec/Makefile | 1 +
libavcodec/allcodecs.c | 1 +
libavcodec/avcodec.h| 1 +
libavcodec/codec_desc.c | 8 +
libavcodec
On Fri, Jun 9, 2017 at 3:08 AM, Martin Vignali
wrote:
> Hello,
>
> You can create a FipsContext struct to store decoding data
> instead of using Fips Header struct
> (you can take a look to other image decoder)
>
> Also you should need to split the patch for libavcodec part and libavformat
> part
It supports all 2-d images alongwith bzero, bscale and blank keywords.
RGBA images are supported as NAXIS3 = 3 or 4 i.e. Planes in RGBA order. Also
CTYPE = 'RGB ' should be present.
It currently does not support XTENSION keyword.
Signed-off-by: Paras Chadha
---
Changelog
On Sat, Mar 11, 2017 at 5:47 PM, Michael Niedermayer wrote:
> On Sat, Mar 11, 2017 at 05:33:30PM +0530, Paras Chadha wrote:
> > Here is the patch with all the changes mentioned above
> >
> > Signed-off-by: Paras
> > ---
> > Changelog | 1 +
&
Xpm decoder was added
Also added xpm_pipe demuxer with its probe function
Signed-off-by: Paras Chadha
---
Changelog| 1 +
doc/general.texi | 2 +
libavcodec/Makefile | 1 +
libavcodec/allcodecs.c | 1 +
libavcodec/avcodec.h | 1 +
libavcodec
* XPM image format
+ *
+ * Copyright (c) 2012 Paul B Mahol
+ * Copyright (c) 2016 Paras Chadha
+ *
+ * This file is part of FFmpeg.
+ *
+ * FFmpeg is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Softw
On Sat, Mar 11, 2017 at 5:40 AM, Moritz Barsnick wrote:
>
> > +static int color_table_compare(const void *lhs, const void *rhs)
> > +{
> > +return av_strcasecmp(lhs, ((const ColorEntry *)rhs)->name);
> > +}
> > +
> > +static const ColorEntry color_table[] = {
> > +{ "AliceBlue",
-- /dev/null
+++ b/libavcodec/xpmdec.c
@@ -0,0 +1,430 @@
+/*
+ * XPM image format
+ *
+ * Copyright (c) 2012 Paul B Mahol
+ * Copyright (c) 2016 Paras Chadha
+ *
+ * This file is part of FFmpeg.
+ *
+ * FFmpeg is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesse
66 matches
Mail list logo
