Opened 9 years ago

Closed 8 years ago

#894 closed defect (fixed)

ljpeg: fix losslesss prediction 0 for 24bpp (and 48bpp)

Reported by: ami_stuff Owned by:
Priority: normal Component: avcodec
Version: git-master Keywords: mjpeg ljpeg
Cc: Blocked By:
Blocking: Reproduced by developer: yes
Analyzed by developer: no

Description

$ ffmpeg -i 24_pred0.jpg out2.bmp
ffmpeg version 0.9.0.git-12d8340 Copyright (c) 2000-2012 the FFmpeg developers
  built on Jan  7 2012 02:04:56 with gcc 4.5.2
  configuration: --disable-ffplay --disable-ffserver --disable-ffprobe --disable
-asm --disable-yasm --disable-shared --enable-static
  libavutil      51. 34.100 / 51. 34.100
  libavcodec     53. 54.100 / 53. 54.100
  libavformat    53. 29.100 / 53. 29.100
  libavdevice    53.  4.100 / 53.  4.100
  libavfilter     2. 57.101 /  2. 57.101
  libswscale      2.  1.100 /  2.  1.100
  libswresample   0.  5.100 /  0.  5.100
Input #0, image2, from '24_pred0.jpg':
  Duration: 00:00:00.04, start: 0.000000, bitrate: N/A
    Stream #0:0: Video: mjpeg, bgr24, 640x480, 25 tbr, 25 tbn, 25 tbc
[buffer @ 016c1940] w:640 h:480 pixfmt:bgr24 tb:1/1000000 sar:0/1 sws_param:
Output #0, image2, to 'out2.bmp':
  Metadata:
    encoder         : Lavf53.29.100
    Stream #0:0: Video: bmp, bgr24, 640x480, q=2-31, 200 kb/s, 90k tbn, 25 tbc
Stream mapping:
  Stream #0:0 -> #0:0 (mjpeg -> bmp)
Press [q] to stop, [?] for help
frame=    1 fps=  0 q=0.0 Lsize=       0kB time=00:00:00.04 bitrate=   0.0kbits/
s
video:900kB audio:0kB global headers:0kB muxing overhead -100.000000%

Attachments (2)

24_pred0.jpg (1.3 MB ) - added by ami_stuff 9 years ago.
48_pred0.rar (1.0 MB ) - added by ami_stuff 9 years ago.

Change History (7)

by ami_stuff, 9 years ago

Attachment: 24_pred0.jpg added

by ami_stuff, 9 years ago

Attachment: 48_pred0.rar added

comment:1 by Carl Eugen Hoyos, 9 years ago

Component: undeterminedavcodec
Keywords: mjpeg ljpeg added
Reproduced by developer: set
Status: newopen
Version: unspecifiedgit-master

There is an ugly vertical line at the left side of the images.

comment:2 by Michael Niedermayer, 9 years ago

Resolution: fixed
Status: openclosed

Fix on ML

comment:3 by Carl Eugen Hoyos, 9 years ago

Resolution: fixed
Status: closedreopened

Could you clarify if 48_pred0.jpg really is 48bit?
Is there an application that can be used to show?

in reply to:  3 comment:4 by ami_stuff, 9 years ago

Replying to cehoyos:

Could you clarify if 48_pred0.jpg really is 48bit?

afaict, yes. when you use PIX_FMT_BGR48LE then only a half of the output will be filled with image, so probably some code needs to be added to ljpeg decoder to correctly support it.

Is there an application that can be used to show?

http://www.myviewpad.com/index.php

comment:5 by Michael Niedermayer, 8 years ago

Resolution: fixed
Status: reopenedclosed
Note: See TracTickets for help on using tickets.