--- loncom/interface/statistics/lonproblemstatistics.pm	2002/11/01 22:09:39	1.35
+++ loncom/interface/statistics/lonproblemstatistics.pm	2003/08/29 21:11:25	1.57
@@ -1,7 +1,6 @@
 # The LearningOnline Network with CAPA
-# (Publication Handler
 #
-# $Id: lonproblemstatistics.pm,v 1.35 2002/11/01 22:09:39 minaeibi Exp $
+# $Id: lonproblemstatistics.pm,v 1.57 2003/08/29 21:11:25 matthew Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -26,477 +25,790 @@
 # http://www.lon-capa.org/
 #
 # (Navigate problems for statistical reports
-# YEAR=2001
-# 5/5,7/9,7/25/1,8/11,9/13,9/26,10/5,10/9,10/22,10/26 Behrouz Minaei
-# 11/1,11/4,11/16,12/14,12/16,12/18,12/20,12/31 Behrouz Minaei
-# YEAR=2002
-# 1/22,2/1,2/6,2/25,3/2,3/26,4/7,5/6 Behrouz Minaei
-# 5/12,5/26,7/16,7/29,8/5,10/31  Behrouz Minaei
 #
-###
+###############################################
+###############################################
 
-package Apache::lonproblemstatistics; 
-
-use strict;
-use Apache::lonnet();
-use Apache::lonhtmlcommon;
-use Apache::loncoursedata;
-use GDBM_File;
+=pod
 
+=head1 NAME
 
-sub InitializeProblemStatistics {
-    my ($cacheDB, $students, $courseID, $c, $r)=@_;
-    my %cache;
-
-    unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER(),0640)) {
-        $r->print('Unable to tie database1.');
-        return ('ERROR', undef);
-    }
-
-    # Remove students who don't have the proper section.
-    my @sectionsSelected = split(':',$cache{'sectionsSelected'});
-    for(my $studentIndex=((scalar @$students)-1); $studentIndex>=0;
-        $studentIndex--) {
-        my $value = $cache{$students->[$studentIndex].':section'};
-        my $found = 0;
-        foreach (@sectionsSelected) {
-            if($_ eq 'none') {
-                if($value eq '' || !defined($value) || $value eq ' ') {
-                    $found = 1;
-                    last;
-                }
-            } else {
-                if($value eq $_) {
-                    $found = 1;
-                    last;
-                }
-            }
-        }
-        if($found == 0) {
-            splice(@$students, $studentIndex, 1);
-        }
-    }
+lonproblemstatistics
 
-    my $isNotCached = 0;
-    my $lastStatus = (defined($cache{'StatisticsLastStatus'})) ?
-                     $cache{'StatisticsLastStatus'} : 'Nothing';
-    my $whichStudents = join(':::',sort(@$students));
-    if(!defined($cache{'StatisticsCached'}) ||
-       $lastStatus ne $cache{'Status'} ||
-       $whichStudents ne $cache{'StatisticsWhichStudents'}) {
-        $isNotCached = 1;
-    }
+=head1 SYNOPSIS
 
-    untie(%cache);
-    unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_WRCREAT(),0640)) {
-        $r->print('Unable to tie database.2');
-        return ('ERROR', undef);
-    }
-    if($isNotCached && defined($cache{'StatisticsCached'})) {
-        my @statkeys = split(':::', $cache{'StatisticsKeys'});
-        delete $cache{'StatisticsKeys'};
-        delete $cache{'StatisticsCached'};
-        foreach(@statkeys) {
-            delete $cache{$_};
-        }
-    }
+Routines to present problem statistics to instructors via tables,
+Excel files, and plots.
 
-    untie(%cache);
-    if($isNotCached) {
-        &Apache::loncoursedata::DownloadStudentCourseDataSeparate($students,
-                                                                  'true',
-                                                                  $cacheDB,
-                                                                  'true',
-                                                                  'true',
-                                                                  $courseID,
-                                                                  $r, $c);
-    }
-    if($c->aborted()) { return ('ERROR', undef); }
+=over 4
 
-    unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER(),0640)) {
-        $r->print('Unable to tie database.3');
-        return ('ERROR', undef);
-    }
-    my $problemData;
-    if($isNotCached) {
-        ($problemData) = &ExtractStudentData(\%cache, $students);
-        &CalculateStatistics($problemData, \%cache, $courseID);
-    }
-    untie(%cache);
+=cut
 
-    unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_WRCREAT(),0640)) {
-        $r->print('Unable to tie database.4');
-        return ('ERROR', undef);
-    }
-    if($isNotCached) {
-        foreach(keys(%$problemData)) {
-            $cache{$_} = $problemData->{$_};
-        }
-        $cache{'StatisticsKeys'} = join(':::', keys(%$problemData));
-        $cache{'StatisticsCached'} = 'true';
-        $cache{'StatisticsLastStatus'} = $cache{'Status'};
-        $cache{'StatisticsWhichStudents'} = $whichStudents;
-    }
-    untie(%cache);
+###############################################
+###############################################
 
-    unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER(),0640)) {
-        $r->print('Unable to tie database.5');
-        return ('ERROR', undef);
-    }
+package Apache::lonproblemstatistics;
 
-    my $orderedProblems = &SortProblems(\%cache,
-                                        $cache{'ProblemStatisticsSort'},
-                                        $cache{'SortProblems'},
-                                        $cache{'ProblemStatisticsAscend'});
-    untie(%cache);
+use strict;
+use Apache::lonnet();
+use Apache::lonhtmlcommon;
+use Apache::loncoursedata;
+use Apache::lonstatistics;
+use Spreadsheet::WriteExcel;
 
