[BACK]Return to lonproblemanalysis.pm CVS log [TXT] [DIR] Up to [LON-CAPA] / loncom / interface / statistics

Diff for /loncom/interface/statistics/lonproblemanalysis.pm between versions 1.98 and 1.103

version 1.98, 2004/10/29 15:52:32 version 1.103, 2004/11/10 16:56:39
Line 152  sub BuildProblemAnalysisPage { Line 152  sub BuildProblemAnalysisPage {
                                          $problem_data,                                           $problem_data,
                                          \@Students);                                           \@Students);
             } elsif ($current_problem->{'resptype'} eq 'numerical') {              } elsif ($current_problem->{'resptype'} eq 'numerical') {
                 ##                   &numerical_response_analysis($r,$current_problem,
                 ## analyze all responses of a problem at once                                               $problem_data,\@Students);
                 my $res = $current_problem->{'resource'};  
                 foreach my $partid (@{$res->{'parts'}}) {  
                     $current_problem->{'part'} = $partid;  
                     foreach my $respid (@{$res->{'partdata'}->{$partid}->{'ResponseIds'}}) {  
                         $current_problem->{'respid'}=$respid;  
                         &NumericalResponseAnalysis($r,$current_problem,  
                                                    $problem_data,\@Students);  
                     }  
                 }  
             } else {              } else {
                 $r->print('<h2>This analysis is not supported</h2>');                  $r->print('<h2>Analysis of '.$current_problem->{'resptype'}.' is not supported</h2>');
             }              }
         }          }
         $r->print('<hr />');          $r->print('<hr />');
Line 178  sub BuildProblemAnalysisPage { Line 169  sub BuildProblemAnalysisPage {
     }      }
 }  }
   
   
 #########################################################  #########################################################
 #########################################################  #########################################################
 ##  ##
