--- loncom/homework/functionplotresponse.pm	2010/11/10 22:53:32	1.32
+++ loncom/homework/functionplotresponse.pm	2010/11/11 03:52:35	1.33
@@ -1,7 +1,7 @@
 # LearningOnline Network with CAPA
 # option list style responses
 #
-# $Id: functionplotresponse.pm,v 1.32 2010/11/10 22:53:32 www Exp $
+# $Id: functionplotresponse.pm,v 1.33 2010/11/11 03:52:35 www Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -389,7 +389,8 @@ sub start_functionplotrule {
            &Apache::lonxml::get_param('xfinal',$parstack,$safeeval),
            $endinglabel,
            $relationship,
-           &Apache::lonxml::get_param('value',$parstack,$safeeval)
+           &Apache::lonxml::get_param('value',$parstack,$safeeval),
+           &Apache::lonxml::get_param('percenterror',$parstack,$safeeval)
           )));
    } elsif ($target eq 'edit') {
         $result=&Apache::edit::tag_start($target,$token,'Function Plot Evaluation Rule').
@@ -791,10 +792,13 @@ sub compare_rel {
 }
 
 sub functionplotrulecheck {
-   my ($rule,$xmin,$xmax,$tolfunc,$toldfdx,$told2fdx2)=@_;
-#   &Apache::lonnet::logthis("Rule $rule TolFunc $tolfunc TolDfDx $toldfdx TolD2fDx2 $told2fdx2");
-   my ($label,$derivative,$xinitial,$xinitiallabel,$xfinal,$xfinallabel,$relationship,$value)
+   my ($rule,$xmin,$xmax,$ymin,$ymax)=@_;
+
+#   &Apache::lonnet::logthis("Rule $rule $xmin $xmax $ymin $ymax");
+
+   my ($label,$derivative,$xinitial,$xinitiallabel,$xfinal,$xfinallabel,$relationship,$value,$percent)
       =split(/\:/,$rule);
+   $percent=($percent>0?$percent:5);
 
    my $li=0;
    my $lh=400;
@@ -827,7 +831,7 @@ sub functionplotrulecheck {
    }
    unless ($findupper) {
       if (defined($Apache::functionplotresponse::functionplotrulelabels{$xfinallabel})) {
-         $lh=$Apache::functionplotresponse::functionplotrulelabels{$xfinallabel};
+         $lh=$Apache::functionplotresponse::functionplotrulelabels{$xfinallabel}-1;
       } else {
          $lh=&array_index($xmin,$xmax,$xfinal);
       }
@@ -840,11 +844,13 @@ sub functionplotrulecheck {
    }
 
 #   &Apache::lonnet::logthis("Init $xinitial=$li Final $xfinal=$lh Findupper: $findupper");
-   my $tol=$tolfunc;
-   if ($derivative==2) {
-      $tol=$told2fdx2;
-   } elsif ($derivative==1) {
-      $tol=$toldfdx;
+   my $tol=$percent*($ymax-$ymin)/100;
+   if ($xmax>$xmin) {
+      if ($derivative==2) {
+         $tol=4.*$tol/($xmax-$xmin);
+      } elsif ($derivative==1) {
+         $tol=2.*$tol/($xmax-$xmin);
+      }
    }
    for (my $i=$li; $i<=$lh; $i++) {
         my $val;
@@ -856,7 +862,7 @@ sub functionplotrulecheck {
            $val=$Apache::functionplotresponse::func[$i];
         }
         unless (&compare_rel($relationship,$value,$val,$tol)) { 
-#           &Apache::lonnet::logthis("Condition false $findupper at $i with $val");
+#           &Apache::lonnet::logthis("Condition false $findupper at $i with $val tol:$tol from $value");
            if (($findupper) && ($i>$li)) {
               $Apache::functionplotresponse::functionplotrulelabels{$xfinallabel}=$i;
 #              &Apache::lonnet::logthis("Setting $xfinallabel to $i");
@@ -903,17 +909,6 @@ sub end_functionplotresponse {
         my $ymax=&Apache::lonxml::get_param('ymax',$parstack,$safeeval);
         $ymax=(defined($ymax)?$ymax:10);
 
-        my $percent=&Apache::lonxml::get_param('percenterror',$parstack,$safeeval);
-        $percent=(defined($percent)?$percent:5);
-
-        my $tolfunc=$percent*($ymax-$ymin)/100.;
-        my $toldfdx=1;
-        my $told2fdx2=1;
-        if ($xmax>$xmin) {
-           $toldfdx=$tolfunc/($xmax-$xmin);
-           $told2fdx2=$toldfdx/($xmax-$xmin);
-        }
-
         my $ad='';
         undef  %Apache::functionplotresponse::functionplotrulelabels;
         $Apache::functionplotresponse::functionplotrulelabels{'start'}=400;
@@ -923,7 +918,7 @@ sub end_functionplotresponse {
         } else {
 # We have a function that we can actually grade, go through the spline rules.
            foreach my $rule (@Apache::functionplotresponse::functionplotrules) {
-              unless (&functionplotrulecheck($rule,$xmin,$xmax,$tolfunc,$toldfdx,$told2fdx2)) {
+              unless (&functionplotrulecheck($rule,$xmin,$xmax,$ymin,$ymax)) {
                  $ad='INCORRECT';
                  last;
               }