-    return ('OK', $orderedProblems);
-}
+my @Fields = (
+           { name => 'problem_num',
+             title => 'P#',
+             align => 'right',
+             color => '#FFFFE6' },
+           { name   => 'container',
+             title  => 'Sequence or Folder',
+             align  => 'left',
+             color  => '#FFFFE6',
+             sortable => 'yes' },
+           { name   => 'title',
+             title  => 'Title',
+             align  => 'left',
+             color  => '#FFFFE6',
+             special  => 'link',
+             sortable => 'yes', },
+           { name   => 'part', 
+             title  => 'Part',
+             align  => 'left',
+             color  => '#FFFFE6',
+             },
+           { name   => 'num_students',
+             title  => '#Stdnts',
+             align  => 'right',
+             color  => '#EEFFCC',
+             format => '%d',
+             sortable  => 'yes',
+             graphable => 'yes',
+             long_title => 'Number of Students Attempting Problem' },
+           { name   => 'tries',
+             title  => 'Tries',
+             align  => 'right',
+             color  => '#EEFFCC',
+             format => '%d',
+             sortable  => 'yes',
+             graphable => 'yes',
+             long_title => 'Total Number of Tries' },
+           { name   => 'max_tries',
+             title  => 'Max Tries',
+             align  => 'right',
+             color  => '#DDFFFF',
+             format => '%d',
+             sortable  => 'yes',
+             graphable => 'yes',
+             long_title => 'Maximum Number of Tries' },
+           { name   => 'mean_tries',
+             title  => 'Mean Tries',
+             align  => 'right',
+             color  => '#DDFFFF',
+             format => '%5.2f',
+             sortable  => 'yes',
+             graphable => 'yes',
+             long_title => 'Average Number of Tries' },
+           { name   => 'std_tries',
+             title  => 'S.D. tries',
+             align  => 'right',
+             color  => '#DDFFFF',
+             format => '%5.2f',
+             sortable  => 'yes',
+             graphable => 'yes',
+             long_title => 'Standard Deviation of Number of Tries' },
+           { name   => 'skew_tries',
+             title  => 'Skew Tries',
+             align  => 'right',
+             color  => '#DDFFFF',
+             format => '%5.2f',
+             sortable  => 'yes',
+             graphable => 'yes',
+             long_title => 'Skew of Number of Tries' },
+           { name   => 'deg_of_diff',
+             title  => 'DoDiff',
+             align  => 'right',
+             color  => '#DDFFFF',
+             format => '%5.2f',
+             sortable  => 'yes',
+             graphable => 'yes',
+             long_title => 'Degree of Difficulty'.
+                           '[ 1 - ((#YES+#yes) / Tries) ]'},
+           { name   => 'num_solved',
+             title  => '#YES',
+             align  => 'right',
+             color  => '#FFDDDD',
+             format => '%d',
+             sortable  => 'yes',
+             graphable => 'yes',
+             long_title => 'Number of Students able to Solve' },
+           { name   => 'num_override',
+             title  => '#yes',
+             align  => 'right',
+             color  => '#FFDDDD',
+             format => '%d',
+             sortable  => 'yes',
+             graphable => 'yes',
+             long_title => 'Number of Students given Override' },
+           { name   => 'per_wrong',
+             title  => '%Wrng',
+             align  => 'right',
+             color  => '#FFFFE6',
+             format => '%4.1f',
+             sortable  => 'yes',
+             graphable => 'yes',
+             long_title => 'Percent of students whose final answer is wrong' },
+);
+
+###############################################
+###############################################
+
+=pod 
+
+=item &CreateInterface()
+
+Create the main intereface for the statistics page.  Allows the user to
+select sections, maps, and output.
+
+=cut
+
+###############################################
+###############################################
+my @OutputOptions = 
+    (
+     { name  => 'problem statistics grouped by sequence',
+       value => 'HTML problem statistics grouped',
+       description => 'Output statistics for the problem parts.',
+       mode => 'html',
+       show => 'grouped',
+     },
+     { name  => 'problem statistics ungrouped',
+       value => 'HTML problem statistics ungrouped',
+       description => 'Output statistics for the problem parts.',
+       mode => 'html',
+       show => 'ungrouped',
+     },
+     { name  => 'problem statistics, Excel',
+       value => 'Excel problem statistics',
+       description => 'Output statistics for the problem parts '.
+           'in an Excel workbook',
+       mode => 'excel',
+       show => 'all',
+     },
+     );
+
+sub CreateInterface {
+    my $Str = '';
+    $Str .= '<table cellspacing="5">'."\n";
+    $Str .= '<tr>';
+    $Str .= '<td align="center"><b>Sections</b></td>';
+    $Str .= '<td align="center"><b>Enrollment Status</b></td>';
+    $Str .= '<td align="center"><b>Sequences and Folders</b></td>';
+    $Str .= '<td align="center"><b>Output</b></td>';
+    $Str .= '</tr>'."\n";
+    #
+    $Str .= '<tr><td align="center">'."\n";
+    $Str .= &Apache::lonstatistics::SectionSelect('Section','multiple',5);
+    $Str .= '</td><td align="center">';
+    $Str .= &Apache::lonhtmlcommon::StatusOptions(undef,undef,5);
+    $Str .= '</td><td align="center">';
+    #
+    my $only_seq_with_assessments = sub { 
+        my $s=shift;
+        if ($s->{'num_assess'} < 1) { 
+            return 0;
+        } else { 
+            return 1;
+        }
+    };
+    $Str .= &Apache::lonstatistics::MapSelect('Maps','multiple,all',5,
+                                              $only_seq_with_assessments);
+    $Str .= '</td><td>'."\n";
+    my ($html,$outputmode,$show) = 
+        &Apache::lonstatistics::CreateAndParseOutputSelector(
+                                            'statsoutputmode',
+                                            'HTML problem statistics grouped',
+                                            @OutputOptions);
+    $Str .= $html;
+    $Str .= '</td></tr>'."\n";
+    $Str .= '</table>'."\n";
+    $Str .= '<input type="submit" value="Generate Statistics" />';
+    $Str .= '&nbsp;'x5;
+    $Str .= '<input type="submit" name="ClearCache" value="Clear Caches" />';
+    $Str .= '&nbsp;'x5;
+    return ($Str,$outputmode,$show);
+}
+
+###############################################
+###############################################
+
+=pod 
 
-sub BuildProblemStatisticsPage {
-    my ($cacheDB, $students, $courseID, $c, $r)=@_;
+=item &BuildProblemStatisticsPage()
 
-    my @Header = ("Homework Sets Order","#Stdnts","Tries","Mod",
-                  "Mean","#YES","#yes","%Wrng","DoDiff",
-                  "S.D.","Skew.","D.F.1st","D.F.2nd");
-    my $color=&setbgcolor(0);
-    my %cache;
+Main interface to problem statistics.
 
-    unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER(),0640)) {
-        $r->print('Unable to tie database.6');
-        return;
-    }
-    my $Ptr = '';
-    $Ptr .= '<table border="0" cellspacing="5"><tbody>';
-    $Ptr .= '<tr><td align="right"><b>Select Map</b></td>'."\n";
-    $Ptr .= '<td align="left">';
-    $Ptr .= &Apache::lonhtmlcommon::MapOptions(\%cache, 'Statistics',
-                                               '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 .= '<tr><td align="right"><b>Select Sections</b>';
-    $Ptr .= '</td>'."\n";
-    $Ptr .= '<td align="left">'."\n";
-    my @sections = split(':',$cache{'sectionList'});
-    my @sectionsSelected = split(':',$cache{'sectionsSelected'});
-    $Ptr .= &Apache::lonhtmlcommon::MultipleSectionSelect(\@sections,
-                                                          \@sectionsSelected,
-                                                          'Statistics');
-    $Ptr .= '</td></tr>'."\n";
-    $Ptr .= &ProblemStatisticsButtons($cache{'DisplayFormat'},
-                                      $cache{'DisplayLegend'},
-                                      $cache{'SortProblems'});
-    $Ptr .= '</table>';
-    if($cache{'DisplayLegend'} eq 'Show Legend') {
-        $Ptr .= &ProblemStatisticsLegend();
-    }
-    $r->print($Ptr);
-    $r->rflush();
-    untie(%cache);
+=cut
 
-    my ($result, $orderedProblems) =
-        &InitializeProblemStatistics($cacheDB, $students, $courseID, $c, $r);
-    if($result ne 'OK') {
+###############################################
+###############################################
+sub BuildProblemStatisticsPage {
+    my ($r,$c)=@_;
+    #
+    my ($interface,$output_mode,$show) = &CreateInterface();
+    $r->print($interface);
+    $r->print('<input type="hidden" name="statsfirstcall" value="no" />');
+    $r->print('<input type="hidden" name="sortby" value="'.$ENV{'form.sortby'}.
+              '" />');
+    $r->print('<input type="hidden" name="plot" value="" />');
+    if (! exists($ENV{'form.statsfirstcall'})) {
         return;
     }
-
-    unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER(),0640)) {
-        $r->print('Unable to tie database.6');
-        return;
+    #
+    &Apache::lonstatistics::Gather_Student_Data($r);
+    #
+    #
+    if ($output_mode eq 'html') {
+        $r->print("<h2>".
+                  $ENV{'course.'.$ENV{'request.course.id'}.'.description'}.
+                  "</h2>\n");
+        $r->print("<h3>".localtime(time)."</h3>");
+        $r->rflush();
+        if ($show eq 'grouped') {
+            &output_html_grouped_by_sequence($r);
+        } elsif ($show eq 'ungrouped') {
+            &output_html_ungrouped($r);
+        }
+    } elsif ($output_mode eq 'excel') {
+        $r->print("<h2>Preparing Excel Spreadsheet</h2>");
+        &output_excel($r);
+    } else {
+        $r->print("<h1>Not implemented</h1>");
     }
-    &BuildStatisticsTable(\%cache, $cache{'DisplayFormat'},
-                          $cache{'SortProblems'}, $orderedProblems,
-                          \@Header, $r, $color);
-    untie(%cache);
-
     return;
 }
 
-sub BuildGraphicChart {
-    my ($graph,$cacheDB,$courseDescription,$students,$courseID,$r,$c)=@_;
-    my %cache;
-    my $max = 0;
-    unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER(),0640)) {
-        return 'Unable to tie database.7';
-    }
+###############################################
+###############################################
 
