--- ray/src/rt/ambcomp.c	1991/06/13 12:00:29	1.7
+++ ray/src/rt/ambcomp.c	1992/03/04 16:28:25	2.3
@@ -60,21 +60,21 @@ AMBHEMI  *h;
 RAY  *r;
 {
 	RAY  ar;
-	int  hlist[4];
+	int  hlist[3];
+	double  spt[2];
 	double  xd, yd, zd;
 	double  b2;
 	double  phi;
 	register int  i;
 
-	if (rayorigin(&ar, r, AMBIENT, 0.5) < 0)
+	if (rayorigin(&ar, r, AMBIENT, AVGREFL) < 0)
 		return(-1);
 	hlist[0] = r->rno;
 	hlist[1] = dp->t;
 	hlist[2] = dp->p;
-	hlist[3] = 0;
-	zd = sqrt((dp->t+urand(ilhash(hlist,4)+dp->n))/h->nt);
-	hlist[3] = 1;
-	phi = 2.0*PI * (dp->p+urand(ilhash(hlist,4)+dp->n))/h->np;
+	multisamp(spt, 2, urand(ilhash(hlist,3)+dp->n));
+	zd = sqrt((dp->t + spt[0])/h->nt);
+	phi = 2.0*PI * (dp->p + spt[1])/h->np;
 	xd = cos(phi) * zd;
 	yd = sin(phi) * zd;
 	zd = sqrt(1.0 - zd*zd);
@@ -86,7 +86,7 @@ RAY  *r;
 	rayvalue(&ar);
 	ndims--;
 	addcolor(dp->v, ar.rcol);
-	if (ar.rt < FHUGE)
+	if (ar.rt > FTINY && ar.rt < FHUGE)
 		dp->r += 1.0/ar.rt;
 					/* (re)initialize error */
 	if (dp->n++) {
@@ -100,9 +100,10 @@ RAY  *r;
 
 
 double
-doambient(acol, r, pg, dg)		/* compute ambient component */
+doambient(acol, r, wt, pg, dg)		/* compute ambient component */
 COLOR  acol;
 RAY  *r;
+double  wt;
 FVECT  pg, dg;
 {
 	double  b, d;
@@ -116,12 +117,12 @@ FVECT  pg, dg;
 					/* initialize color */
 	setcolor(acol, 0.0, 0.0, 0.0);
 					/* initialize hemisphere */
-	inithemi(&hemi, r);
+	inithemi(&hemi, r, wt);
 	ndivs = hemi.nt * hemi.np;
 	if (ndivs == 0)
 		return(0.0);
 					/* set number of super-samples */
-	ns = ambssamp * r->rweight + 0.5;
+	ns = ambssamp * wt + 0.5;
 	if (ns > 0 || pg != NULL || dg != NULL) {
 		div = (AMBSAMP *)malloc(ndivs*sizeof(AMBSAMP));
 		if (div == NULL)
@@ -205,22 +206,24 @@ FVECT  pg, dg;
 	b = 1.0/ndivs;
 	scalecolor(acol, b);
 	if (arad <= FTINY)
-		arad = FHUGE;
+		arad = maxarad;
 	else
 		arad = (ndivs+ns)/arad;
-	if (arad > maxarad)
-		arad = maxarad;
-	else if (arad < minarad)
+	if (pg != NULL) {		/* reduce radius if gradient large */
+		d = DOT(pg,pg);
+		if (d*arad*arad > 1.0)
+			arad = 1.0/sqrt(d);
+	}
+	if (arad < minarad) {
 		arad = minarad;
-	arad /= sqrt(r->rweight);
-	if (pg != NULL) {		/* clip pos. gradient if too large */
-		d = 4.0*DOT(pg,pg)*arad*arad;
-		if (d > 1.0) {
-			d = 1.0/sqrt(d);
+		if (pg != NULL && d*arad*arad > 1.0) {	/* cap gradient */
+			d = 1.0/arad/sqrt(d);
 			for (i = 0; i < 3; i++)
 				pg[i] *= d;
 		}
 	}
+	if ((arad /= sqrt(wt)) > maxarad)
+		arad = maxarad;
 	return(arad);
 oopsy:
 	if (div != NULL)
@@ -229,14 +232,19 @@ oopsy:
 }
 
 
-inithemi(hp, r)			/* initialize sampling hemisphere */
+inithemi(hp, r, wt)		/* initialize sampling hemisphere */
 register AMBHEMI  *hp;
 RAY  *r;
+double  wt;
 {
 	register int  i;
 					/* set number of divisions */
-	hp->nt = sqrt(ambdiv * r->rweight * 0.5) + 0.5;
-	hp->np = 2 * hp->nt;
+	if (wt < (.25*PI)/ambdiv+FTINY) {
+		hp->nt = hp->np = 0;
+		return;			/* zero samples */
+	}
+	hp->nt = sqrt(ambdiv * wt / PI) + 0.5;
+	hp->np = PI * hp->nt + 0.5;
 					/* make axes */
 	VCOPY(hp->uz, r->ron);
 	hp->uy[0] = hp->uy[1] = hp->uy[2] = 0.0;
@@ -305,10 +313,10 @@ register AMBHEMI  *hp;
 posgradient(gv, da, hp)				/* compute position gradient */
 FVECT  gv;
 AMBSAMP  *da;			/* assumes standard ordering */
-AMBHEMI  *hp;
+register AMBHEMI  *hp;
 {
 	register int  i, j;
-	double  b, d;
+	double  nextsine, lastsine, b, d;
 	double  mag0, mag1;
 	double  phi, cosp, sinp, xd, yd;
 	register AMBSAMP  *dp;
@@ -317,6 +325,7 @@ AMBHEMI  *hp;
 	for (j = 0; j < hp->np; j++) {
 		dp = da + j;
 		mag0 = mag1 = 0.0;
+		lastsine = 0.0;
 		for (i = 0; i < hp->nt; i++) {
 #ifdef  DEBUG
 			if (dp->t != i || dp->p != j)
@@ -327,24 +336,26 @@ AMBHEMI  *hp;
 			if (i > 0) {
 				d = dp[-hp->np].r;
 				if (dp[0].r > d) d = dp[0].r;
-				d *= 1.0 - sqrt((double)i/hp->nt);
+							/* sin(t)*cos(t)^2 */
+				d *= lastsine * (1.0 - (double)i/hp->nt);
 				mag0 += d*(b - bright(dp[-hp->np].v));
 			}
+			nextsine = sqrt((double)(i+1)/hp->nt);
 			if (j > 0) {
 				d = dp[-1].r;
 				if (dp[0].r > d) d = dp[0].r;
-				mag1 += d*(b - bright(dp[-1].v));
+				mag1 += d * (nextsine - lastsine) *
+						(b - bright(dp[-1].v));
 			} else {
 				d = dp[hp->np-1].r;
 				if (dp[0].r > d) d = dp[0].r;
-				mag1 += d*(b - bright(dp[hp->np-1].v));
+				mag1 += d * (nextsine - lastsine) *
+						(b - bright(dp[hp->np-1].v));
 			}
 			dp += hp->np;
+			lastsine = nextsine;
 		}
-		if (hp->nt > 1) {
-			mag0 /= (double)hp->np;
-			mag1 /= (double)hp->nt;
-		}
+		mag0 *= 2.0*PI / hp->np;
 		phi = 2.0*PI * (double)j/hp->np;
 		cosp = cos(phi); sinp = sin(phi);
 		xd += mag0*cosp - mag1*sinp;
@@ -358,7 +369,7 @@ AMBHEMI  *hp;
 dirgradient(gv, da, hp)				/* compute direction gradient */
 FVECT  gv;
 AMBSAMP  *da;			/* assumes standard ordering */
-AMBHEMI  *hp;
+register AMBHEMI  *hp;
 {
 	register int  i, j;
 	double  mag;
@@ -375,7 +386,8 @@ AMBHEMI  *hp;
 				error(CONSISTENCY,
 					"division order in dirgradient");
 #endif
-			mag += sqrt((i+.5)/hp->nt)*bright(dp->v);
+							/* tan(t) */
+			mag += bright(dp->v)/sqrt(hp->nt/(i+.5) - 1.0);
 			dp += hp->np;
 		}
 		phi = 2.0*PI * (j+.5)/hp->np + PI/2.0;
@@ -383,5 +395,5 @@ AMBHEMI  *hp;
 		yd += mag * sin(phi);
 	}
 	for (i = 0; i < 3; i++)
-		gv[i] = (xd*hp->ux[i] + yd*hp->uy[i])*PI/(hp->nt*hp->np);
+		gv[i] = (xd*hp->ux[i] + yd*hp->uy[i])/(hp->nt*hp->np);
 }