ViewVC Help
View File | Revision Log | Show Annotations | Download File | Root Listing
root/radiance/ray/src/rt/ambcomp.c
(Generate patch)

Comparing ray/src/rt/ambcomp.c (file contents):
Revision 2.86 by greg, Wed Feb 17 01:29:22 2021 UTC vs.
Revision 2.100 by greg, Mon Apr 28 19:30:01 2025 UTC

# Line 24 | Line 24 | static const char      RCSid[] = "$Id$";
24   #ifndef MINADIV
25   #define MINADIV         7       /* minimum # divisions in each dimension */
26   #endif
27 + #ifndef MINSDIST
28 + #define MINSDIST        0.25    /* def. min. spacing = 1/4th division */
29 + #endif
30  
28 – extern void             SDsquare2disk(double ds[2], double seedx, double seedy);
29 –
31   typedef struct {
31 –        COLOR   v;              /* hemisphere sample value */
32 –        float   d;              /* reciprocal distance */
32          FVECT   p;              /* intersection point */
33 +        float   d;              /* reciprocal distance */
34 +        SCOLOR  v;              /* hemisphere sample value */
35   } AMBSAMP;              /* sample value */
36  
37   typedef struct {
38          RAY     *rp;            /* originating ray sample */
39          int     ns;             /* number of samples per axis */
40          int     sampOK;         /* acquired full sample set? */
41 <        COLOR   acoef;          /* division contribution coefficient */
42 <        double  acol[3];        /* accumulated color */
41 >        int     atyp;           /* RAMBIENT or TAMBIENT */
42 >        SCOLOR  acoef;          /* division contribution coefficient */
43 >        SCOLOR  acol;           /* accumulated color */
44 >        FVECT   onrm;           /* oriented unperturbed surface normal */
45          FVECT   ux, uy;         /* tangent axis unit vectors */
46          AMBSAMP sa[1];          /* sample array (extends struct) */
47   }  AMBHEMI;             /* ambient sample hemisphere */
# Line 62 | Line 65 | ambcollision(                          /* proposed direciton collides? */
65   {
66          double  cos_thresh;
67          int     ii, jj;
68 <                                        /* min. spacing = 1/4th division */
69 <        cos_thresh = (PI/4.)/(double)hp->ns;
68 >
69 >        cos_thresh = (PI*MINSDIST)/(double)hp->ns;
70          cos_thresh = 1. - .5*cos_thresh*cos_thresh;
71                                          /* check existing neighbors */
72          for (ii = i-1; ii <= i+1; ii++) {
# Line 89 | Line 92 | ambcollision(                          /* proposed direciton collides? */
92   }
93  
94  
95 + #define XLOTSIZ         251             /* size of used car lot */
96 + #define CFIRST          0               /* first corner */
97 + #define COTHER          (CFIRST+4)      /* non-corner sample */
98 + #define CMAXTARGET      (int)(XLOTSIZ*MINSDIST/(1-MINSDIST))
99 + #define CXCOPY(d,s)     (excharr[d][0]=excharr[s][0], excharr[d][1]=excharr[s][1])
100 +
101   static int
102 + psample_class(double ss[2])             /* classify patch sample */
103 + {
104 +        if (ss[0] < MINSDIST) {
105 +                if (ss[1] < MINSDIST)
106 +                        return(CFIRST);
107 +                if (ss[1] > 1.-MINSDIST)
108 +                        return(CFIRST+2);
109 +        } else if (ss[0] > 1.-MINSDIST) {
110 +                if (ss[1] < MINSDIST)
111 +                        return(CFIRST+1);
112 +                if (ss[1] > 1.-MINSDIST)
113 +                        return(CFIRST+3);
114 +        }
115 +        return(COTHER);                 /* not in a corner */
116 + }
117 +
118 + static void
119 + trade_patchsamp(double ss[2])           /* trade in problem patch position */
120 + {
121 +        static float    excharr[XLOTSIZ][2];
122 +        static short    gterm[COTHER+1];
123 +        double          srep[2];
124 +        int             sclass, rclass;
125 +        int             x;
126 +                                        /* reset on corner overload */
127 +        if (gterm[COTHER-1] >= (CMAXTARGET+XLOTSIZ)/2)
128 +                memset(gterm, 0, sizeof(gterm));
129 +                                        /* (re-)initialize? */
130 +        while (gterm[COTHER] < XLOTSIZ) {
131 +                excharr[gterm[COTHER]][0] = frandom();
132 +                excharr[gterm[COTHER]][1] = frandom();
133 +                ++gterm[COTHER];
134 +        }                               /* get trade-in candidate... */
135 +        sclass = psample_class(ss);     /* submitted corner or not? */
136 +        switch (sclass) {
137 +        case COTHER:                    /* trade mid-edge with corner/any */
138 +                x = irandom( gterm[COTHER-1] > CMAXTARGET
139 +                                ? gterm[COTHER-1] : XLOTSIZ );
140 +                break;
141 +        case CFIRST:                    /* kick out of first corner */
142 +                x = gterm[CFIRST] + irandom(XLOTSIZ - gterm[CFIRST]);
143 +                break;
144 +        default:                        /* kick out of 2nd-4th corner */
145 +                x = irandom(XLOTSIZ - (gterm[sclass] - gterm[sclass-1]));
146 +                x += (x >= gterm[sclass-1])*(gterm[sclass] - gterm[sclass-1]);
147 +                break;
148 +        }
149 +        srep[0] = excharr[x][0];        /* save selected replacement (result) */
150 +        srep[1] = excharr[x][1];
151 +                                        /* identify replacement class */
152 +        for (rclass = CFIRST; rclass < COTHER; rclass++)
153 +                if (x < gterm[rclass])
154 +                        break;          /* repark to keep classes grouped */
155 +        while (rclass > sclass) {       /* replacement group after submitted? */
156 +                CXCOPY(x, gterm[rclass-1]);
157 +                x = gterm[--rclass]++;
158 +        }
159 +        while (rclass < sclass) {       /* replacement group before submitted? */
160 +                --gterm[rclass];
161 +                CXCOPY(x, gterm[rclass]);
162 +                x = gterm[rclass++];
163 +        }
164 +        excharr[x][0] = ss[0];          /* complete the trade-in */
165 +        excharr[x][1] = ss[1];
166 +        ss[0] = srep[0];
167 +        ss[1] = srep[1];
168 + }
169 +
170 + #undef CXCOPY
171 + #undef XLOTSIZ
172 + #undef COTHER
173 + #undef CFIRST
174 +
175 +
176 + static int
177   ambsample(                              /* initial ambient division sample */
178          AMBHEMI *hp,
179          int     i,
# Line 100 | Line 184 | ambsample(                             /* initial ambient division sample */
184          AMBSAMP *ap = &ambsam(hp,i,j);
185          RAY     ar;
186          int     hlist[3], ii;
187 <        double  spt[2], zd;
187 >        double  ss[2];
188 >        RREAL   spt[2];
189 >        double  zd;
190                                          /* generate hemispherical sample */
191                                          /* ambient coefficient for weight */
192          if (ambacc > FTINY)
193 <                setcolor(ar.rcoef, AVGREFL, AVGREFL, AVGREFL);
193 >                setscolor(ar.rcoef, AVGREFL, AVGREFL, AVGREFL);
194          else
195 <                copycolor(ar.rcoef, hp->acoef);
196 <        if (rayorigin(&ar, AMBIENT, hp->rp, ar.rcoef) < 0)
195 >                copyscolor(ar.rcoef, hp->acoef);
196 >        if (rayorigin(&ar, hp->atyp, hp->rp, ar.rcoef) < 0)
197                  return(0);
198          if (ambacc > FTINY) {
199 <                multcolor(ar.rcoef, hp->acoef);
200 <                scalecolor(ar.rcoef, 1./AVGREFL);
199 >                smultscolor(ar.rcoef, hp->acoef);
200 >                scalescolor(ar.rcoef, 1./AVGREFL);
201          }
202          hlist[0] = hp->rp->rno;
203 <        hlist[1] = j;
204 <        hlist[2] = i;
205 <        multisamp(spt, 2, urand(ilhash(hlist,3)+n));
206 < resample:
207 <        SDsquare2disk(spt, (j+spt[1])/hp->ns, (i+spt[0])/hp->ns);
203 >        hlist[1] = AI(hp,i,j);
204 >        hlist[2] = samplendx;
205 >        multisamp(ss, 2, urand(ilhash(hlist,3)+n));
206 > patch_redo:
207 >        square2disk(spt, (j+ss[1])/hp->ns, (i+ss[0])/hp->ns);
208          zd = sqrt(1. - spt[0]*spt[0] - spt[1]*spt[1]);
209          for (ii = 3; ii--; )
210                  ar.rdir[ii] =   spt[0]*hp->ux[ii] +
211                                  spt[1]*hp->uy[ii] +
212 <                                zd*hp->rp->ron[ii];
212 >                                zd*hp->onrm[ii];
213          checknorm(ar.rdir);
214                                          /* avoid coincident samples */
215 <        if (!n && ambcollision(hp, i, j, ar.rdir)) {
216 <                spt[0] = frandom(); spt[1] = frandom();
217 <                goto resample;          /* reject this sample */
215 >        if (!n & (hp->ns >= 4) && ambcollision(hp, i, j, ar.rdir)) {
216 >                trade_patchsamp(ss);
217 >                goto patch_redo;
218          }
219          dimlist[ndims++] = AI(hp,i,j) + 90171;
220          rayvalue(&ar);                  /* evaluate ray */
# Line 136 | Line 222 | resample:
222          zd = raydistance(&ar);
223          if (zd <= FTINY)
224                  return(0);              /* should never happen */
225 <        multcolor(ar.rcol, ar.rcoef);   /* apply coefficient */
225 >        smultscolor(ar.rcol, ar.rcoef); /* apply coefficient */
226          if (zd*ap->d < 1.0)             /* new/closer distance? */
227                  ap->d = 1.0/zd;
228          if (!n) {                       /* record first vertex & value */
229                  if (zd > 10.0*thescene.cusize + 1000.)
230                          zd = 10.0*thescene.cusize + 1000.;
231                  VSUM(ap->p, ar.rorg, ar.rdir, zd);
232 <                copycolor(ap->v, ar.rcol);
232 >                copyscolor(ap->v, ar.rcol);
233          } else {                        /* else update recorded value */
234 <                hp->acol[RED] -= colval(ap->v,RED);
149 <                hp->acol[GRN] -= colval(ap->v,GRN);
150 <                hp->acol[BLU] -= colval(ap->v,BLU);
234 >                sopscolor(hp->acol, -=, ap->v);
235                  zd = 1.0/(double)(n+1);
236 <                scalecolor(ar.rcol, zd);
236 >                scalescolor(ar.rcol, zd);
237                  zd *= (double)n;
238 <                scalecolor(ap->v, zd);
239 <                addcolor(ap->v, ar.rcol);
238 >                scalescolor(ap->v, zd);
239 >                saddscolor(ap->v, ar.rcol);
240          }
241 <        addcolor(hp->acol, ap->v);      /* add to our sum */
241 >        saddscolor(hp->acol, ap->v);    /* add to our sum */
242          return(1);
243   }
244  
# Line 163 | Line 247 | resample:
247   static float *
248   getambdiffs(AMBHEMI *hp)
249   {
250 <        const double    normf = 1./bright(hp->acoef);
251 <        float   *earr = (float *)calloc(hp->ns*hp->ns, sizeof(float));
250 >        const double    normf = 1./(pbright(hp->acoef) + FTINY);
251 >        float   *earr = (float *)calloc(2*hp->ns*hp->ns, sizeof(float));
252          float   *ep;
253          AMBSAMP *ap;
254          double  b, b1, d2;
# Line 173 | Line 257 | getambdiffs(AMBHEMI *hp)
257          if (earr == NULL)               /* out of memory? */
258                  return(NULL);
259                                          /* sum squared neighbor diffs */
260 <        for (ap = hp->sa, ep = earr, i = 0; i < hp->ns; i++)
260 >        ap = hp->sa;
261 >        ep = earr + hp->ns*hp->ns;      /* original estimates to scratch */
262 >        for (i = 0; i < hp->ns; i++)
263              for (j = 0; j < hp->ns; j++, ap++, ep++) {
264 <                b = bright(ap[0].v);
264 >                b = pbright(ap[0].v);
265                  if (i) {                /* from above */
266 <                        b1 = bright(ap[-hp->ns].v);
266 >                        b1 = pbright(ap[-hp->ns].v);
267                          d2 = b - b1;
268 <                        d2 *= d2*normf/(b + b1);
268 >                        d2 *= d2*normf/(b + b1 + FTINY);
269                          ep[0] += d2;
270                          ep[-hp->ns] += d2;
271                  }
272                  if (!j) continue;
273                                          /* from behind */
274 <                b1 = bright(ap[-1].v);
274 >                b1 = pbright(ap[-1].v);
275                  d2 = b - b1;
276 <                d2 *= d2*normf/(b + b1);
276 >                d2 *= d2*normf/(b + b1 + FTINY);
277                  ep[0] += d2;
278                  ep[-1] += d2;
279                  if (!i) continue;
280                                          /* diagonal */
281 <                b1 = bright(ap[-hp->ns-1].v);
281 >                b1 = pbright(ap[-hp->ns-1].v);
282                  d2 = b - b1;
283 <                d2 *= d2*normf/(b + b1);
283 >                d2 *= d2*normf/(b + b1 + FTINY);
284                  ep[0] += d2;
285                  ep[-hp->ns-1] += d2;
286              }
287                                          /* correct for number of neighbors */
288 <        earr[0] *= 8./3.;
289 <        earr[hp->ns-1] *= 8./3.;
290 <        earr[(hp->ns-1)*hp->ns] *= 8./3.;
291 <        earr[(hp->ns-1)*hp->ns + hp->ns-1] *= 8./3.;
288 >        ep = earr + hp->ns*hp->ns;
289 >        ep[0] *= 6./3.;
290 >        ep[hp->ns-1] *= 6./3.;
291 >        ep[(hp->ns-1)*hp->ns] *= 6./3.;
292 >        ep[(hp->ns-1)*hp->ns + hp->ns-1] *= 6./3.;
293          for (i = 1; i < hp->ns-1; i++) {
294 <                earr[i*hp->ns] *= 8./5.;
295 <                earr[i*hp->ns + hp->ns-1] *= 8./5.;
294 >                ep[i*hp->ns] *= 6./5.;
295 >                ep[i*hp->ns + hp->ns-1] *= 6./5.;
296          }
297          for (j = 1; j < hp->ns-1; j++) {
298 <                earr[j] *= 8./5.;
299 <                earr[(hp->ns-1)*hp->ns + j] *= 8./5.;
298 >                ep[j] *= 6./5.;
299 >                ep[(hp->ns-1)*hp->ns + j] *= 6./5.;
300          }
301 +                                        /* blur final map to reduce bias */
302 +        for (i = 0; i < hp->ns-1; i++) {
303 +            float  *ep2;
304 +            ep = earr + i*hp->ns;
305 +            ep2 = ep + hp->ns*hp->ns;
306 +            for (j = 0; j < hp->ns-1; j++, ep++, ep2++) {
307 +                ep[0] += .5*ep2[0] + .125*(ep2[1] + ep2[hp->ns]);
308 +                ep[1] += .125*ep2[0];
309 +                ep[hp->ns] += .125*ep2[0];
310 +            }
311 +        }
312          return(earr);
313   }
314  
# Line 246 | Line 344 | done:
344  
345   static AMBHEMI *
346   samp_hemi(                              /* sample indirect hemisphere */
347 <        COLOR   rcol,
347 >        SCOLOR  rcol,
348          RAY     *r,
349          double  wt
350   )
351   {
352 +        int     backside = (wt < 0);
353          AMBHEMI *hp;
354          double  d;
355          int     n, i, j;
356                                          /* insignificance check */
357 <        if (bright(rcol) <= FTINY)
357 >        d = sintens(rcol);
358 >        if (d <= FTINY)
359                  return(NULL);
360                                          /* set number of divisions */
361 +        if (backside) wt = -wt;
362          if (ambacc <= FTINY &&
363 <                        wt > (d = 0.8*intens(rcol)*r->rweight/(ambdiv*minweight)))
363 >                        wt > (d *= 0.8*r->rweight/(ambdiv*minweight + 1e-20)))
364                  wt = d;                 /* avoid ray termination */
365          n = sqrt(ambdiv * wt) + 0.5;
366          i = 1 + (MINADIV-1)*(ambacc > FTINY);
# Line 269 | Line 370 | samp_hemi(                             /* sample indirect hemisphere */
370          hp = (AMBHEMI *)malloc(sizeof(AMBHEMI) + sizeof(AMBSAMP)*(n*n - 1));
371          if (hp == NULL)
372                  error(SYSTEM, "out of memory in samp_hemi");
373 +
374 +        if (backside) {
375 +                hp->atyp = TAMBIENT;
376 +                hp->onrm[0] = -r->ron[0];
377 +                hp->onrm[1] = -r->ron[1];
378 +                hp->onrm[2] = -r->ron[2];
379 +        } else {
380 +                hp->atyp = RAMBIENT;
381 +                VCOPY(hp->onrm, r->ron);
382 +        }
383          hp->rp = r;
384          hp->ns = n;
385 <        hp->acol[RED] = hp->acol[GRN] = hp->acol[BLU] = 0.0;
385 >        scolorblack(hp->acol);
386          memset(hp->sa, 0, sizeof(AMBSAMP)*n*n);
387          hp->sampOK = 0;
388                                          /* assign coefficient */
389 <        copycolor(hp->acoef, rcol);
389 >        copyscolor(hp->acoef, rcol);
390          d = 1.0/(n*n);
391 <        scalecolor(hp->acoef, d);
391 >        scalescolor(hp->acoef, d);
392                                          /* make tangent plane axes */
393 <        if (!getperpendicular(hp->ux, r->ron, 1))
393 >        if (!getperpendicular(hp->ux, hp->onrm, 1))
394                  error(CONSISTENCY, "bad ray direction in samp_hemi");
395 <        VCROSS(hp->uy, r->ron, hp->ux);
395 >        VCROSS(hp->uy, hp->onrm, hp->ux);
396                                          /* sample divisions */
397          for (i = hp->ns; i--; )
398              for (j = hp->ns; j--; )
399                  hp->sampOK += ambsample(hp, i, j, 0);
400 <        copycolor(rcol, hp->acol);
400 >        copyscolor(rcol, hp->acol);
401          if (!hp->sampOK) {              /* utter failure? */
402                  free(hp);
403                  return(NULL);
# Line 298 | Line 409 | samp_hemi(                             /* sample indirect hemisphere */
409          if (hp->sampOK <= MINADIV*MINADIV)
410                  return(hp);             /* don't bother super-sampling */
411          n = ambssamp*wt + 0.5;
412 <        if (n > 8) {                    /* perform super-sampling? */
412 >        if (n >= 4*hp->ns) {            /* perform super-sampling? */
413                  ambsupersamp(hp, n);
414 <                copycolor(rcol, hp->acol);
414 >                copyscolor(rcol, hp->acol);
415          }
416          return(hp);                     /* all is well */
417   }
# Line 312 | Line 423 | back_ambval(AMBHEMI *hp, const int n1, const int n2, c
423   {
424          if (hp->sa[n1].d <= hp->sa[n2].d) {
425                  if (hp->sa[n1].d <= hp->sa[n3].d)
426 <                        return(colval(hp->sa[n1].v,CIEY));
427 <                return(colval(hp->sa[n3].v,CIEY));
426 >                        return(hp->sa[n1].v[0]);
427 >                return(hp->sa[n3].v[0]);
428          }
429          if (hp->sa[n2].d <= hp->sa[n3].d)
430 <                return(colval(hp->sa[n2].v,CIEY));
431 <        return(colval(hp->sa[n3].v,CIEY));
430 >                return(hp->sa[n2].v[0]);
431 >        return(hp->sa[n3].v[0]);
432   }
433  
434  
# Line 546 | Line 657 | ambHessian(                            /* anisotropic radii & pos. gradient */
657          for (j = 0; j < hp->ns-1; j++) {
658                  comp_fftri(&fftr, hp, AI(hp,0,j), AI(hp,0,j+1));
659                  if (hessrow != NULL)
660 <                        comp_hessian(hessrow[j], &fftr, hp->rp->ron);
660 >                        comp_hessian(hessrow[j], &fftr, hp->onrm);
661                  if (gradrow != NULL)
662 <                        comp_gradient(gradrow[j], &fftr, hp->rp->ron);
662 >                        comp_gradient(gradrow[j], &fftr, hp->onrm);
663          }
664                                          /* sum each row of triangles */
665          for (i = 0; i < hp->ns-1; i++) {
# Line 556 | Line 667 | ambHessian(                            /* anisotropic radii & pos. gradient */
667              FVECT       gradcol;
668              comp_fftri(&fftr, hp, AI(hp,i,0), AI(hp,i+1,0));
669              if (hessrow != NULL)
670 <                comp_hessian(hesscol, &fftr, hp->rp->ron);
670 >                comp_hessian(hesscol, &fftr, hp->onrm);
671              if (gradrow != NULL)
672 <                comp_gradient(gradcol, &fftr, hp->rp->ron);
672 >                comp_gradient(gradcol, &fftr, hp->onrm);
673              for (j = 0; j < hp->ns-1; j++) {
674                  FVECT   hessdia[3];     /* compute triangle contributions */
675                  FVECT   graddia;
# Line 568 | Line 679 | ambHessian(                            /* anisotropic radii & pos. gradient */
679                                          /* diagonal (inner) edge */
680                  comp_fftri(&fftr, hp, AI(hp,i,j+1), AI(hp,i+1,j));
681                  if (hessrow != NULL) {
682 <                    comp_hessian(hessdia, &fftr, hp->rp->ron);
682 >                    comp_hessian(hessdia, &fftr, hp->onrm);
683                      rev_hessian(hesscol);
684                      add2hessian(hessian, hessrow[j], hessdia, hesscol, backg);
685                  }
686                  if (gradrow != NULL) {
687 <                    comp_gradient(graddia, &fftr, hp->rp->ron);
687 >                    comp_gradient(graddia, &fftr, hp->onrm);
688                      rev_gradient(gradcol);
689                      add2gradient(gradient, gradrow[j], graddia, gradcol, backg);
690                  }
691                                          /* initialize edge in next row */
692                  comp_fftri(&fftr, hp, AI(hp,i+1,j+1), AI(hp,i+1,j));
693                  if (hessrow != NULL)
694 <                    comp_hessian(hessrow[j], &fftr, hp->rp->ron);
694 >                    comp_hessian(hessrow[j], &fftr, hp->onrm);
695                  if (gradrow != NULL)
696 <                    comp_gradient(gradrow[j], &fftr, hp->rp->ron);
696 >                    comp_gradient(gradrow[j], &fftr, hp->onrm);
697                                          /* new column edge & paired triangle */
698                  backg = back_ambval(hp, AI(hp,i+1,j+1),
699                                          AI(hp,i+1,j), AI(hp,i,j+1));
700                  comp_fftri(&fftr, hp, AI(hp,i,j+1), AI(hp,i+1,j+1));
701                  if (hessrow != NULL) {
702 <                    comp_hessian(hesscol, &fftr, hp->rp->ron);
702 >                    comp_hessian(hesscol, &fftr, hp->onrm);
703                      rev_hessian(hessdia);
704                      add2hessian(hessian, hessrow[j], hessdia, hesscol, backg);
705                      if (i < hp->ns-2)
706                          rev_hessian(hessrow[j]);
707                  }
708                  if (gradrow != NULL) {
709 <                    comp_gradient(gradcol, &fftr, hp->rp->ron);
709 >                    comp_gradient(gradcol, &fftr, hp->onrm);
710                      rev_gradient(graddia);
711                      add2gradient(gradient, gradrow[j], graddia, gradcol, backg);
712                      if (i < hp->ns-2)
# Line 631 | Line 742 | ambdirgrad(AMBHEMI *hp, FVECT uv[2], float dg[2])
742                                          /* use vector for azimuth + 90deg */
743                  VSUB(vd, ap->p, hp->rp->rop);
744                                          /* brightness over cosine factor */
745 <                gfact = colval(ap->v,CIEY) / DOT(hp->rp->ron, vd);
745 >                gfact = ap->v[0] / DOT(hp->onrm, vd);
746                                          /* sine = proj_radius/vd_length */
747                  dgsum[0] -= DOT(uv[1], vd) * gfact;
748                  dgsum[1] += DOT(uv[0], vd) * gfact;
# Line 687 | Line 798 | ambcorral(AMBHEMI *hp, FVECT uv[2], const double r0, c
798  
799   int
800   doambient(                              /* compute ambient component */
801 <        COLOR   rcol,                   /* input/output color */
801 >        SCOLOR  rcol,                   /* input/output color */
802          RAY     *r,
803 <        double  wt,
803 >        double  wt,                     /* negative for back side */
804          FVECT   uv[2],                  /* returned (optional) */
805          float   ra[2],                  /* returned (optional) */
806          float   pg[2],                  /* returned (optional) */
# Line 721 | Line 832 | doambient(                             /* compute ambient component */
832                  free(hp);               /* Hessian not requested/possible */
833                  return(-1);             /* value-only return value */
834          }
835 <        if ((d = bright(rcol)) > FTINY) {       /* normalize Y values */
836 <                d = 0.99*(hp->ns*hp->ns)/d;
835 >        if ((d = scolor_mean(rcol)) > FTINY) {
836 >                d = 0.99*(hp->ns*hp->ns)/d;     /* normalize avg. values */
837                  K = 0.01;
838          } else {                        /* or fall back on geometric Hessian */
839                  K = 1.0;
# Line 730 | Line 841 | doambient(                             /* compute ambient component */
841                  dg = NULL;
842                  crlp = NULL;
843          }
844 <        ap = hp->sa;                    /* relative Y channel from here on... */
844 >        ap = hp->sa;                    /* single channel from here on... */
845          for (i = hp->ns*hp->ns; i--; ap++)
846 <                colval(ap->v,CIEY) = bright(ap->v)*d + K;
846 >                ap->v[0] = scolor_mean(ap->v)*d + K;
847  
848          if (uv == NULL)                 /* make sure we have axis pointers */
849                  uv = my_uv;
# Line 756 | Line 867 | doambient(                             /* compute ambient component */
867                          if (ra[1] < minarad)
868                                  ra[1] = minarad;
869                  }
870 <                ra[0] *= d = 1.0/sqrt(wt);
870 >                ra[0] *= d = 1.0/sqrt(fabs(wt));
871                  if ((ra[1] *= d) > 2.0*ra[0])
872                          ra[1] = 2.0*ra[0];
873                  if (ra[1] > maxarad) {
# Line 765 | Line 876 | doambient(                             /* compute ambient component */
876                                  ra[0] = maxarad;
877                  }
878                                          /* flag encroached directions */
879 <                if (crlp != NULL)
879 >                if (crlp != NULL)       /* XXX doesn't update with changes to ambacc */
880                          *crlp = ambcorral(hp, uv, ra[0]*ambacc, ra[1]*ambacc);
881                  if (pg != NULL) {       /* cap gradient if necessary */
882                          d = pg[0]*pg[0]*ra[0]*ra[0] + pg[1]*pg[1]*ra[1]*ra[1];
# Line 779 | Line 890 | doambient(                             /* compute ambient component */
890          free(hp);                       /* clean up and return */
891          return(1);
892   }
782 –

Diff Legend

– Removed lines
+ Added lines
< Changed lines
> Changed lines