-    my $title = '';
-    if($graph eq 'DoDiffGraph') {
-	$title = 'Degree-of-Difficulty';
-    } else {
-	$title = 'Wrong-Percentage';
-    }
+=pod 
 
-    my $currentSequence = -1;
-    my $sortProblems = 'Sort Within Sequence';
+=item &output_html_grouped_by_sequence()
 
-    my ($result, $orderedProblems) =
-        &InitializeProblemStatistics($cacheDB, $students, $courseID, $c, $r);
-    if($result ne 'OK') {
-        return;
+Presents the statistics data as an html table organized by the order
+the assessments appear in the course.
+
+=cut
+
+###############################################
+###############################################
+sub output_html_grouped_by_sequence {
+    my ($r) = @_;
+    my $problem_num = 0;
+    #$r->print(&ProblemStatisticsLegend());
+    foreach my $sequence (&Apache::lonstatistics::Sequences_with_Assess()) {
+        next if ($sequence->{'num_assess'}<1);
+        $r->print("<h3>".$sequence->{'title'}."</h3>");
+        $r->print('<table border="0"><tr><td bgcolor="#777777">'."\n");
+        $r->print('<table border="0" cellpadding="3">'."\n");
+        $r->print('<tr bgcolor="#FFFFE6">');
+        my $Str = &statistics_table_header('no container no plots');
+        $r->print('<tr bgcolor="#FFFFE6">'.$Str."</tr>\n");
+        foreach my $resource (@{$sequence->{'contents'}}) {
+            next if ($resource->{'type'} ne 'assessment');
+            foreach my $part (@{$resource->{'parts'}}) {
+                $problem_num++;
+                my $data = &get_statistics($sequence,$resource,$part,
+                                           $problem_num);
+                my $option = '';
+                $r->print('<tr>'.&statistics_html_table_data($data,
+                                                             'no container').
+                          "</tr>\n");
+            }
+        }
+        $r->print("</table>\n");
+        $r->print("</td></tr></table>\n");
+        $r->rflush();
     }
+    #
+    return;
+}
 
-    my @values = ();
+###############################################
+###############################################
 
-    foreach(@$orderedProblems) {
-        my ($sequence,$problem,$part)=split(':', $_);
-        if($cache{'StatisticsMaps'} ne 'All Maps'  &&
-           $cache{'StatisticsMaps'} ne $cache{$sequence.':title'}) {
-             next;
-        }
-
-        if( $currentSequence == -1 ||
-            ($sortProblems eq 'Sort Within Sequence' &&
-            $currentSequence != $sequence)) {
-	    if($currentSequence != -1) {
-		&DrawGraph(\@values,$courseDescription,$title,$max,$r);
-	    }
-            if($sortProblems eq 'Sort Within Sequence') {
-                $r->print('<b>'.$cache{$sequence.':title'}.'</b>');
-            }
+=pod 
+
+=item &output_html_ungrouped()
 
-            $currentSequence = $sequence;
-            @values = ();
+Presents the statistics data in a single html table which can be sorted by
+different columns.
+
+=cut
+
+###############################################
+###############################################
+sub output_html_ungrouped {
+    my ($r,$option) = @_;
+    #
+    if (exists($ENV{'form.plot'}) && $ENV{'form.plot'} ne '') {
+        &plot_statistics($r,$ENV{'form.plot'});
+    }
+    #
+    my $problem_num = 0;
+    my $show_container = 0;
+    my $show_part = 0;
+    #$r->print(&ProblemStatisticsLegend());
+    my $sortby = undef;
+    foreach my $field (@Fields) {
+        if ($ENV{'form.sortby'} eq $field->{'name'}) {
+            $sortby = $field->{'name'};
+        }
+    }
+    if (! defined($sortby) || $sortby eq '' || $sortby eq 'problem_num') {
+        $sortby = 'container';
+    }
+    # If there is more than one sequence, list their titles
+    my @Sequences = &Apache::lonstatistics::Sequences_with_Assess();
+    if (@Sequences < 1) {
+        $option .= ' no container';
+    }
+    #
+    # Compile the data
+    my @Statsarray;
+    foreach my $sequence (@Sequences) {
+        next if ($sequence->{'num_assess'}<1);
+        foreach my $resource (@{$sequence->{'contents'}}) {
+            next if ($resource->{'type'} ne 'assessment');
+            foreach my $part (@{$resource->{'parts'}}) {
+                $problem_num++;
+                my $data = &get_statistics($sequence,$resource,$part,
+                                           $problem_num);
+                $show_part = 1 if ($part ne '0');
+                #
+                push (@Statsarray,$data);
+            }
         }
-        my $data = 0;
-        if($graph eq 'DoDiffGraph') {
-            $data = sprintf("%.2f", $cache{$_.':degreeOfDifficulty'}),
+    }
+    #
+    # Sort the data
+    my @OutputOrder;
+    if ($sortby eq 'container') {
+        @OutputOrder = @Statsarray;
+    } else {
+        # $sortby is already defined, so we can charge ahead
+        if ($sortby =~ /^(title|part)$/i) {
+            # Alpha comparison
+            @OutputOrder = sort {
+                lc($a->{$sortby}) cmp lc($b->{$sortby}) ||
+                    lc($a->{'title'}) cmp lc($b->{'title'}) ||
+                        lc($a->{'part'}) cmp lc($b->{'part'});
+            } @Statsarray;
         } else {
-            $data = sprintf("%.1f", $cache{$_.':percentWrong'}),
+            # Numerical comparison
+            @OutputOrder = sort {
+                my $retvalue = 0;
+                if ($b->{$sortby} eq 'nan') {
+                    if ($a->{$sortby} ne 'nan') {
+                        $retvalue = -1;
+                    } else {
+                        $retvalue = 0;
+                    }
+                }
+                if ($a->{$sortby} eq 'nan') {
+                    if ($b->{$sortby} ne 'nan') {
+                        $retvalue = 1;
+                    }
+                }
+                if ($retvalue eq '0') {
+                    $retvalue = $b->{$sortby} <=> $a->{$sortby} ||
+                                lc($a->{'title'}) <=> lc($b->{'title'}) ||
+                                lc($a->{'part'})  <=> lc($b->{'part'});
+                }
+                $retvalue;
+            } @Statsarray;
         }
-        if($max < $data) {
-            $max = $data;
+    }
+    $option .= 'no part' if (! $show_part);
+    my $num_output = 0;
+    #
+    # output the headers
+    $r->print('<table border="0"><tr><td bgcolor="#777777">'."\n");
+    $r->print('<table border="0" cellpadding="3">'."\n");
+    my $Str = &statistics_table_header($option.' sortable');
+    $r->print('<tr bgcolor="#FFFFE6">'.$Str."</tr>\n");
+    #
+    foreach my $rowdata (@OutputOrder) {
+        $num_output++;
+        if ($num_output % 25 == 0) {
+            $r->print("</table>\n</td></tr></table>\n");
+            #
+            $r->print('<table border="0"><tr><td bgcolor="#777777">'."\n");
+            $r->print('<table border="0" cellpadding="3">'."\n");
+            my $Str = &statistics_table_header($option.' sortable');
+            $r->print('<tr bgcolor="#FFFFE6">'.$Str."</tr>\n");
+            $r->rflush();
         }
-        push(@values, $data);
+        $r->print('<tr>'.&statistics_html_table_data($rowdata,$option).
+                  "</tr>\n");
     }
-    untie(%cache);
-
-    &DrawGraph(\@values,$courseDescription,$title,$max,$r);
-
+    $r->print("</table>\n");
+    $r->print("</td></tr></table>\n");
+    $r->rflush();
+    #
     return;
 }
 
+###############################################
+###############################################
 
-sub DrawGraph {
-    my ($values,$courseDescription,$title,$Max,$r)=@_;
-    my $sendValues = join(',', @$values);
-    my $sendCount = scalar(@$values);
+=pod 
 
-    if ( $Max > 1 ) {
-	if ($Max % 10) {
-	    $Max++;
-	}
-    #(10 - $Max % 10);                                                                      
-	$Max = int($Max);
-    } else { $Max = 1; }
-
-    my @GData = ($courseDescription, 'Problem_number',
-                 $title, $Max, $sendCount, $sendValues);
-    $r->print('</form>'."\n");
-    $r->print('<IMG src="/cgi-bin/graph.gif?'.
-              (join('&', @GData)).'" border="1" />');
-    $r->print('<form>'."\n");
-}
-
-#---- Problem Statistics Web Page ---------------------------------------
-
-sub CreateProblemStatisticsTableHeading {
-    my ($headings,$r)=@_;
-
-    my $Str='';
-    $Str .= '<tr>'."\n";
-    $Str .= '<th bgcolor="#ffffe6">P#</th>'."\n";
-    foreach(@$headings) {
-	$Str .= '<th bgcolor="#ffffe6">';
-        $Str .= '<a href="/adm/statistics?reportSelected=';
-        $Str .= &Apache::lonnet::escape('Problem Statistics');
-        $Str .= '&ProblemStatisticsSort=';
-        $Str .= &Apache::lonnet::escape($_).'">'.$_.'</a>&nbsp</th>'."\n";
-    }
-    $Str .= "\n".'</tr>'."\n";
-
-    return $Str;
-}
-
-sub BuildStatisticsTable {
-    my ($cache,$displayFormat,$sortProblems,$orderedProblems,$headings,
-        $r,$color)=@_;
-
-    my $count = 1;
-    my $currentSequence = -1;
-    foreach(@$orderedProblems) {
-        my ($sequence,$problem,$part)=split(':', $_); 
-        if($cache->{'StatisticsMaps'} ne 'All Maps'  &&
-           $cache->{'StatisticsMaps'} ne $cache->{$sequence.':title'}) {
-            next;
-        }
-
-        if($currentSequence == -1 ||
-           ($sortProblems eq 'Sort Within Sequence' &&
-            $currentSequence != $sequence)) {
-            if($displayFormat ne 'Display CSV Format') {
-                if($currentSequence ne -1) {
-                    $r->print('</table>');
-                    $r->print('</td></tr></table><br>');
-                }
-                if($sortProblems eq 'Sort Within Sequence') {
-                    $r->print('<b>'.$cache->{$sequence.':title'}.'</b>');
+=item &output_excel()
+
+Presents the statistical data in an Excel 95 compatable spreadsheet file.
+
+=cut
+
+###############################################
+###############################################
+sub output_excel {
+    my ($r) = @_;
+    my $filename = '/prtspool/'.
+        $ENV{'user.name'}.'_'.$ENV{'user.domain'}.'_'.
+            time.'_'.rand(1000000000).'.xls';
+    #
+    my $excel_workbook = undef;
+    my $excel_sheet = undef;
+    #
+    my $rows_output = 0;
+    my $cols_output = 0;
+    #
+    # Create sheet
+    $excel_workbook = Spreadsheet::WriteExcel->new('/home/httpd'.$filename);
+    #
+    # Check for errors
+    if (! defined($excel_workbook)) {
+        $r->log_error("Error creating excel spreadsheet $filename: $!");
+        $r->print("Problems creating new Excel file.  ".
+                  "This error has been logged.  ".
+                  "Please alert your LON-CAPA administrator");
+        return ;
+    }
+    #
+    # The excel spreadsheet stores temporary data in files, then put them
+    # together.  If needed we should be able to disable this (memory only).
+    # The temporary directory must be specified before calling 'addworksheet'.
+    # File::Temp is used to determine the temporary directory.
+    $excel_workbook->set_tempdir($Apache::lonnet::tmpdir);
+    #
+    # Add a worksheet
+    my $sheetname = $ENV{'course.'.$ENV{'request.course.id'}.'.description'};
+    if (length($sheetname) > 31) {
+        $sheetname = substr($sheetname,0,31);
+    }
+    $excel_sheet = $excel_workbook->addworksheet($sheetname);
+    #
+    # Put the course description in the header
+    $excel_sheet->write($rows_output,$cols_output++,
+                   $ENV{'course.'.$ENV{'request.course.id'}.'.description'});
+    $cols_output += 3;
+    #
+    # Put a description of the sections listed
+    my $sectionstring = '';
+    my @Sections = @Apache::lonstatistics::SelectedSections;
+    if (scalar(@Sections) > 1) {
+        if (scalar(@Sections) > 2) {
+            my $last = pop(@Sections);
+            $sectionstring = "Sections ".join(', ',@Sections).', and '.$last;
+        } else {
+            $sectionstring = "Sections ".join(' and ',@Sections);
+        }
+    } else {
+        if ($Sections[0] eq 'all') {
+            $sectionstring = "All sections";
+        } else {
+            $sectionstring = "Section ".$Sections[0];
+        }
+    }
+    $excel_sheet->write($rows_output,$cols_output++,$sectionstring);
+    $cols_output += scalar(@Sections);
+    #
+    # Put the date in there too
+    $excel_sheet->write($rows_output,$cols_output++,
+                        'Compiled on '.localtime(time));
+    #
+    $rows_output++; 
+    $cols_output=0;
+    #
+    # Long Headersheaders
+    foreach my $field (@Fields) {
+        next if ($field->{'name'} eq 'problem_num');
+        if (exists($field->{'long_title'})) {
+            $excel_sheet->write($rows_output,$cols_output++,
+                                $field->{'long_title'});
+        } else {
+            $excel_sheet->write($rows_output,$cols_output++,'');
+        }
+    }
+    $rows_output++;
+    $cols_output=0;
+    # Brief headers
+    foreach my $field (@Fields) {
+        next if ($field->{'name'} eq 'problem_num');
+        $excel_sheet->write($rows_output,$cols_output++,$field->{'title'});
+    }
+    $rows_output++;
+    #
+    # Write the data
+    my $problem_num=0;
+    foreach my $sequence (&Apache::lonstatistics::Sequences_with_Assess()) {
+        next if ($sequence->{'num_assess'}<1);
+        foreach my $resource (@{$sequence->{'contents'}}) {
+            next if ($resource->{'type'} ne 'assessment');
+            foreach my $part (@{$resource->{'parts'}}) {
+                $cols_output=0;
+                $problem_num++;
+                my $data = &get_statistics($sequence,$resource,$part,
+                                           $problem_num);
+                #
+                if (!defined($part) || $part eq '') {
+                    $part = ' ';
                 }
-                $r->print('<table border="0"><tr><td bgcolor="#777777">'."\n");
-                $r->print('<table border="0" cellpadding="3">'."\n");
-                $r->print(&CreateProblemStatisticsTableHeading($headings, $r));
-            } else {
-                if($sortProblems eq 'Sort Within Sequence') {
-                    $r->print('"'.$cache->{$sequence.':title'}.'"');
+                foreach my $field (@Fields) {
+                    next if ($field->{'name'} eq 'problem_num');
+                    $excel_sheet->write($rows_output,$cols_output++,
+                                        $data->{$field->{'name'}});
                 }
-                $r->print('<br>');
+                $rows_output++;
             }
-            $currentSequence = $sequence;
         }
-
-        my $ref = '<a href="'.$cache->{$problem.':source'}.
-                  '" target="_blank">'.$cache->{$problem.':title'}.'</a>';
-        my $title = $cache->{$problem.':title'};
-        if($part != 0) {
-            $title .= ' Part '.$part;
-        }
-        my $source = $cache->{$problem.':source'};
-        my $tableData = join('&', $ref, $title, $source,
-                       $cache->{$_.':studentCount'},
-                       $cache->{$_.':totalTries'},
-                       $cache->{$_.':maxTries'},
-                       $cache->{$_.':mean'},
-                       $cache->{$_.':correct'},
-                       $cache->{$_.':correctByOverride'},
-                       $cache->{$_.':percentWrong'},
-                       $cache->{$_.':degreeOfDifficulty'},
-                       $cache->{$_.':standardDeviation'},
-                       $cache->{$_.':skewness'},
-                       $cache->{$_.':discriminationFactor1'},
-                       $cache->{$_.':discriminationFactor2'});
-
-        &TableRow($displayFormat,$tableData,$count,$r,$color);
-
-        $count++;
-    }
-    if($displayFormat ne 'Display CSV Format') {
-        $r->print('</table>'."\n");
-        $r->print('</td></tr></table>');
-    } else {
-        $r->print('<br>');
     }
-
+    #
+    # Write the excel file
+    $excel_workbook->close();
+    # Tell the user where to get their excel file
+    $r->print('<br />'.
+              '<a href="'.$filename.'">Your Excel spreadsheet.</a>'."\n");
+    $r->rflush();
     return;
 }
 
-sub TableRow {
-    my ($displayFormat,$Str,$RealIdx,$r,$color)=@_;
-    my($ref,$title,$source,$StdNo,$TotalTries,$MxTries,$Avg,$YES,$Override,
-       $Wrng,$DoD,$SD,$Sk,$_D1,$_D2)=split(/\&/,$Str);	
-    my $Ptr;
-    if($displayFormat eq 'Display CSV Format') {
-        $Ptr='"'.$RealIdx.'",'."\n".
-             '"'.$title.'",'."\n".
-             '"'.$source.'",'."\n".
-             '"'.$StdNo.'",'."\n".
-             '"'.$TotalTries.'",'."\n".
-             '"'.$MxTries.'",'."\n".
-             '"'.$Avg.'",'."\n".
-             '"'.$YES.'",'."\n".
-             '"'.$Override.'",'."\n".
-             '"'.$Wrng.'",'."\n".
-             '"'.$DoD.'",'."\n".
-             '"'.$SD.'",'."\n".
-             '"'.$Sk.'",'."\n".
-             '"'.$_D1.'",'."\n".
-             '"'.$_D2.'"'."\n".
-             "<br>\n";
+###############################################
+###############################################
 
-        $r->print("\n".$Ptr);
-    } else {
-        $Ptr='<tr>'."\n".
-             '<td bgcolor="#ffffe6">'.$RealIdx.'</td>'."\n".
-             '<td bgcolor="#ffffe6">'.$ref.'</td>'."\n".
-             '<td bgcolor='.$color->{"yellow"}.'> '.$StdNo.'</td>'."\n".
-             '<td bgcolor='.$color->{"yellow"}.'>'.$TotalTries.'</td>'."\n".
-             '<td bgcolor='.$color->{"yellow"}.'>'.$MxTries.'</td>'."\n".
-             '<td bgcolor='.$color->{"gb"}.'>'.$Avg.'</td>'."\n".
-             '<td bgcolor='.$color->{"gb"}.'> '.$YES.'</td>'."\n".
-             '<td bgcolor='.$color->{"gb"}.'> '.$Override.'</td>'."\n".
-             '<td bgcolor='.$color->{"red"}.'> '.$Wrng.'</td>'."\n".
-             '<td bgcolor='.$color->{"red"}.'> '.$DoD.'</td>'."\n".
-             '<td bgcolor='.$color->{"green"}.'> '.$SD.'</td>'."\n".
-             '<td bgcolor='.$color->{"green"}.'> '.$Sk.'</td>'."\n".
-             '<td bgcolor='.$color->{"purple"}.'> '.$_D1.'</td>'."\n".
-	     '<td bgcolor='.$color->{"purple"}.'> '.$_D2.'</td>'."\n";
-        $r->print($Ptr.'</tr>'."\n");
-    }
+=pod 
 
-    return;
-}
+=item &statistics_html_table_data()
 
-# For loading the colored table for display or un-colored for print
-sub setbgcolor {
-    my $PrintTable=shift;
-    my %color;
-    if ($PrintTable){
-	$color{"gb"}="#FFFFFF";
-	$color{"red"}="#FFFFFF";
-	$color{"yellow"}="#FFFFFF";
-	$color{"green"}="#FFFFFF";
-	$color{"purple"}="#FFFFFF";
-    } else {
-	$color{"gb"}="#DDFFFF";
-	$color{"red"}="#FFDDDD";
-	$color{"yellow"}="#EEFFCC";
-	$color{"green"}="#DDFFDD";
-	$color{"purple"}="#FFDDFF";
-    }
-
-    return \%color;
-}
-
-sub ProblemStatisticsButtons {
-    my ($displayFormat, $displayLegend, $sortProblems)=@_;
-
-    my $Ptr = '<tr><td></td><td align="left">';
-    $Ptr .= '<input type="submit" name="DoDiffGraph" ';
-    $Ptr .= 'value="Plot Degree of Difficulty" />'."\n";
-    $Ptr .= '</td><td align="left">';
-    $Ptr .= '<input type="submit" name="PercentWrongGraph" ';
-    $Ptr .= 'value="Plot Percent Wrong" />'."\n";
-    $Ptr .= '</td></tr><tr><td></td><td>'."\n";
-    $Ptr .= '<input type="submit" name="SortProblems" ';
-    if($sortProblems eq 'Sort All Problems') {
-        $Ptr .= 'value="Sort Within Sequence" />'."\n";
-    } else {
-        $Ptr .= 'value="Sort All Problems" />'."\n";
+Help function used to format the rows for HTML table output.
+
+=cut
+
+###############################################
+###############################################
+sub statistics_html_table_data {
+    my ($data,$options) = @_;
+    my $row = '';
+    foreach my $field (@Fields) {
+        next if ($options =~ /no $field->{'name'}/);
+        $row .= '<td bgcolor="'.$field->{'color'}.'"';
+        if (exists($field->{'align'})) {
+            $row .= ' align="'.$field->{'align'}.'"';
+            }
+        $row .= '>';
+        if (exists($field->{'special'}) && $field->{'special'} eq 'link') {
+            $row .= '<a href="'.$data->{$field->{'name'}.'.link'}.'">';
+        }
+        if (exists($field->{'format'})) {
+            $row .= sprintf($field->{'format'},$data->{$field->{'name'}});
+        } else {
+            $row .= $data->{$field->{'name'}};
+        }
+        if (exists($field->{'special'}) && $field->{'special'} eq 'link') {
+            $row.= '</a>';
+        }
+        $row .= '</td>';
     }
-    $Ptr .= '</td><td align="left">';
-    $Ptr .= '<input type="submit" name="DisplayLegend" ';
-    if($displayLegend eq 'Show Legend') {
-        $Ptr .= 'value="Hide Legend" />'."\n";
-    } else {
-        $Ptr .= 'value="Show Legend" />'."\n";
+    return $row;
+}
+
+sub statistics_table_header {
+    my ($options) = @_;
+    my $header_row;
+    foreach my $field (@Fields) {
+        next if ($options =~ /no $field->{'name'}/);
+        $header_row .= '<th>';
+        if ($options =~ /sortable/ && 
+            exists($field->{'sortable'}) && $field->{'sortable'} eq 'yes') {
+            $header_row .= '<a href="javascript:'.
+                'document.Statistics.sortby.value='."'".$field->{'name'}."'".
+                    ';document.Statistics.submit();">';
+        }
+        $header_row .= $field->{'title'};
+        if ($options =~ /sortable/) {
+            $header_row.= '</a>';
+        }
+        if ($options !~ /no plots/        && 
+            exists($field->{'graphable'}) && 
+            $field->{'graphable'} eq 'yes') {
+            $header_row.=' (';
+            $header_row .= '<a href="javascript:'.
+                "document.Statistics.plot.value='$field->{'name'}'".
+                    ';document.Statistics.submit();">';
+            $header_row .= 'plot</a>)';
+        }
+        $header_row .= '</th>';
+    }
+    return $header_row;
+}
+
+###############################################
+###############################################
+
+=pod 
+
+=item &plot_statistics()
+
+=cut
+
+###############################################
+###############################################
+sub plot_statistics {
+    my ($r,$datafield) = @_;
+    my @Data;
+    #
+    #
+    my $sortfield = undef;
+    my $title = undef;
+    foreach my $field (@Fields) {
+        if ($datafield eq $field->{'name'} &&
+            exists($field->{'graphable'}) && $field->{'graphable'} eq 'yes') {
+            $sortfield = $field->{'name'};
+            $title = $field->{'long_title'};
+        }
+    }
+    return if (! defined($sortfield) || $sortfield eq '');
+    #
+    my $Max = 0;
+    my $problem_num = 0;
+    foreach my $sequence (&Apache::lonstatistics::Sequences_with_Assess()) {
+        next if ($sequence->{'num_assess'}<1);
+        foreach my $resource (@{$sequence->{'contents'}}) {
+            next if ($resource->{'type'} ne 'assessment');
+            foreach my $part (@{$resource->{'parts'}}) {
+                my $problem_number++;
+                my $data = &get_statistics($sequence,$resource,$part,
+                                           $problem_num);
+                my $value = $data->{$sortfield};
+                $Max = $value if ($Max < $value);
+                push (@Data,$value);
+            }
+        }
     }
-    $Ptr .= '</td><td align="left">';
-    $Ptr .= '<input type="submit" name="DisplayCSVFormat" ';
-    if($displayFormat eq 'Display CSV Format') {
-        $Ptr .= 'value="Display Table Format" />'."\n";
-    } else {
-        $Ptr .= 'value="Display CSV Format" />'."\n";
+    #
+    # Print out plot request
+    my $yaxis = '';
+    if ($sortfield eq 'per_wrong') {
+        $yaxis = 'Percent';
+    }
+    #
+    # Determine appropriate value for $Max
+    if ($sortfield eq 'deg_of_diff') {
+        if ($Max > 0.5) {
+            $Max = 1;
+        } elsif ($Max > 0.2) {
+            $Max = 0.5;
+        } elsif ($Max > 0.1) {
+            $Max = 0.2;
+        }
+    } elsif ($sortfield eq 'per_wrong') {
+        if ($Max > 50) {
+            $Max = 100;
+        } elsif ($Max > 25) {
+            $Max = 50;
+        } elsif ($Max > 20) {
+            $Max = 25;
+        } elsif ($Max > 10) {
+            $Max = 20;
+        } elsif ($Max > 5) {
+            $Max = 10;
+        } else {
+            $Max = 5;
+        }
     }
-    $Ptr .= '</td></tr>';
-
-    return $Ptr;
+    
+    $r->print("<p>".&DrawGraph(\@Data,$title,'Problem Number',$yaxis,
+                               $Max)."</p>\n");
+    #
+    # Print out the data
+    $ENV{'form.sortby'} = 'Contents';
+#    &output_html_ungrouped($r);
+    return;
 }
 
+###############################################
+###############################################
+
+=pod 
+
+=item &DrawGraph()
+
+=cut
+
+###############################################
+###############################################
+sub DrawGraph {
+    my ($values,$title,$xaxis,$yaxis,$Max)=@_;
+    $title = '' if (! defined($title));
+    $xaxis = '' if (! defined($xaxis));
+    $yaxis = '' if (! defined($yaxis));
+    #
+    my $sendValues = join(',', @$values);
+    my $sendCount = scalar(@$values);
+    $Max =1 if ($Max < 1);
+    if ( int($Max) < $Max ) {
+        $Max++;
+        $Max = int($Max);
+    }
+    my @GData = ($title,$xaxis,$yaxis,$Max,$sendCount,$sendValues);
+    return '<IMG src="/cgi-bin/graph.png?'.
+        (join('&', @GData)).'" border="1" />';
+}
+
+sub get_statistics {
+    my ($sequence,$resource,$part,$problem_num) = @_;
+    #
+    my $symb = $resource->{'symb'};
+    my $courseid = $ENV{'request.course.id'};
+    #
+    my $students = \@Apache::lonstatistics::Students;
+    if ($Apache::lonstatistics::SelectedSections[0] eq 'all') {
+        $students = undef;
+    }
+    my $data = &Apache::loncoursedata::get_problem_statistics
+                        ($students,$symb,$part,$courseid);
+    $data->{'part'}        = $part;
+    $data->{'problem_num'} = $problem_num;
+    $data->{'container'}   = $sequence->{'title'};
+    $data->{'title'}       = $resource->{'title'};
+    $data->{'title.link'}  = $resource->{'src'}.'?symb='.
+        &Apache::lonnet::escape($resource->{'symb'});
+    #
+    return $data;
+}
+
+###############################################
+###############################################
+
+=pod 
+
+=item &ProblemStatisticsLegend()
+
+=cut
+
+###############################################
+###############################################
 sub ProblemStatisticsLegend {
     my $Ptr = '';
     $Ptr = '<table border="0">';
@@ -507,7 +819,7 @@ sub ProblemStatisticsLegend {
     $Ptr .= '<b>Tries</b></td>';
     $Ptr .= '<td>Total number of tries for solving the problem.';
     $Ptr .= '</td></tr><tr><td>';
-    $Ptr .= '<b>Mod</b></td>';
+    $Ptr .= '<b>Max Tries</b></td>';
     $Ptr .= '<td>Largest number of tries for solving the problem by a student.';
     $Ptr .= '</td></tr><tr><td>';
     $Ptr .= '<b>Mean</b></td>';
@@ -549,307 +861,9 @@ sub ProblemStatisticsLegend {
     $Ptr .= '<tr><td><b>Disc.</b></td>';
     $Ptr .= '<td>Number of Students had at least one discussion.';
     $Ptr .= '</td></tr></table>';
-
     return $Ptr;
 }
 
-sub ExtractStudentData {
-    my ($cache, $students)=@_;
-
-    my @problemList=();
-    my %problemData;
-    foreach my $sequence (split(':', $cache->{'orderedSequences'})) {
-        foreach my $problemID (split(':', $cache->{$sequence.':problems'})) {
-            foreach my $part (split(/\:/,$cache->{$sequence.':'.
-                                                  $problemID.
-                                                  ':parts'})) {
-                my $id = $sequence.':'.$problemID.':'.$part;
-                push(@problemList, $id);
-                my $totalTries = 0;
-                my $totalAwarded = 0;
-                my $correct = 0;
-                my $correctByOverride = 0;
-                my $studentCount = 0;
-                my $maxTries = 0;
-                my $totalFirst = 0;
-                my @studentTries=();
-                foreach(@$students) {
-                    my $code = $cache->{"$_:$problemID:$part:code"};
-
-                    if(defined($cache->{$_.':error'}) || $code eq ' ' ||
-                       $cache->{"$_:$problemID:NoVersion"} eq 'true') {
-                        next;
-                    }
-
-                    $studentCount++;
-                    my $tries =  $cache->{"$_:$problemID:$part:tries"};
-                    if($maxTries < $tries) {
-                        $maxTries = $tries;
-                    }
-                    $totalTries += $tries;
-                    push(@studentTries, $tries);
-
-                    my $awarded = $cache->{"$_:$problemID:$part:awarded"};
-                    $totalAwarded += $awarded;
-
-                    if($code eq '*') {
-                        $correct++;
-                        if($tries == 1) {
-                            $totalFirst++;
-                        }
-                    } elsif($code eq '+') {
-                        $correctByOverride++;
-                    }
-                }
-
-                my $studentTriesJoined = join(':::', @studentTries);
-                $problemData{$id.':sequenceTitle'} =
-                    $cache->{$sequence.':title'};
-                $problemData{$id.':studentCount'} = $studentCount;
-                $problemData{$id.':totalTries'} = $totalTries;
-                $problemData{$id.':studentTries'} = $studentTriesJoined;
-                $problemData{$id.':totalAwarded'} = $totalAwarded;
-                $problemData{$id.':correct'} = $correct;
-                $problemData{$id.':correctByOverride'} = $correctByOverride;
-                $problemData{$id.':wrong'} = $studentCount - 
-                                             ($correct + $correctByOverride);
-                $problemData{$id.':maxTries'} = $maxTries;
-                $problemData{$id.':totalFirst'} = $totalFirst;
-            }
-        }
-    }
-
-    my @upperStudents1=();
-    my @lowerStudents1=();
-    my @upperStudents2=();
-    my @lowerStudents2=();
-    my $upperCount = int(0.27*scalar(@$students));
-    # Discriminant Factor criterion 1
-    my $sortedStudents = &SortDivideByTries($students,$cache,':totalAwarded');
-
-    for(my $i=0; $i<$upperCount; $i++) {
-        push(@lowerStudents1, $sortedStudents->[$i]);
-        push(@upperStudents1, $sortedStudents->[(scalar(@$students)-$i-1)]);
-    }
-
-    $problemData{'studentsUpperListCriterion1'}=join(':::', @upperStudents1);
-    $problemData{'studentsLowerListCriterion1'}=join(':::', @lowerStudents1);
-
-    # Discriminant Factor criterion 2
-    $sortedStudents = &SortDivideByTries($students, $cache, ':totalSolved');
-
-    for(my $i=0; $i<$upperCount; $i++) {
-        push(@lowerStudents2, $sortedStudents->[$i]);
-        push(@upperStudents2, $sortedStudents->[(scalar(@$students)-$i-1)]);
-    }
-    $problemData{'studentsUpperListCriterion2'}=join(':::', @upperStudents2);
-    $problemData{'studentsLowerListCriterion2'}=join(':::', @lowerStudents2);
-
-    $problemData{'problemList'} = join(':::', @problemList);
-
-    return \%problemData;
-}
-
-sub SortDivideByTries {
-    my ($toSort, $data, $sortOn)=@_;
-    my @orderedData = sort { ($data->{$a.':totalTries'}) ? 
-                             ($data->{$a.$sortOn}/$data->{$a.':totalTries'}):0
-                             <=>
-                             ($data->{$b.':totalTries'}) ?
-                             ($data->{$b.$sortOn}/$data->{$b.':totalTries'}):0
-                           } @$toSort;
-
-    return \@orderedData;
-}
-
-sub SortProblems {
-    my ($problemData,$sortBy,$sortProblems,$ascend)=@_;
-
-    my @problems = split(':::', $problemData->{'problemList'});
-    if($sortBy eq "Homework Sets Order") {
-        return \@problems;
-    }
-
-    my $data;
-
-    if   ($sortBy eq "#Stdnts") { $data = ':studentCount'; }
-    elsif($sortBy eq "Tries")   { $data = ':totalTries'; }
-    elsif($sortBy eq "Mod")     { $data = ':maxTries'; }
-    elsif($sortBy eq "Mean")    { $data = ':mean'; }
-    elsif($sortBy eq "#YES")    { $data = ':correct'; }
-    elsif($sortBy eq "#yes")    { $data = ':correctByOverride'; }
-    elsif($sortBy eq "%Wrng")   { $data = ':percentWrong'; }
-    elsif($sortBy eq "DoDiff")  { $data = ':degreeOfDifficulty'; }
-    elsif($sortBy eq "S.D.")    { $data = ':standardDeviation'; }
-    elsif($sortBy eq "Skew.")   { $data = ':skewness'; }
-    elsif($sortBy eq "D.F.1st") { $data = ':discriminationFactor1'; }
-    elsif($sortBy eq "D.F.2nd") { $data = ':discriminationFactor2'; }
-    else                        { return \@problems; }
-
-    my %temp;
-    my @sequenceList=();
-    foreach(@problems) {
-        my ($sequence) = split(':', $_);
-
-        my @array=();
-        my $tempArray;
-        if(defined($temp{$sequence})) {
-            $tempArray = $temp{$sequence};
-        } else {
-            push(@sequenceList, $sequence);
-            $tempArray = \@array;
-            $temp{$sequence} = $tempArray;
-        }
-
-        push(@$tempArray, $_);
-    }
-
-    my @orderedProblems;
-    if($sortProblems eq "Sort Within Sequence") {
-        foreach(keys(%temp)) {
-            my $tempArray = $temp{$_};
-            my @tempOrder =
-                sort { $problemData->{$a.$data} <=> $problemData->{$b.$data} }
-            @$tempArray;
-            $temp{$_} = \@tempOrder;
-        }
-        foreach(@sequenceList) {
-            my $tempArray = $temp{$_};
-            @orderedProblems = (@orderedProblems, @$tempArray);
-        }
-    } else {
-        @orderedProblems = 
-            sort { $problemData->{$a.$data} <=> $problemData->{$b.$data} }
-        @problems;
-    }
-
-    if($ascend eq 'Descending') {
-        @orderedProblems = reverse(@orderedProblems);
-    }
-
-    return \@orderedProblems;
-}
-
-sub CalculateStatistics {
-    my ($data, $cache, $courseID)=@_;
-
-    my @problems = split(':::', $data->{'problemList'});
-    foreach(@problems) {
-        # Mean
-        my $mean = ($data->{$_.':studentCount'}) ? 
-            ($data->{$_.':totalTries'} / $data->{$_.':studentCount'}) : 0;
-        $data->{$_.':mean'} = sprintf("%.2f", $mean);
-
-        # %Wrong
-        my $pw = ($data->{$_.':studentCount'}) ?
-            (($data->{$_.':wrong'} / $data->{$_.':studentCount'}) * 100.0) : 
-            100.0;
-        $data->{$_.':percentWrong'} = sprintf("%.1f", $pw);
-
-        # Degree of Difficulty
-        my $dod = ($data->{$_.':totalTries'}) ?
-            (1 - (($data->{$_.':correct'} + $data->{$_.':correctByOverride'}) /
-                  $data->{$_.':totalTries'})) : 0;
-
-        $data->{$_.':degreeOfDifficulty'} = sprintf("%.2f", $dod);
-
-        # Factor in mean
-        my @studentTries = split(':::', $data->{$_.':studentTries'});
-        foreach(my $index=0; $index < scalar(@studentTries); $index++) {
-            $studentTries[$index] -= $mean;
-        }
-        my $sumSquared = 0;
-        my $sumCubed = 0;
-        foreach(@studentTries) {
-            my $squared = ($_ * $_);
-            my $cubed = ($squared * $_);
-            $sumSquared += $squared;
-            $sumCubed += $cubed;
-        }
-
-        # Standard deviation
-        my $standardDeviation;
-        if($data->{$_.':studentCount'} - 1 > 0) {
-            $standardDeviation = (sqrt($sumSquared)) / 
-                                 ($data->{$_.':studentCount'} - 1);
-        } else {
-            $standardDeviation =  0.0;
-        }
-        $data->{$_.':standardDeviation'} = sprintf("%.1f", $standardDeviation);
-
-        # Skewness
-        my $skew;
-        if($standardDeviation > 0.0999 && $data->{$_.':studentCount'} > 0) {
-            $skew = (((sqrt($sumSquared)) / $data->{$_.':studentCount'}) / 
-                     ($standardDeviation * 
-                      $standardDeviation * 
-                      $standardDeviation));
-        } else {
-            $skew = 0.0;
-        }
-
-        $data->{$_.':skewness'} = sprintf("%.1f", $skew);
-
-        # Discrimination Factor 1
-        my ($sequence, $problem, $part) = split(':', $_);
-
-        my @upper1 = split(':::', $data->{'studentsUpperListCriterion1'});
-        my @lower1 = split(':::', $data->{'studentsLowerListCriterion1'});
-
-        my $upper1Sum=0;
-        foreach my $name (@upper1) {
-            $upper1Sum += $cache->{"$name:$problem:$part:awarded"};
-        }
-        $upper1Sum = (scalar(@upper1)) ? ($upper1Sum/(scalar(@upper1))) : 0;
-
-        my $lower1Sum=0;
-        foreach my $name (@lower1) {
-            $lower1Sum += $cache->{"$name:$problem:$part:awarded"};
-        }
-        $lower1Sum = (scalar(@lower1)) ? ($lower1Sum/(scalar(@lower1))) : 0;
-
-        my $df1 = $upper1Sum - $lower1Sum;
-        $data->{$_.':discriminationFactor1'} = sprintf("%.2f", $df1);
-
-        # Discrimination Factor 2
-        my @upper2 = split(':::', $data->{'studentsUpperListCriterion2'});
-        my @lower2 = split(':::', $data->{'studentsLowerListCriterion2'});
-
-        my $upper2Sum=0;
-        foreach my $name (@upper2) {
-            $upper2Sum += $cache->{"$name:$problem:$part:awarded"};
-        }
-        $upper2Sum = (scalar(@upper2)) ? ($upper2Sum/(scalar(@upper2))) : 0;
-
-        my $lower2Sum=0;
-        foreach my $name (@lower2) {
-            $lower2Sum += $cache->{"$name:$problem:$part:awarded"};
-        }
-        $lower2Sum = (scalar(@lower2)) ? ($lower2Sum/(scalar(@lower2))) : 0;
-
-        my $df2 = $upper2Sum - $lower2Sum;
-        $data->{$_.':discriminationFactor2'} = sprintf("%.2f", $df2);
-
-        my %storestats;
-        my $Average = ($data->{$_.':studentCount'}) ? 
-            $data->{$_.':totalTries'}/$data->{$_.':studentCount'} : 0;
-        $storestats{$courseID.'___'.$cache->{$sequence.':source'}.
-                        '___timestamp'}=time;
-        $storestats{$courseID.'___'.$cache->{$sequence.':source'}.
-                        '___stdno'}=$data->{$_.':studentCount'};
-        $storestats{$courseID.'___'.$cache->{$sequence.':source'}.
-                        '___avetries'}=$Average;
-        $storestats{$courseID.'___'.$cache->{$sequence.':source'}.
-                        '___difficulty'}=$data->{$_.':degreeOfDifficulty'};
-        $cache->{$sequence.':source'} =~ /^(\w+)\/(\w+)/;
-        if($data->{$_.':studentCount'}) { 
-            &Apache::lonnet::put('nohist_resevaldata',\%storestats,$1,$2);
-        }
-    }
-
-    return;
-}
-
 #---- END Problem Statistics Web Page ----------------------------------------
 
 1;