Diff for /loncom/interface/statistics/lonproblemstatistics.pm between versions 1.6 and 1.9

version 1.6, 2002/07/29 22:01:11 version 1.9, 2002/08/01 20:49:06
Line 56  sub BuildProblemStatisticsPage { Line 56  sub BuildProblemStatisticsPage {
     $Ptr .= '<table border="0"><tbody>';      $Ptr .= '<table border="0"><tbody>';
     $Ptr .= '<tr><td align="right"><b>Select Map</b></td>'."\n";      $Ptr .= '<tr><td align="right"><b>Select Map</b></td>'."\n";
     $Ptr .= '<td align="left">';      $Ptr .= '<td align="left">';
     $Ptr .= &Apache::lonhtmlcommon::MapOptions(\%cache, 'ProblemStatistics');      $Ptr .= &Apache::lonhtmlcommon::MapOptions(\%cache, 'ProblemStatistics',
                                                  'Statistics');
       $Ptr .= '</td></tr>'."\n";
       $Ptr .= '<tr><td align="right"><b>Sorting Type:</b></td>'."\n";
       $Ptr .= '<td align="left">'."\n";
       $Ptr .= &Apache::lonhtmlcommon::AscendOrderOptions(
                                               $cache{'ProblemStatisticsAscend'}, 
                                               'ProblemStatistics',
                                               'Statistics');
     $Ptr .= '</td></tr>'."\n";      $Ptr .= '</td></tr>'."\n";
     $Ptr .= &AscendOrderOptions($cache{'Ascend'});  
     $Ptr .= &ProblemStatisticsButtons($cache{'DisplayFormat'});      $Ptr .= &ProblemStatisticsButtons($cache{'DisplayFormat'});
     $Ptr .= '</table>';      $Ptr .= '</table>';
     $Ptr .= &ProblemStatisticsLegend();      $Ptr .= &ProblemStatisticsLegend();
Line 91  sub BuildProblemStatisticsPage { Line 98  sub BuildProblemStatisticsPage {
     }      }
     #&Apache::lonstatistics::Close_PrgWin($r);      #&Apache::lonstatistics::Close_PrgWin($r);
     
     my ($upper, $lower) = &Discriminant(\%discriminant);      my ($upper, $lower) = &Discriminant(\%discriminant,$r);
     my %Header = (0,"Homework Sets Order",1,"#Stdnts",2,"Tries",3,"Mod",      my %Header = (0,"Homework Sets Order",1,"#Stdnts",2,"Tries",3,"Mod",
                   4,"Mean",5,"#YES",6,"#yes",7,"%Wrng",8,"DoDiff",                    4,"Mean",5,"#YES",6,"#yes",7,"%Wrng",8,"DoDiff",
                   9,"S.D.",10,"Skew.",11,"D.F.1st",12,"D.F.2nd", 13, "Disc.");                    9,"S.D.",10,"Skew.",11,"D.F.1st",12,"D.F.2nd", 13, "Disc.");
     my $color=&setbgcolor(0);      my $color=&setbgcolor(0);
     my %GraphDat= &BuildStatisticsTable(\%cache, \%discriminant, \@list,       my %GraphDat= &BuildStatisticsTable(\%cache, $upper, $lower, \@list, 
                                         \%Header, $students, $r, $color);                                          \%Header, $students, $r, $color);
     untie(%cache);      untie(%cache);
 }  }
