A590IS porting... - page 65 - DryOS Development - CHDK Forum
« previous next »
Pages: 1 ... 60 61 62 63 64 [65] 66 67 68 69 70 ... 97   Go Down

A590IS porting...

  • 968 Replies
  • 418684 Views
*

Offline reyalp

  • ******
  • 14125
Re: A590IS porting...
« Reply #640 on: 25 / October / 2009, 21:58:09 »
Advertisements
Quote from: RaduP on 25 / October / 2009, 21:44:25
That's normal, those are called inactive pixels.
If the sensor active area is set right, you shouldn't see them in DNG most dng viewers/converters. They are a normal part of the CHDK RAW, although your raw converter should know, or let you specify the active area.

To determine the active area, you should look at a CHDK CRW, which includes the entire sensor area. The inactive area should be black (i think it's zero, not blacklevel but I'm not 100% sure this is always true.) You probably want to remove your badpixel.bin file before doing this, because AFAIK the badpixel code does not honor the active area settings and would interpolate the border pixels to non-zero values.
Logged
Don't forget what the H stands for.
*

Offline dvip

  • ****
  • 451
Re: A590IS porting...
« Reply #641 on: 25 / October / 2009, 23:15:16 »
reyaldp,
I see them in the DNG too, and it looks similar in a
CHDK CRW with and without badpixel.bin present.

It's like a dark line 10 pixels thick and another one
parallel to it about 5 pixels which somehow blends with the
picture.

A 15 pixels thick frame that needs to be trimmed :)

Could it be a sensor dimensions error in the active area
defined in camera.h or somewhere in the source code for
the A590IS? Is there any way to trim the borders a bit there? About 15 pixels?

Regarding the raw converter I use dcraw.exe, but you can
see the problem in  IrfanView, FastStone and UFraw.



Logged
*

Offline reyalp

  • ******
  • 14125
Re: A590IS porting...
« Reply #642 on: 25 / October / 2009, 23:26:23 »
Quote from: dvip on 25 / October / 2009, 23:15:16
reyaldp,
ould it be a sensor dimensions error in the active area defined in camera.h or somewhere in the source code for the A590IS? Is there any way to trim the borders a bit there? About 15 pixels?
Yes, this is all in camera.h, and considering the many other problems in the a590 port, I wouldn't be at all surprised if it were wrong.
Logged
Don't forget what the H stands for.
*

Offline RaduP

  • *****
  • 926
Re: A590IS porting...
« Reply #643 on: 25 / October / 2009, 23:36:20 »
I also have that border on my SD980, I thought it was normal. There are some things in camera.h where you can define the crop area, but I didn't investigate their use. I'll take a  look in the DNG/RAW code and see what I can find, when I have some time.
Logged
*

Offline dvip

  • ****
  • 451
Re: A590IS porting...
« Reply #644 on: 26 / October / 2009, 00:05:03 »
@reyalp, I hope is just a simple sensor re-dimensions in
camera.h and a re-compile.

@RaduP, yes, there are some #define in camera.h I saw way
back.   I hope it is possible to do a little trim there in the sensor
active area, or that it can be set right.


Logged
*

Offline reyalp

  • ******
  • 14125
Re: A590IS porting...
« Reply #645 on: 26 / October / 2009, 00:24:42 »
Quote from: dvip on 26 / October / 2009, 00:05:03
@reyalp, I hope is just a simple sensor re-dimensions in
camera.h and a re-compile.
Yes, just needs someone who has the camera (or a raw from it) to figure out the correct dimensions.

If you don't know what to do, I think Whim has one, maybe he can do it.
Logged
Don't forget what the H stands for.
*

Offline RaduP

  • *****
  • 926
Re: A590IS porting...
« Reply #646 on: 26 / October / 2009, 03:10:33 »
Well, modifying
    #define CAM_ACTIVE_AREA_X1
    #define CAM_ACTIVE_AREA_Y1
    #define CAM_ACTIVE_AREA_X2
    #define CAM_ACTIVE_AREA_Y2
helped.
However, I am a little disappointed that the file size did not change :/
Logged
*

Offline dvip

  • ****
  • 451
Re: A590IS porting...
« Reply #647 on: 26 / October / 2009, 09:57:40 »
@reyalp, I don't have a way to recompile it.
Nice to know that Whim has a Canon A590IS.

@RaduP, I hope these changes can be added to
the next A590IS build. It will make using RAW
even better. No more trim in PP, no more ugly
frame around the CRW/DNG.


Logged
*

Offline RaduP

  • *****
  • 926
Re: A590IS porting...
« Reply #648 on: 26 / October / 2009, 13:21:25 »
Those changes are camera dependent, so they won't work on any camera. Compiling CHDK is relatively easy, it doesn't require programming experience or anything.
Logged
*

Offline whim

  • ******
  • 2046
  • A495/590/620/630 ixus70/115/220/230/300/870 S95
Re: A590IS porting...
« Reply #649 on: 26 / October / 2009, 13:42:35 »
Quote
Compiling CHDK is relatively easy, it doesn't require programming experience or anything.
i would agree with that  :D

@dvip
I noticed that there's several other cams that have an 1/2.5" / 8 MP sensor, maybe we could have a shot first
at replacing the CAM_ACTIVE_blahs with those from ixus80, A720, etc ? Are you willing to test a couple of
binaries, if i compile & upload them ?

HTH,

wim


Logged
Pages: 1 ... 60 61 62 63 64 [65] 66 67 68 69 70 ... 97   Go Up
« previous next »
 

Related Topics


SimplePortal © 2008-2014, SimplePortal