Merge pull request #3147 from Beep6581/xtrans_fix

Recalculate XTrans matrix after cropping by camconst settings, fixes …
This commit is contained in:
Ingo Weyrich
2016-02-12 19:57:14 +01:00
3 changed files with 32 additions and 10 deletions

View File

@@ -927,6 +927,13 @@ Quality X: unknown, ie we knowing to little about the camera properties to know
"ranges": { "white": 4040 } "ranges": { "white": 4040 }
}, },
{ // Quality B
"make_model": "FUJIFILM X-PRO2",
"dcraw_matrix": [ 11434,-4948,-1210,-3746,12042,1903,-666,1479,5235 ], // DNG_v9.4 D65
"raw_crop": [ 0, 0, 6032, 4032 ], // full raw 6160,4032, Usable 6032,4032 - experimental crop
"ranges": { "white": 16100 }
},
{ // Quality B, Matrix from Adobe's dcp D65 instead of the internal in Leica's DNG { // Quality B, Matrix from Adobe's dcp D65 instead of the internal in Leica's DNG
"make_model": [ "LEICA SL (Typ 601)", "LEICA Q (Typ 116)" ], "make_model": [ "LEICA SL (Typ 601)", "LEICA Q (Typ 116)" ],
"dcraw_matrix": [ 10068,-4043,-1068,-5319,14268,1044,-765,1701,6522 ], // DCP D65 "dcraw_matrix": [ 10068,-4043,-1068,-5319,14268,1044,-765,1701,6522 ], // DCP D65

View File

@@ -7,7 +7,7 @@
* it under the terms of the GNU General Public License as published by * it under the terms of the GNU General Public License as published by
* the Free Software Foundation, either version 3 of the License, or * the Free Software Foundation, either version 3 of the License, or
* (at your option) any later version. * (at your option) any later version.
* *
* RawTherapee is distributed in the hope that it will be useful, * RawTherapee is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of * but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
@@ -59,7 +59,7 @@ public:
,RT_blacklevel_from_constant(0) ,RT_blacklevel_from_constant(0)
,RT_matrix_from_constant(0) ,RT_matrix_from_constant(0)
,getbithuff(this,ifp,zero_after_ff) ,getbithuff(this,ifp,zero_after_ff)
,ph1_bithuff(this,ifp,order) ,ph1_bithuff(this,ifp,order)
,pana_bits(ifp,load_flags) ,pana_bits(ifp,load_flags)
{ {
memset(&hbd, 0, sizeof(hbd)); memset(&hbd, 0, sizeof(hbd));
@@ -100,7 +100,7 @@ protected:
double gamm[6]; double gamm[6];
dcrawImage_t image; dcrawImage_t image;
float bright, threshold, user_mul[4]; float bright, threshold, user_mul[4];
int half_size, four_color_rgb, document_mode, highlight; int half_size, four_color_rgb, document_mode, highlight;
int verbose, use_auto_wb, use_camera_wb, use_camera_matrix; int verbose, use_auto_wb, use_camera_wb, use_camera_matrix;
int output_color, output_bps, output_tiff, med_passes; int output_color, output_bps, output_tiff, med_passes;
@@ -111,7 +111,7 @@ protected:
int RT_matrix_from_constant; int RT_matrix_from_constant;
float cam_mul[4], pre_mul[4], cmatrix[3][4], rgb_cam[3][4]; float cam_mul[4], pre_mul[4], cmatrix[3][4], rgb_cam[3][4];
int histogram[4][0x2000]; int histogram[4][0x2000];
void (DCraw::*write_thumb)(), (DCraw::*write_fun)(); void (DCraw::*write_thumb)(), (DCraw::*write_fun)();
void (DCraw::*load_raw)(), (DCraw::*thumb_load_raw)(); void (DCraw::*load_raw)(), (DCraw::*thumb_load_raw)();
@@ -163,7 +163,7 @@ protected:
int rat[10]; int rat[10];
unsigned gps[26]; unsigned gps[26];
char desc[512], make[64], model[64], soft[32], date[20], artist[64]; char desc[512], make[64], model[64], soft[32], date[20], artist[64];
}; };
protected: protected:
int fcol (int row, int col); int fcol (int row, int col);
@@ -362,7 +362,7 @@ void crop_masked_pixels();
void tiff_get (unsigned base, unsigned *tag, unsigned *type, unsigned *len, unsigned *save); void tiff_get (unsigned base, unsigned *tag, unsigned *type, unsigned *len, unsigned *save);
void parse_thumb_note (int base, unsigned toff, unsigned tlen); void parse_thumb_note (int base, unsigned toff, unsigned tlen);
int parse_tiff_ifd (int base); int parse_tiff_ifd (int base);
void parse_makernote (int base, int uptag); void parse_makernote (int base, int uptag);
void get_timestamp (int reversed); void get_timestamp (int reversed);
void parse_exif (int base); void parse_exif (int base);
@@ -397,6 +397,19 @@ void identify();
void apply_profile (const char *input, const char *output); void apply_profile (const char *input, const char *output);
void jpeg_thumb() {} // not needed void jpeg_thumb() {} // not needed
bool dcraw_coeff_overrides(const char make[], const char model[], int iso_speed, short trans[12], int *black_level, int *white_level); bool dcraw_coeff_overrides(const char make[], const char model[], int iso_speed, short trans[12], int *black_level, int *white_level);
void shiftXtransMatrix( const int offsy, const int offsx) {
char xtransTemp[6][6];
for(int row = 0;row < 6; row++) {
for(int col = 0;col < 6; col++) {
xtransTemp[row][col] = xtrans[(row+offsy)%6][(col+offsx)%6];
}
}
for(int row = 0;row < 6; row++) {
for(int col = 0;col < 6; col++) {
xtrans[row][col] = xtransTemp[row][col];
}
}
}
}; };

View File

@@ -495,11 +495,13 @@ int RawImage::loadRaw (bool loadData, bool closeFile, ProgressListener *plistene
if (cc && cc->has_rawCrop()) { if (cc && cc->has_rawCrop()) {
int lm, tm, w, h; int lm, tm, w, h;
cc->get_rawCrop(lm, tm, w, h); cc->get_rawCrop(lm, tm, w, h);
if(isXtrans()) {
if(((int)top_margin - tm) & 1) { // we have an odd border difference shiftXtransMatrix(6 - ((top_margin - tm)%6), 6 - ((left_margin - lm)%6));
filters = (filters << 4) | (filters >> 28); // left rotate filters by 4 bits } else {
if(((int)top_margin - tm) & 1) { // we have an odd border difference
filters = (filters << 4) | (filters >> 28); // left rotate filters by 4 bits
}
} }
left_margin = lm; left_margin = lm;
top_margin = tm; top_margin = tm;