Very glad to communicate on IRC a bit. Writing out some other standing
issues, but happy to bring questions back to #darktable.

On Mon, May 5, 2014, at 08:28 AM, Tobias Ellinghaus wrote:
> > - design decisions I made may be flawed, especially setting up the
> > UI/parameters for x-trans in the demosaic iop
> 
> We will see ...

In iop/demosaic.c, there needs to be a way to shoehorn separate Bayer &
X-Trans demosaic methods into dt_iop_demosaic_method_t and the UI. For
now Bayer methods (as always) are numbered 0 (PPG) or 1 (AMaZE) while
X-Trans methods get a mask of 1024 and hence are 1024 (linear), 1025
(VNG), 1026 (Markesteijn 1 pass) or 1027 (Markesteijn 3 pass). Also for
now in the UI there are two different comboboxes, one for Bayer, one for
X-Trans demosaic methods. Depending on the raw file mosaic, the
appropriate widget gets shown.

One big catch here is that in darktableconfig.xml, option
plugins/darkroom/demosaic/quality currently defaults to "at most PPG
(reasonable)". To be correct for X-Trans, this should say something more
like, "at most PPG/VNG (reasonable)". Is it worth altering something
across all configurations just for pedantry, though?

> So you based your work on libraw? I guess for the moment that is ok. Most
> work 
> probably went into our code I guess, so changing the input over to
> rawspeed 
> once that supports X-Trans shouldn't be much work?

Am based off of libraw for now as it can read the x-trans format (via
its dcraw code) in recent versions. This is great, but also means a huge
part of the x-trans branch is an update to LibRaw 0.16.0, as the 0.14.7
version in Darktable can't deal with x-trans. It sounds as the rawspeed
is working on x-trans support but it's not yet released? It should be
easy to switch over when that happens.

------------------------------------------------------------------------------
Is your legacy SCM system holding you back? Join Perforce May 7 to find out:
• 3 signs your SCM is hindering your productivity
• Requirements for releasing software faster
• Expert tips and advice for migrating your SCM now
http://p.sf.net/sfu/perforce
_______________________________________________
darktable-devel mailing list
darktable-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/darktable-devel

Reply via email to