In Darktable 1.2 (and perhaps other versions), there is a discrepancy between the tooltip for the "core options/demosaicing for zoomed out darkroom mode" preference and the actual behavior of demosaicing in darkroom mode: The tooltip (and the name of the option) suggests that demosaicing algorithm is only overwritten by this option when zoom is less than 1:1. In fact, it is overwritten even when zoom is 1:1 or larger. The relevant code is line 671 in demosaic.c:

  const int qual = get_quality();
  int demosaicing_method = data->demosaicing_method;
  if(piece->pipe->type == DT_DEV_PIXELPIPE_FULL && qual < 2) // only overwrite 
setting if quality << requested and in dr mode
    demosaicing_method = DT_IOP_DEMOSAIC_PPG;

I think this should be:

  const int qual = get_quality();
  int demosaicing_method = data->demosaicing_method;
  if(piece->pipe->type == DT_DEV_PIXELPIPE_FULL && qual < 2*  && 
roi_out->scale<=.99999f*) // only overwrite setting if quality << requested and in dr mode
    demosaicing_method = DT_IOP_DEMOSAIC_PPG;


Naturally, I would be happy to submit this as a patch, but it is perhaps so simple that some existing developer can just take care of it. (Assuming that you agree.)

- Daniel Wagenaar
--
Daniel A. Wagenaar, PhD
Assistant Professor, arriving August 2013
Department of Biological Sciences
McMicken College of Arts and Sciences
University of Cincinnati
Cincinnati, OH 45221
Phone: +1 (626) 395-2577
Email: d...@caltech.edu
Web: http://www.danielwagenaar.net
------------------------------------------------------------------------------
Precog is a next-generation analytics platform capable of advanced
analytics on semi-structured data. The platform includes APIs for building
apps and a phenomenal toolset for data science. Developers can use
our toolset for easy data analysis & visualization. Get a free account!
http://www2.precog.com/precogplatform/slashdotnewsletter
_______________________________________________
darktable-devel mailing list
darktable-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/darktable-devel

Reply via email to