Line 157  sub ExtractStudentData { Line 164  sub ExtractStudentData {
     my $TotFirst = 0;      my $TotFirst = 0;
     my $TimeTot = 0;      my $TimeTot = 0;
     my $Discussed=0;      my $Discussed=0;
       my $discrim='';
     my $tempSequenceOrder=100;      my $tempSequenceOrder=100;
   
 #$Apache::lonxml::debug=1;  #$Apache::lonxml::debug=1;
Line 165  sub ExtractStudentData { Line 173  sub ExtractStudentData {
   
     foreach my $sequence (split(':', $cache->{'orderedSequences'})) {      foreach my $sequence (split(':', $cache->{'orderedSequences'})) {
  my $tempProblemOrder=100;   my $tempProblemOrder=100;
  $tempSequenceOrder++;          if($cache->{'ProblemStatisticsMaps'} ne 'All Maps'  &&
 #        if($cache->{'ProblemStatisticsMap'} ne 'All Maps'  &&             $cache->{'ProblemStatisticsMaps'} ne $cache->{$sequence.':title'}) {
 #           $cache->{'ProblemStatisticsMap'} ne $cache->{$sequence.':title'}) {  
 #    $r->print("<br>sequences=$cache->{'orderedSequences'}");  #    $r->print("<br>sequences=$cache->{'orderedSequences'}");
 #    $r->print("<br>".$cache->{$sequence.':title'});  #    $r->print("<br>".$cache->{$sequence.':title'});
 #            next;              next;
 #        }          }
    $tempSequenceOrder++;
   
         my $Dis = '';          #$discrim .= '&';
         foreach my $problemID (split(':', $cache->{$sequence.':problems'})) {          foreach my $problemID (split(':', $cache->{$sequence.':problems'})) {
 #    $r->print("<br>problems=$cache->{$sequence.':problems'}");  #    $r->print("<br>problems=$cache->{$sequence.':problems'}");
             my $problem = $cache->{$problemID.':problem'};              my $problem = $cache->{$problemID.':problem'};
Line 192  sub ExtractStudentData { Line 200  sub ExtractStudentData {
             #}              #}
   
             my %partData=undef;              my %partData=undef;
       $partData{'count'}=0;
             # Initialize part data, display skips correctly              # Initialize part data, display skips correctly
             # Skip refers to when a student made no submissions on that              # Skip refers to when a student made no submissions on that
             # part/problem.              # part/problem.
Line 199  sub ExtractStudentData { Line 208  sub ExtractStudentData {
                                                   $problemID.                                                    $problemID.
                                                   ':parts'})) {                                                    ':parts'})) {
  $tempProblemOrder++;   $tempProblemOrder++;
    $partData{'count'}++;
 # $r->print("<br>parts=$cache->{$sequence.':'.$problemID.':parts'}");  # $r->print("<br>parts=$cache->{$sequence.':'.$problemID.':parts'}");
                 $partData{$part.':order'}=$tempProblemOrder;                  $partData{$part.':order'}=$tempProblemOrder;
                 $partData{$part.':tries'}=0;                  $partData{$part.':tries'}=0;
Line 225  sub ExtractStudentData { Line 235  sub ExtractStudentData {
     $Discussed=0;      $Discussed=0;
                     my $code='-';                      my $code='-';
   
                     $awarded = $cache->{$name.                      $awarded = $cache->{"$name:$Version:$problem:resource.".
                                         "$Version:$problem:resource.".  
                                         "$part.awarded"};                                          "$part.awarded"};
                     $partData{$part.':awarded'} = ($awarded) ? $awarded : 0;                      $partData{$part.':awarded'} = ($awarded) ? $awarded : 0;
                     $totalAwarded += $awarded;                      $totalAwarded += $awarded;
   
                     $tries = $cache->{$name.":$Version:$problem".                      $tries = $cache->{"$name:$Version:$problem".
                                       ":resource.$part.tries"};                                        ":resource.$part.tries"};
                     $partData{$part.':tries'} = ($tries) ? $tries : 0;                      $partData{$part.':tries'} = ($tries) ? $tries : 0;
                     $partData{$part.':wrong'} = $partData{$part.':tries'};                      $partData{$part.':wrong'} = $partData{$part.':tries'};
Line 258  sub ExtractStudentData { Line 267  sub ExtractStudentData {
   
             # Loop through all the parts for the current problem in the               # Loop through all the parts for the current problem in the 
             # correct order and prepare the output              # correct order and prepare the output
       my $partCounter=0;
             foreach (split(/\:/,$cache->{$sequence.':'.$problemID.              foreach (split(/\:/,$cache->{$sequence.':'.$problemID.
                                          ':parts'})) {                                           ':parts'})) {
     #$r->print("<br>".$cache->{$sequence.':title'});   $partCounter++;
   
                 my $Yes = 0;                  my $Yes = 0;
                 if($partData{$_.':code'} eq 'C' ||                   if($partData{$_.':code'} eq 'C' || 
                    $partData{$_.':code'} eq 'O') {                     $partData{$_.':code'} eq 'O') {
Line 270  sub ExtractStudentData { Line 279  sub ExtractStudentData {
                 my $pOrder=$partData{$_.':order'};                  my $pOrder=$partData{$_.':order'};
                 my $ptr = $tempSequenceOrder.':'.$pOrder.':'.$problemID;                  my $ptr = $tempSequenceOrder.':'.$pOrder.':'.$problemID;
   
                 if($_ > 1) {                  if($partData{'count'} > 1) {
                     $ptr .= "*(part $_)";                      $ptr .= "*(part $_)";
                     $Dis .= '&';  
                 }                  }
                   #if($partCounter > 1) {
                       $discrim .= '&';
                   #}
   
  my ($pr_no,$dod)=split('&',$ptr);   my ($pr_no,$dod)=split('&',$ptr);
 # my $DoDiff=$DoDiff->{$dod};  # my $DoDiff=$DoDiff->{$dod};
Line 293  sub ExtractStudentData { Line 304  sub ExtractStudentData {
                     $Discussed=1;                      $Discussed=1;
                 }                  }
                 my $time = $cache->{"$name:$LatestVersion:$problem:timestamp"};                  my $time = $cache->{"$name:$LatestVersion:$problem:timestamp"};
                 $Dis .= $tempSequenceOrder.'@'.$pOrder.'='.$DisF.'+'.$Yes;                  $discrim .= $tempSequenceOrder.'@'.$pOrder.'='.$DisF.'+'.$Yes;
                 $ptr .= '&'.$partData{$_.':tries'}.                  $ptr .= '&'.$partData{$_.':tries'}.
                         '&'.$partData{$_.':wrong'}.                          '&'.$partData{$_.':wrong'}.
                         '&'.$partData{$_.':code'};                          '&'.$partData{$_.':code'};
Line 323  sub ExtractStudentData { Line 334  sub ExtractStudentData {
                 $tempProblemOrder++;                  $tempProblemOrder++;
             }              }
         }          }
  my $pstr;      }
         if($totalTries) {      my $pstr;
     my $DisFac = ($totalAwarded/$totalTries);      if($totalTries) {
     my $DisFactor = sprintf( "%.4f", $DisFac );   my $DisFac = ($totalAwarded/$totalTries);
             my $TS = sprintf( "%.2f", $spent );   my $DisFactor = sprintf( "%.4f", $DisFac );
             my $TS_yes = sprintf( "%.2f", $spent_yes );          my $TS = sprintf( "%.2f", $spent );
    # $DiscFac{$DisFactor}=$Dis;          my $TS_yes = sprintf( "%.2f", $spent_yes );
     $pstr=$DisFactor.':'.$name.':'.$ProbTot.':'.$TotalOpend.':'.   $pstr=$DisFactor.':'.$name.':'.$ProbTot.':'.$TotalOpend.':'.
                   $totalTries.':'.$ProbSolved.':'.$TotFirst.':'.                $totalTries.':'.$ProbSolved.':'.$TotFirst.':'.
                   $TS_yes.':'.$TS.':'.$TotDiscuss;                $TS_yes.':'.$TS.':'.$TotDiscuss;
     (%$discriminant)->{$pstr}=$Dis;   (%$discriminant)->{$pstr}=$discrim;
  }  
     }      }
 }  }
   
Line 354  sub MySort { Line 364  sub MySort {
   
   
 sub BuildStatisticsTable {  sub BuildStatisticsTable {
     my ($cache,$discriminantFactor,$list,$headings,$students,$r,$color)=@_;      my ($cache,$upper,$lower,$list,$headings,$students,$r,$color)=@_;
   
     my $NoElements = scalar @$list;      my $NoElements = scalar @$list;
     my @List=sort(@$list);      my @List=sort(@$list);
   
Line 381  sub BuildStatisticsTable { Line 390  sub BuildStatisticsTable {
     my $cIdx=0;      my $cIdx=0;
   
     foreach my $sequence (split(':', $cache->{'orderedSequences'})) {      foreach my $sequence (split(':', $cache->{'orderedSequences'})) {
           if($cache->{'ProblemStatisticsMaps'} ne 'All Maps'  &&
         &CreateProblemStatisticsTableHeading(1,$cache->{$sequence.':source'},             $cache->{'ProblemStatisticsMaps'} ne $cache->{$sequence.':title'}) {
               next;
           }
           &CreateProblemStatisticsTableHeading($cache->{'DisplayFormat'}, 
                                                $cache->{$sequence.':source'},
                                              $cache->{$sequence.':title'},                                                $cache->{$sequence.':title'}, 
                                              $headings,$r);                                               $headings,$r);
   
  my ($tar,$Tries,$Wrongs,$Code,$Disc)=split(/\&/,   my ($tar,$Tries,$Wrongs,$Code,$Disc)=split(/\&/,
                                                    $List[$cIdx]);                                                     $List[$cIdx]);
   
         my ($SqOrd,$PrOrd,$Prob)=split(/\:/,$tar);          my ($SqOrd,$PrOrd,$Prob)=split(/\:/,$tar);
  $sequence+=100;   $sequence+=100;
  while ($SqOrd==$sequence && $cIdx<$NoElements) {   while ($SqOrd==$sequence && $cIdx<$NoElements) {
Line 427  sub BuildStatisticsTable { Line 438  sub BuildStatisticsTable {
     $Temp = '<a href="'.$cache->{$ResId.':source'}.      $Temp = '<a href="'.$cache->{$ResId.':source'}.
                 '" target="_blank">'.$cache->{$ResId.':title'}.$Dummy.'</a>';                  '" target="_blank">'.$cache->{$ResId.':title'}.$Dummy.'</a>';
   
     my $res = &Apache::lonnet::declutter($cache->{'src_'.$ResId});      #$r->print('<br>rid='.$ResId);
       #$r->print('<br>src='.$cache->{$ResId.':source'});
       #$r->print('<br>tit='.$cache->{$ResId.':title'});
   
       my $res = &Apache::lonnet::declutter($cache->{$ResId.':source'});
     my $urlres=$res;      my $urlres=$res;
   
     $ResId=~/(\d+)\.(\d+)/;      $ResId=~/(\d+)\.(\d+)/;
Line 481  sub BuildStatisticsTable { Line 496  sub BuildStatisticsTable {
     }      }
   
 #--------------------- Compute the Discrimination Factors  #--------------------- Compute the Discrimination Factors
             my ($Up1,$Up2)=split(/\:/,':');#jason$DisUp->{$lpr[$i]});              my ($Up1,$Up2)=split(/\:/,$upper->{$sequence.'@'.$pOrd});
     my ($Lw1,$Lw2)=split(/\:/,':');#jason$DisLow->{$lpr[$i]});      my ($Lw1,$Lw2)=split(/\:/,$lower->{$sequence.'@'.$pOrd});
   
     my $Dis1 = $Up1 - $Lw1;      my $Dis1 = $Up1 - $Lw1;
     my $Dis2 = $Up2 - $Lw2;      my $Dis2 = $Up2 - $Lw2;
     my $_D1 = sprintf("%.2f", $Dis1);      my $_D1 = sprintf("%.2f", $Dis1);
Line 578  sub TableRow { Line 594  sub TableRow {
     my($PrOrd,$Temp,$StdNo,$TotalTries,$MxTries,$Avg,$YES,$Override,      my($PrOrd,$Temp,$StdNo,$TotalTries,$MxTries,$Avg,$YES,$Override,
        $Wrng,$DoD,$SD,$Sk,$_D1,$_D2,$DiscNo,$Prob)=split(/\&/,$Str);         $Wrng,$DoD,$SD,$Sk,$_D1,$_D2,$DiscNo,$Prob)=split(/\&/,$Str);
 #    $r->print('<br>'.$Str);  #    $r->print('<br>'.$Str);
     if ($ENV{'form.showcsv'}) {      my $Ptr;
       if($cache->{'DisplayFormat'} eq 'Display CSV Format') {
         my ($ResId,$Dummy)=split(/\*/,$Prob);          my ($ResId,$Dummy)=split(/\*/,$Prob);
         my $Ptr =  "\n".'<br>'.          $Ptr="\n".'<br>'.
                "\n".'"'.($RealIdx+1).'",'.               "\n".'"'.($RealIdx+1).'",'.
                "\n".'"'.$cache->{'title_'.$ResId}.$Dummy.'",'.               "\n".'"'.$cache->{'title_'.$ResId}.$Dummy.'",'.
                "\n".'"'.$cache->{'src_'.$ResId}.'",'.               "\n".'"'.$cache->{'src_'.$ResId}.'",'.
                "\n".'"'.$StdNo.'",'.               "\n".'"'.$StdNo.'",'.
                "\n".'"'.$TotalTries.'",'.               "\n".'"'.$TotalTries.'",'.
                "\n".'"'.$MxTries.'",'.               "\n".'"'.$MxTries.'",'.
                "\n".'"'.$Avg.'",'.               "\n".'"'.$Avg.'",'.
                "\n".'"'.$YES.'",'.               "\n".'"'.$YES.'",'.
                "\n".'"'.$Override.'",'.               "\n".'"'.$Override.'",'.
                "\n".'"'.$Wrng.'",'.               "\n".'"'.$Wrng.'",'.
                "\n".'"'.$DoD.'",'.               "\n".'"'.$DoD.'",'.
                "\n".'"'.$SD.'",'.               "\n".'"'.$SD.'",'.
                "\n".'"'.$Sk.'",'.               "\n".'"'.$Sk.'",'.
                "\n".'"'.$_D1.'",'.               "\n".'"'.$_D1.'",'.
        "\n".'"'.$_D2.'"'.       "\n".'"'.$_D2.'"'.
        "\n".'"'.$DiscNo.'"';       "\n".'"'.$DiscNo.'"';
   
         $r->print("\n".$Ptr);          $r->print("\n".$Ptr);
     }      } else {
     else{          $Ptr="\n".'<tr>'.
         my $Ptr =  "\n".'<tr>'.               "\n".'<td>'.($RealIdx+1).'</td>'.
                "\n".'<td>'.($RealIdx+1).'</td>'.            #  "\n".'<td>'.$PrOrd.$Temp.'</td>'.
           #     "\n".'<td>'.$PrOrd.$Temp.'</td>'.               "\n".'<td>'.$Temp.'</td>'.
                "\n".'<td>'.$Temp.'</td>'.               "\n".'<td bgcolor='.$color->{"yellow"}.'> '.$StdNo.'</td>'.
                "\n".'<td bgcolor='.$color->{"yellow"}.'> '.$StdNo.'</td>'.               "\n".'<td bgcolor='.$color->{"yellow"}.'>'.$TotalTries.'</td>'.
                "\n".'<td bgcolor='.$color->{"yellow"}.'>'.$TotalTries.'</td>'.               "\n".'<td bgcolor='.$color->{"yellow"}.'>'.$MxTries.'</td>'.
                "\n".'<td bgcolor='.$color->{"yellow"}.'>'.$MxTries.'</td>'.               "\n".'<td bgcolor='.$color->{"gb"}.'>'.$Avg.'</td>'.
                "\n".'<td bgcolor='.$color->{"gb"}.'>'.$Avg.'</td>'.               "\n".'<td bgcolor='.$color->{"gb"}.'> '.$YES.'</td>'.
                "\n".'<td bgcolor='.$color->{"gb"}.'> '.$YES.'</td>'.               "\n".'<td bgcolor='.$color->{"gb"}.'> '.$Override.'</td>'.
                "\n".'<td bgcolor='.$color->{"gb"}.'> '.$Override.'</td>'.               "\n".'<td bgcolor='.$color->{"red"}.'> '.$Wrng.'</td>'.
                "\n".'<td bgcolor='.$color->{"red"}.'> '.$Wrng.'</td>'.               "\n".'<td bgcolor='.$color->{"red"}.'> '.$DoD.'</td>'.
                "\n".'<td bgcolor='.$color->{"red"}.'> '.$DoD.'</td>'.               "\n".'<td bgcolor='.$color->{"green"}.'> '.$SD.'</td>'.
                "\n".'<td bgcolor='.$color->{"green"}.'> '.$SD.'</td>'.               "\n".'<td bgcolor='.$color->{"green"}.'> '.$Sk.'</td>'.
                "\n".'<td bgcolor='.$color->{"green"}.'> '.$Sk.'</td>'.               "\n".'<td bgcolor='.$color->{"purple"}.'> '.$_D1.'</td>'.
                "\n".'<td bgcolor='.$color->{"purple"}.'> '.$_D1.'</td>'.       "\n".'<td bgcolor='.$color->{"purple"}.'> '.$_D2.'</td>'.
        "\n".'<td bgcolor='.$color->{"purple"}.'> '.$_D2.'</td>'.               "\n".'<td bgcolor='.$color->{"yellow"}.'> '.$DiscNo.'</td>';
                "\n".'<td bgcolor='.$color->{"yellow"}.'> '.$DiscNo.'</td>';  
         $r->print("\n".$Ptr.'</tr>' );          $r->print("\n".$Ptr.'</tr>' );
     }      }
 #    $GraphDat->{$RealIdx}=$DoD.':'.$Wrng;  #    $GraphDat->{$RealIdx}=$DoD.':'.$Wrng;
Line 645  sub setbgcolor { Line 661  sub setbgcolor {
     return \%color;      return \%color;
 }  }
   
   
 sub StatusOptions {  
     my ($cache)=@_;  
   
     my $Status = $cache->{'Status'};  
     my $OpSel1 = '';  
     my $OpSel2 = '';  
     my $OpSel3 = '';  
   
     if($Status eq 'Any')         { $OpSel3 = ' selected'; }  
     elsif($Status eq 'Expired' ) { $OpSel2 = ' selected'; }  
     else                         { $OpSel1 = ' selected'; }  
   
     my $Ptr = '';  
     $Ptr .= '<tr><td align="right"><b>Student Status:</b></td>'."\n";  
     $Ptr .= '<td align="left"><select name="Status">';  
     $Ptr .= '<option'.$OpSel1.'>Active</option>'."\n";  
     $Ptr .= '<option'.$OpSel2.'>Expired</option>'."\n";  
     $Ptr .= '<option'.$OpSel3.'>Any</option>'."\n";  
     $Ptr .= '</select></td></tr>'."\n";  
   
     return $Ptr;  
 }  
   
 sub AscendOrderOptions {  
     my ($order)=@_;  
   
     my $OpSel1 = '';  
     my $OpSel2 = '';  
   
     if($order eq 'Ascending') {  
         $OpSel1 = ' selected';  
     } else {  
         $OpSel2 = ' selected';  
     }  
   
     my $Ptr = '';  
     $Ptr .= '<tr><td align="right"><b>Sorting Type:</b></td>'."\n";  
     $Ptr .= '<td align="left"><select name="Ascend">'."\n";  
     $Ptr .= '<option'.$OpSel1.'>Ascending</option>'."\n".  
     '<option'.$OpSel2.'>Descending</option>'."\n";  
     $Ptr .= '</select></td></tr>'."\n";  
   
     return $Ptr;  
 }  
   
 sub ProblemStatisticsButtons {  sub ProblemStatisticsButtons {
     my ($displayFormat)=@_;      my ($displayFormat)=@_;
   
     my $Ptr = '<tr><td></td><td align="left">';      my $Ptr = '<tr><td></td><td align="left">';
     $Ptr .= '<input type=submit name="ProblemStatisticsRecalculate" ';  
     $Ptr .= 'value="Recalculate Statistics"/>'."\n";  
     $Ptr .= '&nbsp;&nbsp;&nbsp;';  
     $Ptr .= '<input type="submit" name="DoDiffGraph" ';      $Ptr .= '<input type="submit" name="DoDiffGraph" ';
     $Ptr .= 'value="DoDiff Graph" />'."\n";      $Ptr .= 'value="DoDiff Graph" />'."\n";
     $Ptr .= '&nbsp;&nbsp;&nbsp;';      $Ptr .= '&nbsp;&nbsp;&nbsp;';
Line 706  sub ProblemStatisticsButtons { Line 673  sub ProblemStatisticsButtons {
     $Ptr .= '&nbsp;&nbsp;&nbsp;';      $Ptr .= '&nbsp;&nbsp;&nbsp;';
     $Ptr .= '<input type="submit" name="DisplayCSVFormat" ';      $Ptr .= '<input type="submit" name="DisplayCSVFormat" ';
     if($displayFormat eq 'Display CSV Format') {      if($displayFormat eq 'Display CSV Format') {
         $Ptr .= 'value="Display CSV Format" />'."\n";  
     } else {  
         $Ptr .= 'value="Display Table Format" />'."\n";          $Ptr .= 'value="Display Table Format" />'."\n";
       } else {
           $Ptr .= 'value="Display CSV Format" />'."\n";
     }      }
     $Ptr .= '</td></tr>';      $Ptr .= '</td></tr>';
   
Line 773  sub ProblemStatisticsLegend { Line 740  sub ProblemStatisticsLegend {
   
 #------- Processing upperlist and lowerlist according to each problem  #------- Processing upperlist and lowerlist according to each problem
 sub ProcessDiscriminant {  sub ProcessDiscriminant {
     my ($List) = @_;      my ($List,$r) = @_;
     my @sortedList = sort (@$List);      my @sortedList = sort (@$List);
     my $Count = scalar @sortedList;      my $Count = scalar @sortedList;
     my $Problem;      my $Problem;
Line 786  sub ProcessDiscriminant { Line 753  sub ProcessDiscriminant {
     my $nStudent=0;      my $nStudent=0;
     my %Proc=undef;      my %Proc=undef;
     while ($nIndex<$Count) {      while ($nIndex<$Count) {
   #        $r->print("<br> $nIndex) $sortedList[$nIndex]");
  ($Problem,$tmp)=split(/\=/,$sortedList[$nIndex]);   ($Problem,$tmp)=split(/\=/,$sortedList[$nIndex]);
  @Dis=split(/\+/,$tmp);   @Dis=split(/\+/,$tmp);
  my $Temp = $Problem;   my $Temp = $Problem;
Line 797  sub ProcessDiscriminant { Line 765  sub ProcessDiscriminant {
     ($Problem,$tmp)=split(/\=/,$sortedList[$nIndex]);      ($Problem,$tmp)=split(/\=/,$sortedList[$nIndex]);
     @Dis=split(/\+/,$tmp);      @Dis=split(/\+/,$tmp);
  } while ( $Problem eq $Temp && $nIndex < $Count );   } while ( $Problem eq $Temp && $nIndex < $Count );
 # $Proc{$Temp}=($Sum1/$nStudent).':'.$nStudent;  
  $Proc{$Temp}=($Sum1/$nStudent).':'.($Sum2/$nStudent);   $Proc{$Temp}=($Sum1/$nStudent).':'.($Sum2/$nStudent);
 #       $r->print("$nIndex) $Temp --> ($nStudent) $Proc{$Temp} <br>");  #        $r->print("<br> $nIndex) $Temp --> ($nStudent) $Proc{$Temp}");
  $Sum1=0;   $Sum1=0;
  $Sum2=0;   $Sum2=0;
  $nStudent=0;   $nStudent=0;
Line 810  sub ProcessDiscriminant { Line 777  sub ProcessDiscriminant {
   
 #------- Creating Discimination factor     #------- Creating Discimination factor   
 sub Discriminant {  sub Discriminant {
     my ($discriminantFactor)=@_;      my ($discriminant,$r)=@_;
     my @discriminantKeys=keys(%$discriminantFactor);  #$Apache::lonxml::debug=1;
   #&Apache::lonhomework::showhash(%$discriminant);
   #$Apache::lonxml::debug=0;
       my @discriminantKeys=keys(%$discriminant);
     my $Count = scalar @discriminantKeys;      my $Count = scalar @discriminantKeys;
   
     my $UpCnt = int(0.27*$Count);      my $UpCnt = int(0.27*$Count);
Line 825  sub Discriminant { Line 795  sub Discriminant {
  $Count++;       $Count++;    
  if($low < $UpCnt || $Count > $up) {   if($low < $UpCnt || $Count > $up) {
             $low++;              $low++;
             my $str=$discriminantFactor->{$key};              my $str=$discriminant->{$key};
             foreach(split(/\:/,$str)){              foreach(split(/\&/,$str)){
                 if($_) {                  if($_) {
                     if($low<$UpCnt) { push(@LowList,$_); }                      if($low<$UpCnt) { push(@LowList,$_); }
                     else            { push(@UpList,$_);  }                      else            { push(@UpList,$_);  }
Line 834  sub Discriminant { Line 804  sub Discriminant {
             }              }
         }          }
     }      }
     my %DisUp =  &ProcessDiscriminant(\@UpList);      my %DisUp =  &ProcessDiscriminant(\@UpList,$r);
     my %DisLow = &ProcessDiscriminant(\@LowList);      my %DisLow = &ProcessDiscriminant(\@LowList,$r);
   
     return (\%DisUp, \%DisLow);      return (\%DisUp, \%DisLow);
 }  }   
   
      
   
 #---- END Problem Statistics Web Page ----------------------------------------  #---- END Problem Statistics Web Page ----------------------------------------
   
Line 852  sub BuildDiffGraph { Line 820  sub BuildDiffGraph {
     my ($r)=@_;      my ($r)=@_;
   
     my $graphData = &GetGraphData('DiffGraph', $r);      my $graphData = &GetGraphData('DiffGraph', $r);
     return '<IMG src="/cgi-bin/graph.gif?'.$graphData.'" />';      $r->print('<IMG src="/cgi-bin/graph.gif?'.$graphData.'" />');
   
       return;
 }  }
   
 sub BuildWrongGraph {  sub BuildWrongGraph {
     my ($r)=@_;      my ($r)=@_;
   
     my $graphData = &GetGraphData('WrongGraph', $r);      my $graphData = &GetGraphData('WrongGraph', $r);
     return '<IMG src="/cgi-bin/graph.gif?'.$graphData.'" />';      $r->print('<IMG src="/cgi-bin/graph.gif?'.$graphData.'" />');
   
       return;
 }  }
   
   

Removed from v.1.6  
changed lines
  Added in v.1.9


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