Line 186  sub BuildProblemAnalysisPage { Line 176  sub BuildProblemAnalysisPage {
 ##  ##
 #########################################################  #########################################################
 #########################################################  #########################################################
 sub NumericalResponseAnalysis {  sub numerical_response_analysis {
     my ($r,$problem,$problem_data,$Students) = @_;      my ($r,$problem,$problem_analysis,$students) = @_;
     my $c = $r->connection();      my $c = $r->connection();
     my ($resource,$partid,$respid) = ($problem->{'resource'},  
                                       $problem->{'part'},  
                                       $problem->{'respid'});  
     #      #
     if (scalar(@{$resource->{'parts'}})>1) {      if ($ENV{'form.AnalyzeOver'} !~ /^(tries|time)$/) {
         if (@{$resource->{'partdata'}->{$partid}->{'ResponseIds'}}>1) {          $r->print('Bad request');
             $r->print('<h3>'.  
                       &mt('Part [_1], response [_2].',$partid,$respid).  
                       '</h3>');  
         } else {   
             $r->print('<h3>'.  
                       &mt('Part [_1]',$partid,$respid).  
                       '</h3>');  
         }  
     } elsif (@{$resource->{'partdata'}->{$partid}->{'ResponseIds'}}>1) {  
         $r->print('<h3>'.&mt('Response [_1]',$respid).'</h3>');  
     }      }
     #      #
     my $analysis_html;      my ($resource,$partid,$respid) = ($problem->{'resource'},
     my $PerformanceData = &Apache::loncoursedata::get_response_data                                        $problem->{'part'},
                                         $problem->{'respid'});
       # Gather student data
       my $response_data = &Apache::loncoursedata::get_response_data
         (\@Apache::lonstatistics::SelectedSections,          (\@Apache::lonstatistics::SelectedSections,
          $Apache::lonstatistics::enrollment_status,           $Apache::lonstatistics::enrollment_status,
          $resource->{'symb'},$respid);           $resource->{'symb'},$respid);
     if (! defined($PerformanceData) ||       #
         ref($PerformanceData) ne 'ARRAY' ) {      for (my $plot_num = 1;$plot_num<=$ENV{'form.NumPlots'};$plot_num++) {
         $analysis_html = '<h2>'.          my $restriction_function;
             &mt('There is no submission data for this resource').          $restriction_function = sub {($_[0]->{'tries'} == $plot_num?1:0)};
             '</h2>';          #
           my ($correct,$intervals,$answers) = 
               &numerical_response_determine_intervals($r,$resource,$partid,
                                                       $respid,$students);
           if ($c->aborted()) { return; };
           #
           my $responses = &classify_response_data($response_data,
                                                   $correct,
                                                   $restriction_function);
           my $student_count = $responses->{'_students'};
           my $correct_count = $responses->{'_correct'};
           my $total_count   = $responses->{'_count'};
           my $max           = $responses->{'_max'};
           #
           my $analysis_html = '<h2>'.&mt('Attempt [_1]',$plot_num).'</h2>';
           $analysis_html.= &numerical_one_dimensional_plot($r,600,150,
                                                            scalar(@$students),
                                                            $answers);
           $analysis_html.= '<table><tr>'.
               '<td valign="top" align="center">'.
               &numerical_plot_percent($r,$responses).'</td>'.
               '<td align="center" valign="top">'.
               &numerical_plot_differences($r,$responses).'</td>'.
               '</tr>'.$/.'</table>';
         $r->print($analysis_html);          $r->print($analysis_html);
         return;  
     }      }
     #      #
     # This next call causes all the waiting around that people complain about  
     &Apache::lonstathelpers::GetStudentAnswers($r,$problem,$Students,  
                                                'Statistics',  
                                                'stats_status');  
     return if ($c->aborted());  
     #  
     # Collate the data  
     my %Data;  
     foreach my $student (@$Students) {  
         my $answer = $student->{'answer'};  
         $Data{$answer}++;  
     }  
     my @Labels = sort {$a <=> $b } keys(%Data);  
     my @PlotData = @Data{@Labels};  
     #  
     my $width  = 500;   
     my $height = 100;  
     my $plot = &one_dimensional_plot($r,500,100,scalar(@$Students),  
                                      \@Labels,\@PlotData);  
     $r->print($plot);  
     return;      return;
 }  }
   
 sub one_dimensional_plot {  sub numerical_plot_percent {
     my ($r,$width,$height,$N,$Labels,$Data)=@_;      my ($r,$responses) = @_;
     #      #
     # Compute data -> image scaling factors      my $total = $responses->{'_count'};
     my $min = $Labels->[0];      return '' if ($total == 0);
     my $max = $Labels->[-1];      my $minbin = 0.5;
     if ($max == $min) {      while (my ($interval,$submissions) = each(%$responses)) {
         $max =$min+1;          next if ($interval =~ /^_/);
           my ($ans,$ans_low,$ans_high) = split(" ",$interval);
           my $low_percent  = abs(($ans-$ans_low)/$ans);
           my $high_percent = abs(($ans_high-$ans)/$ans);
           if ($minbin > $high_percent) { $minbin = $high_percent; }
           if ($minbin > $low_percent) { $minbin = $low_percent; }
       }    
       #
       my @bins;
       if ($minbin < 1) {
           @bins = (0.1, ".5",1,1.5,2,2.5,3,4,5,10,20,50,100);        
       } elsif ($minbin < 2) {
           @bins = (0.5,1,1.5,2,2.5,3,4,5,10,20,50,100);
       } elsif ($minbin < 5) {
           @bins = (1,2,3,4,5,10,25,50,75,100,200);
       } elsif ($minbin < 10) {
           @bins = (2,4,6,8,10,12,15,20,25,30,50,75,100,200);
       } else {
           @bins = (5,10,15,20,25,30,50,75,100,200);
       }
       my @labels = (1..scalar(@bins));
       #
       my @correct;
       my @incorrect;
       while (my ($interval,$submissions) = each(%$responses)) {
           next if ($interval =~ /^_/);
           my ($ans,$ans_low,$ans_high) = split(" ",$interval);
           while (my ($submission,$count) = each(%$submissions)) {
               my $scaled_value = abs(($submission-$ans)/$ans);
               my $bin=0;
               for ($bin=0;$bin<$#bins;$bin++) { # not <= for a reason
                   last if ($bins[$bin]>$scaled_value);
               }
               if (($submission <= $ans_high) && ($submission >= $ans_low)) {
                   $correct[$bin]+=$count;
               } else {
                   $incorrect[$bin]+=$count;
               }
           }
       }
       #
       for (my $i=0;$i<=$#bins;$i++) {
           if (! defined(  $correct[$i])) {   $correct[$i]=0;}
           if (! defined($incorrect[$i])) { $incorrect[$i]=0;}
           $correct[$i]*=100/$total;
           $incorrect[$i]*=100/$total;
       }
       my $title = &mt('Distribution by Percent');
       my $graph = &Apache::loncommon::DrawBarGraph
           ($title,'Percent difference from correct','Number of answers',
            100,['#33FF00','#FF3300'],\@labels,\@correct,\@incorrect,
            {xskip=>1});
       #
       my $table = $graph.'<table><tr><th>'.&mt('Bar').'</th>'.
           '<th colspan="3">'.&mt('Range').'</th>';
       for (my $i=0;$i<=$#bins;$i++) {
           my $lownum;
           if ($i == 0) {
               $lownum = 0;
           } else {
               $lownum = $bins[$i-1];
           }
           my $highnum = $bins[$i];
           $table .= 
               '<tr>'.
               '<td>'.$labels[$i].'</td>'.
               '<td align="right">'.$lownum.'</td>'.
               '<td>&nbsp;-&nbsp;</td>'.
               '<td align="right">'.$highnum.'</td>'.'</tr>'.$/;
       }
       $table.= '</table>'.$/;
       return $table;
   }
   
   sub numerical_plot_differences {
       my ($r,$responses) = @_;
       #
       my $total = $responses->{'_count'};
       return '' if ($total == 0);
       my $minbin = undef;
       my $maxbin = undef;
       while (my ($interval,$submissions) = each(%$responses)) {
           next if ($interval =~ /^_/);
           my ($ans,$ans_low,$ans_high) = split(" ",$interval);
           my $low_diff  = abs($ans-$ans_low);
           my $high_diff = abs($ans_high-$ans);
           if (! defined($maxbin)) { $maxbin = $low_diff;}
           if (! defined($minbin)) { $minbin = $low_diff;}
           #
           if ($minbin > $high_diff) { $minbin = $high_diff; }
           if ($minbin > $low_diff ) { $minbin = $low_diff; }
           #
           if ($maxbin < $high_diff) { $maxbin = $high_diff; }
           if ($maxbin < $low_diff ) { $maxbin = $low_diff; }
       }    
       #
       my @bins;
       my @labels;
       # Hmmmm, should switch to absolute difference
       for (my $i=1;$i<=20;$i++) {
           push(@bins,$i*$minbin/2);
           push(@labels,$i);
       }
       #
       my @correct;
       my @incorrect;
       while (my ($interval,$submissions) = each(%$responses)) {
           next if ($interval =~ /^_/);
           my ($ans,$ans_low,$ans_high) = split(" ",$interval);
           while (my ($submission,$count) = each(%$submissions)) {
               my $value = abs($submission-$ans);
               my $bin=0;
               for ($bin=0;$bin<$#bins;$bin++) { # not <= for a reason
                   last if ($bins[$bin]>$value);
               }
               if (($submission <= $ans_high) && ($submission >= $ans_low)) {
                   $correct[$bin]+=$count;
               } else {
                   $incorrect[$bin]+=$count;
               }
           }
     }      }
     my $h_scale = ($width-10)/($max-$min);  
     #      #
       for (my $i=0;$i<=$#bins;$i++) {
           if (! defined(  $correct[$i])) {   $correct[$i]=0;}
           if (! defined($incorrect[$i])) { $incorrect[$i]=0;}
           $correct[$i]*=100/$total;
           $incorrect[$i]*=100/$total;
       }
       my $title = &mt('Distribution by Magnitude');
       my $graph = &Apache::loncommon::DrawBarGraph
           ($title,'magnitude difference from correct','Number of answers',
            100,['#33FF00','#FF3300'],\@labels,\@correct,\@incorrect,{xskip=>1});
       #
       my $table = $graph.'<table><tr><th>'.&mt('Bar').'</th>'.
           '<th colspan="3">'.&mt('Range').'</th>';
       for (my $i=0;$i<=$#bins;$i++) {
           my $lownum;
           if ($i == 0) {
               $lownum = 0;
           } else {
               $lownum = $bins[$i-1];
           }
           my $highnum = $bins[$i];
           $table .= 
               '<tr>'.
               '<td>'.$labels[$i].'</td>'.
               '<td align="right">'.$lownum.'</td>'.
               '<td>&nbsp;-&nbsp;</td>'.
               '<td align="right">'.$highnum.'</td>'.'</tr>'.$/;
       }
       $table.= '</table>'.$/;
       return $table;
   }
   
   sub numerical_response_determine_intervals {
       my ($r,$resource,$partid,$respid,$students)=@_;
       my $c = $r->connection();
       #
       # FIX ME: Need progress dialog updates
       #
       # Read in the cache (if it exists) before we start timing things.
       &Apache::lonstathelpers::ensure_proper_cache($resource->{'symb'});
       #
       my $correct;
       # %intervals differs from %answers because it may be possible for two
       # students to have the same correct answer but different intervals.
       my %intervals; 
       my %answers;
       foreach my $student (@$students) {
           last if ($c->aborted());
           my $sname = $student->{'username'};
           my $sdom = $student->{'domain'};
           # analyze problem
           my $analysis = 
               &Apache::lonstathelpers::analyze_problem_as_student($resource,
                                                                   $sname,
                                                                   $sdom);
           # make the key
           my $key = $partid.'.'.$respid;
           my $interval = join(' ',($analysis->{$key.'.answer'}->[0],
                                    $analysis->{$key.'.ans_low'}->[0],
                                    $analysis->{$key.'.ans_high'}->[0]));
           $correct->{$sname.':'.$sdom} = $interval;
           $intervals{$interval}++;
           $answers{$analysis->{$key.'.answer'}->[0]}++;
       }
       &Apache::lonstathelpers::write_analysis_cache();
       return ($correct,\%intervals,\%answers);
   }
   
   sub numerical_one_dimensional_plot {
       my ($r,$width,$height,$n,$data)=@_;
       #
       # Compute data -> image scaling factors
     my $max_y = 0;      my $max_y = 0;
     foreach (@$Data) {      my $min_x = undef;
         $max_y = $_ if ($max_y < $_);      my $max_x = undef;
       while (my ($answer,$count) = each(%$data)) {
           $max_y = $count if ($max_y < $count);
           if (! defined($min_x) || $answer < $min_x) {
               $min_x = $answer;
           }
           if (! defined($max_x) || $answer > $max_x) {
               $max_x = $answer;
           }
     }      }
       #
       my $h_scale = ($width-10)/($max_x-$min_x);
       #
     my $ticscale = 5;      my $ticscale = 5;
     if ($max_y * $ticscale > $height/2) {      if ($max_y * $ticscale > $height/2) {
         $ticscale = int($height/2/$max_y);          $ticscale = int($height/2/$max_y);
Line 268  sub one_dimensional_plot { Line 452  sub one_dimensional_plot {
     # Create the plot      # Create the plot
     my $plot =       my $plot = 
         qq{<drawimage width="$width" height="$height" bgcolor="transparent" >};          qq{<drawimage width="$width" height="$height" bgcolor="transparent" >};
     for (my $idx=0;$idx<scalar(@$Labels);$idx++) {      while (my ($answer,$count) = each(%$data)) {
         my $xloc = 5+$h_scale*($Labels->[$idx] - $min);          my $xloc = 5+$h_scale*($answer - $min_x);
         my $top    = $height/2-$Data->[$idx]*$ticscale;          my $top    = $height/2-$count*$ticscale;
         my $bottom = $height/2+$Data->[$idx]*$ticscale;          my $bottom = $height/2+$count*$ticscale;
         $plot .=           $plot .= &line($xloc,$top,$xloc,$bottom,'888888',1);
             &line($xloc,$top,$xloc,$bottom,'888888',1);  
     }      }
     #      #
     # Put the scale on last to ensure it is on top of the data.      # Put the scale on last to ensure it is on top of the data.
     if ($min < 0 && $max > 0) {      if ($min_x < 0 && $max_x > 0) {
         my $circle_x = 5+$h_scale*abs($min);  # '0' in data coordinates          my $circle_x = 5+$h_scale*abs($min_x);  # '0' in data coordinates
         my $r = 4;          my $r = 4;
         $plot .= &line(5,$height/2,$circle_x-$r,$height/2,'000000',1);          $plot .= &line(5,$height/2,$circle_x-$r,$height/2,'000000',1);
         $plot .= &circle($circle_x,$height/2,$r+1,'000000');          $plot .= &circle($circle_x,$height/2,$r+1,'000000');
Line 289  sub one_dimensional_plot { Line 472  sub one_dimensional_plot {
     $plot .= '</drawimage>';      $plot .= '</drawimage>';
     my $plotresult =  &Apache::lonxml::xmlparse($r,'web',$plot);      my $plotresult =  &Apache::lonxml::xmlparse($r,'web',$plot);
           
     my $title = 'Distribution of correct answers';  
     my $result = '<table>'.      my $result = '<table>'.
         '<tr><td colspan="3" align="center">'.          '<tr><td colspan="3" align="center">'.
         '<font size="+2">'.$title.' (N='.$N.')'.          '<font size="+2">'.&mt('Distribution of correct answers').'</font>'.
         '</font>'.          '<br />'.&mt('[_1] students, [_2] distinct correct answers',
                        $n,scalar(keys(%$data))).
           '<br />'.&mt('Maximum number of coinciding values: [_1]',$max_y).
         '</td></tr>'.          '</td></tr>'.
         '<tr>'.          '<tr>'.
         '<td valign="center">'.$min.'</td>'.          '<td valign="center">'.$min_x.'</td>'.
         '<td>'.$plotresult.'</td>'.          '<td>'.$plotresult.'</td>'.
         '<td valign="center">'.$max.'</td>'.          '<td valign="center">'.$max_x.'</td>'.
         '</tr>'.          '</tr>'.
         '<tr><td colspan="3" align="center">'.  
         'Maximum Number of Coinciding Values: '.$max_y.  
         '</td></tr>'.  
         '</table>';          '</table>';
     return $result;      return $result;
 }  }
Line 312  sub one_dimensional_plot { Line 493  sub one_dimensional_plot {
 ## These should probably go somewhere more suitable soon.  ## These should probably go somewhere more suitable soon.
 sub line {  sub line {
     my ($x1,$y1,$x2,$y2,$color,$thickness) = @_;      my ($x1,$y1,$x2,$y2,$color,$thickness) = @_;
     return qq{<line x1="$x1" y1="$y1" x2="$x2" y2="$y2" color="$color" thickness="$thickness" />$/};      return qq{<line x1="$x1" y1="$y1" x2="$x2" y2="$y2" color="$color" thickness="$thickness" />};
 }  }
   
 sub text {  sub text {
Line 461  sub radio_response_analysis { Line 642  sub radio_response_analysis {
                  &get_time_from_row($response_data->[-1]),                   &get_time_from_row($response_data->[-1]),
                  $plot_num);                   $plot_num);
             $pre_graph_text =               $pre_graph_text = 
                 'Data from [_5] to [_6], [_2] submissions, [_3] correct, [_4] incorrect';                  'Data from [_6] to [_7]<br /> [_2] submissions from [_5] students, [_3] correct, [_4] incorrect';
             $extra_data[0] = &Apache::lonlocal::locallocaltime($starttime);              $extra_data[0] = &Apache::lonlocal::locallocaltime($starttime);
             $extra_data[1] = &Apache::lonlocal::locallocaltime($endtime);              $extra_data[1] = &Apache::lonlocal::locallocaltime($endtime);
             #              #
Line 485  sub radio_response_analysis { Line 666  sub radio_response_analysis {
             $no_data_text = 'No data for [_5] to [_6]';              $no_data_text = 'No data for [_5] to [_6]';
         }          }
         my $foil_choice_data =          my $foil_choice_data =
             &RR_classify_response_data($response_data,$correct,              &classify_response_data($response_data,$correct,
                                        $restriction_function);                                      $restriction_function);
         # &Apache::lonstathelpers::log_hash_ref($foil_choice_data);          # &Apache::lonstathelpers::log_hash_ref($foil_choice_data);
         my $answers;          my $answers;
         if (ref($correct)) {          if (ref($correct)) {
Line 528  sub radio_response_analysis { Line 709  sub radio_response_analysis {
                 '<font size="+1">'.                  '<font size="+1">'.
                 &mt($pre_graph_text,                  &mt($pre_graph_text,
                     $plot_num,$foil_choice_data->{'_count'},                      $plot_num,$foil_choice_data->{'_count'},
                     $correct,                                          $correct,
                     $foil_choice_data->{'_count'}-$correct,                      $foil_choice_data->{'_count'}-$correct,
                       $foil_choice_data->{'_students'},
                     @extra_data).                      @extra_data).
                     '</td></tr>'.$/;                      '</td></tr>'.$/;
             $analysis_html.=              $analysis_html.=
Line 631  sub RR_concept_plot { Line 813  sub RR_concept_plot {
 sub RR_create_percent_selected_plot {  sub RR_create_percent_selected_plot {
     my ($concepts,$foils,$foil_data,$title) = @_;      my ($concepts,$foils,$foil_data,$title) = @_;
     #      #
     my %foil_selections;      if ($foil_data->{'_count'} == 0) { return ''; };
     my %true;      my %correct_selections;
       my %incorrect_selections;
     foreach my $foil (@$foils) {      foreach my $foil (@$foils) {
         # foil_data has format $foil_data->{true_foil}->{selected foil}          # foil_data has format $foil_data->{true_foil}->{selected foil}=count
         next if (! exists($foil_data->{$foil}));          next if (! exists($foil_data->{$foil}));
         $true{$foil}++;  
         while (my ($f,$count)= each(%{$foil_data->{$foil}})) {          while (my ($f,$count)= each(%{$foil_data->{$foil}})) {
             $foil_selections{$f}+=$count;              if ($f eq $foil) {
                   $correct_selections{$foil} += $count;
               } else {
                   $incorrect_selections{$f} += $count;
               }
         }          }
     }      }
     #       # 
Line 647  sub RR_create_percent_selected_plot { Line 833  sub RR_create_percent_selected_plot {
     my @correct;      my @correct;
     my @incorrect;      my @incorrect;
     #      #
     my $total =0;      my $total = $foil_data->{'_count'};
     for (my $i=0;$i<scalar(@$foils);$i++) {      for (my $i=0;$i<scalar(@$foils);$i++) {
         my $foil = $foils->[$i];          my $foil = $foils->[$i];
         if ($true{$foil}) {          $correct[$i]   = $correct_selections{$foil};
             $correct[$i]   = $foil_selections{$foil};          $incorrect[$i] = $incorrect_selections{$foil};
             $incorrect[$i] = 0;  
         } else {  
             $correct[$i]   = 0;  
             $incorrect[$i] = $foil_selections{$foil};  
         }  
         $total+=$foil_selections{$foil};  
     }      }
     if ($total == 0) { return ''; };  
     for (my $i=0;$i<=$#correct;$i++) {       for (my $i=0;$i<=$#correct;$i++) { 
         $correct[$i] = sprintf('%0f',$correct[$i]/$total*100);          $correct[$i] = sprintf('%2f',$correct[$i]/$total*100);
     }      }
     for (my $i=0;$i<=$#incorrect;$i++) {      for (my $i=0;$i<=$#incorrect;$i++) {
         $incorrect[$i] = sprintf('%0f',$incorrect[$i]/$total*100);          $incorrect[$i] = sprintf('%2f',$incorrect[$i]/$total*100);
     }      }
     #      #
     # Put a blank in the data sets between concepts, if there are concepts      # Put a blank in the data sets between concepts, if there are concepts
Line 759  sub RR_create_stacked_selection_plot { Line 938  sub RR_create_stacked_selection_plot {
     return ($graph,\%count_per_foil);      return ($graph,\%count_per_foil);
 }  }
   
   
   #########################################################
   #########################################################
   ##
   ##       Misc routines
   ##
   #########################################################
   #########################################################
   
 # if $correct is a hash ref, it is assumed to be indexed by student names.  # if $correct is a hash ref, it is assumed to be indexed by student names.
 #    the values are assumed to be hash refs with a key of 'answer'.  #    the values are assumed to be hash refs with a key of 'answer'.
 sub RR_classify_response_data {  sub classify_response_data {
     my ($full_row_data,$correct,$function) = @_;      my ($full_row_data,$correct,$function) = @_;
     my %submission_data;      my %submission_data;
       my %students;
       my $max=0;
     foreach my $row (@$full_row_data) {      foreach my $row (@$full_row_data) {
         my %subm = &hashify_attempt($row);          my %subm = &hashify_attempt($row);
         if (ref($correct) eq 'HASH') {          if (ref($correct) eq 'HASH') {
Line 773  sub RR_classify_response_data { Line 963  sub RR_classify_response_data {
         }          }
         $subm{'submission'} =~ s/=\d+\s*$//;          $subm{'submission'} =~ s/=\d+\s*$//;
         if (&$function(\%subm)) {          if (&$function(\%subm)) {
               $students{$subm{'student'}}++;
             $submission_data{'_count'}++;              $submission_data{'_count'}++;
               
             if (&submission_is_correct($subm{'award'})) {               if (&submission_is_correct($subm{'award'})) { 
                 $submission_data{'_correct'}++;                  $submission_data{'_correct'}++;
             }              }
             $submission_data{$subm{'correct'}}->{$subm{'submission'}}++;              
               if($max<++$submission_data{$subm{'correct'}}->{$subm{'submission'}}) {
                   $max=$submission_data{$subm{'correct'}}->{$subm{'submission'}};
               }
         }          }
     }      }
       $submission_data{'_max'} = $max;
       $submission_data{'_students'}=scalar(keys(%students));
     return \%submission_data;      return \%submission_data;
 }  }
   
Line 1102  sub OR_time_analysis { Line 1299  sub OR_time_analysis {
             &OR_time_process_data($performance_data,$begin_index,$end_index);              &OR_time_process_data($performance_data,$begin_index,$end_index);
         ##          ##
         $table .= '<tr><td colspan="4" align="center"><font size="+1">'.          $table .= '<tr><td colspan="4" align="center"><font size="+1">'.
             &mt('[_1] submissions from [_2] students submitting, [_3] correct, [_4] incorrect',              &mt('[_1] submissions from [_2] students, [_3] correct, [_4] incorrect',
                 $data_count,$student_count,$correct,$data_count-$correct).                  $data_count,$student_count,$correct,$data_count-$correct).
                 '</font></td></tr>'.$/;                  '</font></td></tr>'.$/;
         my $concept_correct_plot = '';          my $concept_correct_plot = '';
Line 1236  sub OR_time_process_data { Line 1433  sub OR_time_process_data {
     if ($end_index == scalar(@$performance_data)-1) {      if ($end_index == scalar(@$performance_data)-1) {
         $end_index++;          $end_index++;
     }      }
     &Apache::lonnet::logthis('  '.$begin_index.':'.$end_index);  
     my $count;      my $count;
     for (my $i=$begin_index;$i<$end_index;$i++) {      for (my $i=$begin_index;$i<$end_index;$i++) {
         my $attempt = $performance_data->[$i];          my $attempt = $performance_data->[$i];
Line 1256  sub OR_time_process_data { Line 1452  sub OR_time_process_data {
             }              }
         }          }
     }      }
     &Apache::lonnet::logthis('count = '.$count);  
     return (\%processed_time_data,$correct,$data_count,      return (\%processed_time_data,$correct,$data_count,
             scalar(keys(%distinct_students)));              scalar(keys(%distinct_students)));
 }  }
Removed from v.1.98  
changed lines
  Added in v.1.103

FreeBSD-CVSweb <freebsd-cvsweb@FreeBSD.org>