--- ray/src/px/macbethcal.c 2003/10/27 10:24:51 2.21 +++ ray/src/px/macbethcal.c 2016/02/02 18:02:32 2.27 @@ -1,5 +1,5 @@ #ifndef lint -static const char RCSid[] = "$Id: macbethcal.c,v 2.21 2003/10/27 10:24:51 schorsch Exp $"; +static const char RCSid[] = "$Id: macbethcal.c,v 2.27 2016/02/02 18:02:32 greg Exp $"; #endif /* * Calibrate a scanned MacBeth Color Checker Chart @@ -15,11 +15,13 @@ static const char RCSid[] = "$Id: macbethcal.c,v 2.21 #include #include "platform.h" -#include "rtprocess.h" +#include "paths.h" +#include "rtio.h" #include "color.h" #include "resolu.h" #include "pmap.h" #include "warp3d.h" +#include "mx3.h" /* MacBeth colors */ #define DarkSkin 0 @@ -46,33 +48,33 @@ static const char RCSid[] = "$Id: macbethcal.c,v 2.21 #define Neutral5 21 #define Neutral35 22 #define Black 23 - /* computed from 5nm spectral measurements */ + /* computed from 10nm spectral measurements */ /* CIE 1931 2 degree obs, equal-energy white */ float mbxyY[24][3] = { - {0.462, 0.3769, 0.0932961}, /* DarkSkin */ - {0.4108, 0.3542, 0.410348}, /* LightSkin */ - {0.2626, 0.267, 0.181554}, /* BlueSky */ - {0.36, 0.4689, 0.108447}, /* Foliage */ - {0.2977, 0.2602, 0.248407}, /* BlueFlower */ - {0.2719, 0.3485, 0.401156}, /* BluishGreen */ - {0.52, 0.4197, 0.357899}, /* Orange */ - {0.229, 0.1866, 0.103911}, /* PurplishBlue */ - {0.4909, 0.3262, 0.242615}, /* ModerateRed */ - {0.3361, 0.2249, 0.0600102}, /* Purple */ - {0.3855, 0.4874, 0.42963}, /* YellowGreen */ - {0.4853, 0.4457, 0.476343}, /* OrangeYellow */ - {0.2026, 0.1369, 0.0529249}, /* Blue */ - {0.3007, 0.4822, 0.221226}, /* Green */ - {0.5805, 0.3238, 0.162167}, /* Red */ - {0.4617, 0.472, 0.64909}, /* Yellow */ - {0.4178, 0.2625, 0.233662}, /* Magenta */ - {0.2038, 0.2508, 0.167275}, /* Cyan */ - {0.3358, 0.337, 0.916877}, /* White */ - {0.3338, 0.3348, 0.604678}, /* Neutral.8 */ - {0.3333, 0.3349, 0.364566}, /* Neutral.65 */ - {0.3353, 0.3359, 0.200238}, /* Neutral.5 */ - {0.3363, 0.336, 0.0878721}, /* Neutral.35 */ - {0.3346, 0.3349, 0.0308383} /* Black */ + {0.421236, 0.361196, 0.103392}, /* DarkSkin */ + {0.40868, 0.358157, 0.352867}, /* LightSkin */ + {0.265063, 0.271424, 0.185124}, /* BlueSky */ + {0.362851, 0.43055, 0.132625}, /* Foliage */ + {0.28888, 0.260851, 0.233138}, /* BlueFlower */ + {0.277642, 0.365326, 0.416443}, /* BluishGreen */ + {0.524965, 0.40068, 0.312039}, /* Orange */ + {0.225018, 0.190392, 0.114999}, /* PurplishBlue */ + {0.487199, 0.315372, 0.198616}, /* ModerateRed */ + {0.314245, 0.227231, 0.0646047}, /* Purple */ + {0.396202, 0.489732, 0.440724}, /* YellowGreen */ + {0.493297, 0.435299, 0.43444}, /* OrangeYellow */ + {0.198191, 0.149265, 0.0588122}, /* Blue */ + {0.322838, 0.487601, 0.229258}, /* Green */ + {0.561833, 0.321165, 0.126978}, /* Red */ + {0.468113, 0.467021, 0.605289}, /* Yellow */ + {0.397128, 0.248535, 0.201761}, /* Magenta */ + {0.209552, 0.276256, 0.190917}, /* Cyan */ + {0.337219, 0.339042, 0.912482}, /* White */ + {0.333283, 0.335077, 0.588297}, /* Neutral.8 */ + {0.332747, 0.334371, 0.3594}, /* Neutral.65 */ + {0.331925, 0.334202, 0.19114}, /* Neutral.5 */ + {0.330408, 0.332615, 0.0892964}, /* Neutral.35 */ + {0.331841, 0.331405, 0.0319541}, /* Black */ }; COLOR mbRGB[24]; /* MacBeth RGB values */ @@ -99,7 +101,7 @@ short mbneu[NMBNEU] = {Black,Neutral35,Neutral5,Neutra #define RG_CORR 04 /* corrected color region */ #ifndef DISPCOM -#define DISPCOM "ximage -op %s" +#define DISPCOM "ximage -op \"%s\"" #endif int scanning = 1; /* scanned input (or recorded output)? */ @@ -123,10 +125,29 @@ WARP3D *wcor = NULL; /* color space warp */ FILE *debugfp = NULL; /* debug output picture */ char *progname; +static void init(void); +static int chartndx(int x, int y, int *np); +static void getpicture(void); +static void getcolors(void); +static void bresp(COLOR y, COLOR x); +static void compute(void); +static void putmapping(void); +static void compsoln(COLOR cin[], COLOR cout[], int n); +static void cwarp(void); +static int cvtcolor(COLOR cout, COLOR cin); +static int cresp(COLOR cout, COLOR cin); +static void xyY2RGB(COLOR rgbout, float xyYin[3]); +static void picdebug(void); +static void clrdebug(void); +static void getpos(char *name, int bnds[2], FILE *fp); +static void pickchartpos(char *pfn); -main(argc, argv) -int argc; -char **argv; + +int +main( + int argc, + char **argv +) { int i; @@ -219,9 +240,9 @@ char **argv; printf("# Color correspondence produced by:\n#\t\t"); printargs(argc, argv, stdout); - printf("#\tUsage: pcwarp %s uncorrected.pic > corrected.pic\n", + printf("#\tUsage: pcwarp %s uncorrected.hdr > corrected.hdr\n", i+1 < argc ? argv[i+1] : "{this_file}"); - printf("#\t Or: pcond [options] -m %s orig.pic > output.pic\n", + printf("#\t Or: pcond [options] -m %s orig.hdr > output.hdr\n", i+1 < argc ? argv[i+1] : "{this_file}"); for (j = 0; j < 24; j++) printf("%f %f %f %f %f %f\n", @@ -234,10 +255,10 @@ char **argv; /* print header */ printf("{\n\tColor correction file computed by:\n\t\t"); printargs(argc, argv, stdout); - printf("\n\tUsage: pcomb -f %s uncorrected.pic > corrected.pic\n", + printf("\n\tUsage: pcomb -f %s uncorrected.hdr > corrected.hdr\n", i+1 < argc ? argv[i+1] : "{this_file}"); if (!scanning) - printf("\t Or: pcond [options] -f %s orig.pic > output.pic\n", + printf("\t Or: pcond [options] -f %s orig.hdr > output.hdr\n", i+1 < argc ? argv[i+1] : "{this_file}"); printf("}\n"); putmapping(); /* put out color mapping */ @@ -251,15 +272,17 @@ char **argv; exit(0); userr: fprintf(stderr, -"Usage: %s [-d dbg.pic][-P | -p xul yul xur yur xll yll xlr ylr][-i irrad][-m] input.pic [output.{cal|cwp}]\n", +"Usage: %s [-d dbg.hdr][-P | -p xul yul xur yur xll yll xlr ylr][-i irrad][-m] input.hdr [output.{cal|cwp}]\n", progname); - fprintf(stderr, " or: %s [-d dbg.pic][-i irrad][-m] -c [xyY.dat [output.{cal|cwp}]]\n", + fprintf(stderr, " or: %s [-d dbg.hdr][-i irrad][-m] -c [xyY.dat [output.{cal|cwp}]]\n", progname); exit(1); + return 1; /* pro forma return */ } -init() /* initialize */ +static void +init(void) /* initialize */ { double quad[4][2]; register int i; @@ -285,10 +308,12 @@ init() /* initialize */ } -int -chartndx(x, y, np) /* find color number for position */ -int x, y; -int *np; +static int +chartndx( /* find color number for position */ + int x, + int y, + int *np +) { double ipos[3], cpos[3]; int ix, iy; @@ -317,7 +342,8 @@ int *np; } -getpicture() /* load in picture colors */ +static void +getpicture(void) /* load in picture colors */ { COLR *scanln; COLOR pval; @@ -359,7 +385,8 @@ getpicture() /* load in picture colors */ } -getcolors() /* get xyY colors from standard input */ +static void +getcolors(void) /* get xyY colors from standard input */ { int gotwhite = 0; COLOR whiteclr; @@ -407,8 +434,11 @@ getcolors() /* get xyY colors from standard input */ } -bresp(y, x) /* piecewise linear interpolation of primaries */ -COLOR y, x; +static void +bresp( /* piecewise linear interpolation of primaries */ + COLOR y, + COLOR x +) { register int i, n; @@ -425,7 +455,8 @@ COLOR y, x; } -compute() /* compute color mapping */ +static void +compute(void) /* compute color mapping */ { COLOR clrin[24], clrout[24]; long cflags; @@ -440,6 +471,13 @@ compute() /* compute color mapping */ /* compute piecewise luminance curve */ for (i = 0; i < NMBNEU; i++) { copycolor(bramp[i][0], inpRGB[mbneu[i]]); + for (n = i ? 3 : 0; n--; ) + if (colval(bramp[i][0],n) <= + colval(bramp[i-1][0],n)+1e-7) { + fprintf(stderr, + "%s: non-increasing neutral patch\n", progname); + exit(1); + } copycolor(bramp[i][1], mbRGB[mbneu[i]]); } /* compute color space gamut */ @@ -483,7 +521,8 @@ compute() /* compute color mapping */ } -putmapping() /* put out color mapping */ +static void +putmapping(void) /* put out color mapping */ { static char cchar[3] = {'r', 'g', 'b'}; register int i, j; @@ -531,9 +570,12 @@ putmapping() /* put out color mapping */ } -compsoln(cin, cout, n) /* solve 3xN system using least-squares */ -COLOR cin[], cout[]; -int n; +static void +compsoln( /* solve 3xN system using least-squares */ + COLOR cin[], + COLOR cout[], + int n +) { extern double mx3d_adjoint(), fabs(); double mat[3][3], invmat[3][3]; @@ -591,7 +633,8 @@ int n; } -cwarp() /* compute color warp map */ +static void +cwarp(void) /* compute color warp map */ { register int i; @@ -607,9 +650,11 @@ memerr: } -int -cvtcolor(cout, cin) /* convert color according to our mapping */ -COLOR cout, cin; +static int +cvtcolor( /* convert color according to our mapping */ + COLOR cout, + COLOR cin +) { COLOR ctmp; int clipped; @@ -628,18 +673,22 @@ COLOR cout, cin; } -int -cresp(cout, cin) /* transform color according to matrix */ -COLOR cout, cin; +static int +cresp( /* transform color according to matrix */ + COLOR cout, + COLOR cin +) { colortrans(cout, solmat, cin); return(clipgamut(cout, bright(cout), CGAMUT, colmin, colmax)); } -xyY2RGB(rgbout, xyYin) /* convert xyY to RGB */ -COLOR rgbout; -register float xyYin[3]; +static void +xyY2RGB( /* convert xyY to RGB */ + COLOR rgbout, + register float xyYin[3] +) { COLOR ctmp; double d; @@ -653,7 +702,8 @@ register float xyYin[3]; } -picdebug() /* put out debugging picture */ +static void +picdebug(void) /* put out debugging picture */ { static COLOR blkcol = BLKCOLOR; COLOR *scan; @@ -709,7 +759,8 @@ picdebug() /* put out debugging picture */ } -clrdebug() /* put out debug picture from color input */ +static void +clrdebug(void) /* put out debug picture from color input */ { static COLR blkclr = BLKCOLR; COLR mbclr[24], cvclr[24], orclr[24]; @@ -779,10 +830,12 @@ clrdebug() /* put out debug picture from color input } -getpos(name, bnds, fp) /* get boundary position */ -char *name; -int bnds[2]; -FILE *fp; +static void +getpos( /* get boundary position */ + char *name, + int bnds[2], + FILE *fp +) { char buf[64]; @@ -796,10 +849,12 @@ FILE *fp; } -pickchartpos(pfn) /* display picture and pick chart location */ -char *pfn; +static void +pickchartpos( /* display picture and pick chart location */ + char *pfn +) { - char combuf[512]; + char combuf[PATH_MAX]; FILE *pfp; sprintf(combuf, DISPCOM, pfn);