--- loncom/interface/statistics/lonproblemstatistics.pm	2004/03/26 22:04:22	1.73
+++ loncom/interface/statistics/lonproblemstatistics.pm	2005/01/11 19:45:18	1.96
@@ -1,6 +1,6 @@
 # The LearningOnline Network with CAPA
 #
-# $Id: lonproblemstatistics.pm,v 1.73 2004/03/26 22:04:22 matthew Exp $
+# $Id: lonproblemstatistics.pm,v 1.96 2005/01/11 19:45:18 matthew Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -55,12 +55,14 @@ use Apache::loncommon();
 use Apache::lonhtmlcommon;
 use Apache::loncoursedata;
 use Apache::lonstatistics;
+use LONCAPA::lonmetadata();
 use Apache::lonlocal;
 use Spreadsheet::WriteExcel;
 use Apache::lonstathelpers();
 use Time::HiRes;
 
 my @StatsArray;
+my %SeqStat;    # keys are symbs, values are hash refs
 
 ##
 ## Localization notes:
@@ -69,27 +71,67 @@ my @StatsArray;
 ## header for plots created with Graph.pm, both of which more than likely do
 ## not support localization.
 ##
+#
+#
+##
+## Description of Field attributes
+##
+## Attribute     Required   Value       Meaning or Use
+##
+## name            yes      any scalar  Used to uniquely identify field
+## title           yes      any scalar  This is what the user sees to identify
+##                                      the field.  Passed through &mt().
+## long_title      yes      any scalar  Used as graph heading and in excel
+##                                      output.  NOT translated
+## align           no    (left|right|center)  HTML cell contents alignment
+## color           yes      html color  HTML cell background color
+##                                      used to visually group statistics
+## special         no          (link)   Indicates a link, target is name.link
+##                                      Currently set in &get_statistics()
+## graphable       no      (yes|no)     Can a bar graph of the field be 
+##                                      produced?
+## sortable        no      (yes|no)     Should a sort link be put in the
+##                                      column header?
+## selectable      yes     (yes|no)     Can the column be removed from the
+##                                      statistics display?
+## selected        yes     (yes|no)     Is the column selected by default?
+##
+## format          no      sprintf format string
+##
+## excel_format    no      excel format type 
+##                               (see &Apache::loncommon::define_excel_formats
 my @Fields = (
            { name => 'problem_num',
              title => 'P#',
              align => 'right',
-             color => '#FFFFE6' },
+             color => '#FFFFE6',
+             selectable => 'no',
+             defaultselected => 'yes',
+           },
            { name   => 'container',
              title  => 'Sequence or Folder',
              align  => 'left',
              color  => '#FFFFE6',
-             sortable => 'yes' },
+             sortable => 'yes',
+             selectable => 'no',
+             defaultselected => 'yes',
+           },
            { name   => 'title',
              title  => 'Title',
              align  => 'left',
              color  => '#FFFFE6',
              special  => 'link',
-             sortable => 'yes', },
+             sortable => 'yes', 
+             selectable => 'no',
+             defaultselected => 'yes',
+           },
            { name   => 'part', 
              title  => 'Part',
              align  => 'left',
              color  => '#FFFFE6',
-             },
+             selectable => 'no',
+             defaultselected => 'yes',
+           },
            { name   => 'num_students',
              title  => '#Stdnts',
              align  => 'right',
@@ -97,7 +139,10 @@ my @Fields = (
              format => '%d',
              sortable  => 'yes',
              graphable => 'yes',
-             long_title => 'Number of Students Attempting Problem' },
+             long_title => 'Number of Students Attempting Problem',
+             selectable => 'yes',
+             defaultselected => 'yes',
+           },
            { name   => 'tries',
              title  => 'Tries',
              align  => 'right',
@@ -105,7 +150,10 @@ my @Fields = (
              format => '%d',
              sortable  => 'yes',
              graphable => 'yes',
-             long_title => 'Total Number of Tries' },
+             long_title => 'Total Number of Tries',
+             selectable => 'yes',
+             defaultselected => 'yes',
+           },
            { name   => 'max_tries',
              title  => 'Max Tries',
              align  => 'right',
@@ -113,7 +161,10 @@ my @Fields = (
              format => '%d',
              sortable  => 'yes',
              graphable => 'yes',
-             long_title => 'Maximum Number of Tries' },
+             long_title => 'Maximum Number of Tries',
+             selectable => 'yes',
+             defaultselected => 'yes',
+           },
            { name   => 'min_tries',
              title  => 'Min Tries',
              align  => 'right',
@@ -121,7 +172,10 @@ my @Fields = (
              format => '%d',
              sortable  => 'yes',
              graphable => 'yes',
-             long_title => 'Minumum Number of Tries' },
+             long_title => 'Minumum Number of Tries',
+             selectable => 'yes',
+             defaultselected => 'yes',
+           },
            { name   => 'mean_tries',
              title  => 'Mean Tries',
              align  => 'right',
@@ -129,7 +183,10 @@ my @Fields = (
              format => '%5.2f',
              sortable  => 'yes',
              graphable => 'yes',
-             long_title => 'Average Number of Tries' },
+             long_title => 'Average Number of Tries',
+             selectable => 'yes',
+             defaultselected => 'yes',
+           },
            { name   => 'std_tries',
              title  => 'S.D. tries',
              align  => 'right',
@@ -137,7 +194,10 @@ my @Fields = (
              format => '%5.2f',
              sortable  => 'yes',
              graphable => 'yes',
-             long_title => 'Standard Deviation of Number of Tries' },
+             long_title => 'Standard Deviation of Number of Tries',
+             selectable => 'yes',
+             defaultselected => 'yes',
+           },
            { name   => 'skew_tries',
              title  => 'Skew Tries',
              align  => 'right',
@@ -145,7 +205,10 @@ my @Fields = (
              format => '%5.2f',
              sortable  => 'yes',
              graphable => 'yes',
-             long_title => 'Skew of Number of Tries' },
+             long_title => 'Skew of Number of Tries',
+             selectable => 'yes',
+             defaultselected => 'no',
+           },
            { name   => 'num_solved',
              title  => '#YES',
              align  => 'right',
@@ -153,7 +216,10 @@ my @Fields = (
              format => '%4.1f',#             format => '%d',
              sortable  => 'yes',
              graphable => 'yes',
-             long_title => 'Number of Students able to Solve' },
+             long_title => 'Number of Students able to Solve',
+             selectable => 'yes',
+             defaultselected => 'yes',
+           },
            { name   => 'num_override',
              title  => '#yes',
              align  => 'right',
@@ -161,7 +227,21 @@ my @Fields = (
              format => '%4.1f',#             format => '%d',
              sortable  => 'yes',
              graphable => 'yes',
-             long_title => 'Number of Students given Override' },
+             long_title => 'Number of Students given Override',
+             selectable => 'yes',
+             defaultselected => 'yes',
+           },
+           { name   => 'tries_per_correct',
+             title  => 'tries/correct',
+             align  => 'right',
+             color  => '#FFDDDD',
+             format => '%4.1f',
+             sortable  => 'yes',
+             graphable => 'yes',
+             long_title => 'Tries per Correct Answer',
+             selectable => 'yes',
+             defaultselected => 'yes',
+           },
            { name   => 'num_wrong',
              title  => '#Wrng',
              align  => 'right',
@@ -169,7 +249,21 @@ my @Fields = (
              format => '%4.1f',
              sortable  => 'yes',
              graphable => 'yes',
-             long_title => 'Percent of students whose final answer is wrong' },
+             long_title => 'Number of students whose final answer is wrong',
+             selectable => 'yes',
+             defaultselected => 'yes',
+           },
+           { name   => 'per_wrong',
+             title  => '%Wrng',
+             align  => 'right',
+             color  => '#FFDDDD',
+             format => '%4.1f',
+             sortable  => 'yes',
+             graphable => 'yes',
+             long_title => 'Percent of students whose final answer is wrong',
+             selectable => 'yes',
+             defaultselected => 'yes',
+           },
            { name   => 'deg_of_diff',
              title  => 'DoDiff',
              align  => 'right',
@@ -178,7 +272,10 @@ my @Fields = (
              sortable  => 'yes',
              graphable => 'yes',
              long_title => 'Degree of Difficulty'.
-                           '[ 1 - ((#YES+#yes) / Tries) ]'},
+                           '[ 1 - ((#YES+#yes) / Tries) ]',
+             selectable => 'yes',
+             defaultselected => 'yes',
+           },
            { name   => 'deg_of_disc',
              title  => 'DoDisc',
              align  => 'right',
@@ -186,9 +283,310 @@ my @Fields = (
              format => '%4.2f',
              sortable  => 'yes',
              graphable => 'yes',
-             long_title => 'Degree of Discrimination' },
+             long_title => 'Degree of Discrimination',
+             selectable => 'yes',
+             defaultselected => 'yes',
+           },
+##   duedate included for research purposes.  Commented out most of the time.
+#           { name => 'duedate',
+#             title => 'Due Date',
+#             align => 'left',
+#             color => '#FFFFFF',
+#             sortable => 'yes',
+#             graphable => 'no',
+#             long_title => 'Due date of resource for instructor',
+#             selectable => 'no',
+#             defaultselected => 'yes',
+#            },
+##   opendate included for research purposes.  Commented out most of the time.
+#           { name => 'opendate',
+#             title => 'Open Date',
+#             align => 'left',
+#             color => '#FFFFFF',
+#             sortable => 'yes',
+#             graphable => 'no',
+#             long_title => 'date resource became answerable',
+#             selectable => 'no',
+#             defaultselected => 'yes',
+#            },
+##   symb included for research purposes.  Commented out most of the time.
+#           { name => 'symb',
+#             title => 'Symb',
+#             align => 'left',
+#             color => '#FFFFFF',
+#             sortable => 'yes',
+#             graphable => 'no',
+#             long_title => 'Unique LON-CAPA identifier for problem',
+#             selectable => 'no',
+#             defaultselected => 'yes',
+#            },
+##   resptypes included for research purposes.  Commented out most of the time.
+#           { name => 'resptypes',
+#             title => 'Response Types',
+#             align => 'left',
+#             color => '#FFFFFF',
+#             sortable => 'no',
+#             graphable => 'no',
+#             long_title => 'Response Types used in this problem',
+#             selectable => 'no',
+#             defaultselected => 'yes',
+#            },
+##   maxtries included for research purposes.  Commented out most of the time.
+#           { name => 'maxtries',
+#             title => 'Maxtries',
+#             align => 'left',
+#             color => '#FFFFFF',
+#             sortable => 'no',
+#             graphable => 'no',
+#             long_title => 'Maximum number of tries',
+#             selectable => 'no',
+#             defaultselected => 'yes',
+#            },
+##   hinttries included for research purposes.  Commented out most of the time.
+#           { name => 'hinttries',
+#             title => 'hinttries',
+#             align => 'left',
+#             color => '#FFFFFF',
+#             sortable => 'no',
+#             graphable => 'no',
+#             long_title => 'Number of tries before a hint appears',
+#             selectable => 'no',
+#             defaultselected => 'yes',
+#            },
+##   weight included for research purposes.  Commented out most of the time.
+#           { name => 'weight',
+#             title => 'weight',
+#             align => 'right',
+#             color => '#FFFFFF',
+#             sortable => 'no',
+#             graphable => 'no',
+#             long_title => 'Problem weight (for instructor)',
+#             selectable => 'no',
+#             defaultselected => 'yes',
+#            },
 );
 
+my @SeqFields = (
+           { name   => 'title',
+             title  => 'Sequence',
+             align  => 'left',
+             color  => '#FFFFE6',
+             special  => 'no',
+             sortable => 'no', 
+             selectable => 'yes',
+             defaultselected => 'no',
+           },
+           { name   => 'items',
+             title  => '#Items',
+             align  => 'right',
+             color  => '#FFFFE6',
+             format => '%4d',
+             sortable  => 'no',
+             graphable => 'no',
+             long_title => 'Number of Items in Sequence',
+             selectable => 'yes',
+             defaultselected => 'no',
+           },
+           { name   => 'scoremean',
+             title  => 'Score Mean',
+             align  => 'right',
+             color  => '#FFFFE6',
+             format => '%4.2f',
+             sortable  => 'no',
+             graphable => 'no',
+             long_title => 'Mean Sequence Score',
+             selectable => 'yes',
+             defaultselected => 'no',
+           },
+           { name   => 'scorestd',
+             title  => 'Score STD',
+             align  => 'right',
+             color  => '#FFFFE6',
+             format => '%4.2f',
+             sortable  => 'no',
+             graphable => 'no',
+             long_title => 'Standard Deviation of Sequence Scores',
+             selectable => 'yes',
+             defaultselected => 'no',
+           },
+           { name   => 'scoremax',
+             title  => 'Score Max',
+             align  => 'right',
+             color  => '#FFFFE6',
+             format => '%4.2f',
+             sortable  => 'no',
+             graphable => 'no',
+             long_title => 'Maximum Sequence Score',
+             selectable => 'yes',
+             defaultselected => 'no',
+           },
+           { name   => 'scoremin',
+             title  => 'Score Min',
+             align  => 'right',
+             color  => '#FFFFE6',
+             format => '%4.2f',
+             sortable  => 'no',
+             graphable => 'no',
+             long_title => 'Minumum Sequence Score',
+             selectable => 'yes',
+             defaultselected => 'no',
+           },
+           { name   => 'scorecount',
+             title  => 'Score N',
+             align  => 'right',
+             color  => '#FFFFE6',
+             format => '%4d',
+             sortable  => 'no',
+             graphable => 'no',
+             long_title => 'Number of Students in score computations',
+             selectable => 'yes',
+             defaultselected => 'no',
+           },
+           { name   => 'countmean',
+             title  => 'Count Mean',
+             align  => 'right',
+             color  => '#FFFFFF',
+             format => '%4.2f',
+             sortable  => 'no',
+             graphable => 'no',
+             long_title => 'Mean Sequence Score',
+             selectable => 'yes',
+             defaultselected => 'no',
+           },
+           { name   => 'countstd',
+             title  => 'Count STD',
+             align  => 'right',
+             color  => '#FFFFFF',
+             format => '%4.2f',
+             sortable  => 'no',
+             graphable => 'no',
+             long_title => 'Standard Deviation of Sequence Scores',
+             selectable => 'yes',
+             defaultselected => 'no',
+           },
+           { name   => 'countmax',
+             title  => 'Count Max',
+             align  => 'right',
+             color  => '#FFFFFF',
+             format => '%4.2f',
+             sortable  => 'no',
+             graphable => 'no',
+             long_title => 'Maximum Number of Correct Problems',
+             selectable => 'yes',
+             defaultselected => 'no',
+           },
+           { name   => 'countmin',
+             title  => 'Count Min',
+             align  => 'right',
+             color  => '#FFFFFF',
+             format => '%4.2f',
+             sortable  => 'no',
+             graphable => 'no',
+             long_title => 'Minumum Number of Correct Problems',
+             selectable => 'yes',
+             defaultselected => 'no',
+           },
+           { name   => 'count',
+             title  => 'Count N',
+             align  => 'right',
+             color  => '#FFFFFF',
+             format => '%4d',
+             sortable  => 'no',
+             graphable => 'no',
+             long_title => 'Number of Students in score computations',
+             selectable => 'yes',
+             defaultselected => 'no',
+           },
+           { name   => 'KR-21',
+             title  => 'KR-21',
+             align  => 'right',
+             color  => '#FFAAAA',
+             format => '%4.2f',
+             sortable  => 'no',
+             graphable => 'no',
+             long_title => 'KR-21 reliability statistic',
+             selectable => 'yes',
+             defaultselected => 'no',
+           },           
+);
+
+my %SelectedFields;
+
+sub parse_field_selection {
+    #
+    # Pull out the defaults
+    if (! defined($ENV{'form.fieldselections'})) {
+        $ENV{'form.fieldselections'} = [];
+        foreach my $field (@Fields) {
+            next if ($field->{'selectable'} ne 'yes');
+            if ($field->{'defaultselected'} eq 'yes') {
+                push(@{$ENV{'form.fieldselections'}},$field->{'name'});
+            }
+        }
+    }
+    #
+    # Make sure the data we are plotting is there
+    my %NeededFields;
+    if (exists($ENV{'form.plot'}) && $ENV{'form.plot'} ne '' &&
+        $ENV{'form.plot'} ne 'none') {
+        if ($ENV{'form.plot'} eq 'degrees') {
+            $NeededFields{'deg_of_diff'}++;
+            $NeededFields{'deg_of_disc'}++;
+        } elsif ($ENV{'form.plot'} eq 'tries statistics') {
+            $NeededFields{'mean_tries'}++;
+            $NeededFields{'std_tries'}++;
+            $NeededFields{'problem_num'}++;
+        } else {
+            $NeededFields{$ENV{'form.plot'}}++;
+        }
+    }
+    #
+    # This should not happen, but in case it does...
+    if (ref($ENV{'form.fieldselections'}) ne 'ARRAY') {
+        $ENV{'form.fieldselections'} = [$ENV{'form.fieldselections'}];
+    }
+    #
+    # Set the field data and the selected fields (for easier checking)
+    undef(%SelectedFields);
+    foreach my $field (@Fields) {
+        if ($field->{'selectable'} ne 'yes') {
+            $field->{'selected'} = 'yes';
+        } else {
+            $field->{'selected'} = 'no';
+        }
+        if (exists($NeededFields{$field->{'name'}})) {
+            $field->{'selected'} = 'yes';
+            $SelectedFields{$field->{'name'}}++;
+        }
+        foreach my $selection (@{$ENV{'form.fieldselections'}}) {
+            if ($selection eq $field->{'name'} || $selection eq 'all') {
+                $field->{'selected'} = 'yes';
+                $SelectedFields{$field->{'name'}}++;
+            }
+        }
+    }
+    #
+    # Always show all the sequence statistics (for now)
+    foreach my $field (@SeqFields) {
+        $field->{'selected'} = 'yes';
+    }
+    return;
+}
+
+sub field_selection_input {
+    my $Str = '<select name="fieldselections" multiple size="5">'."\n";
+    $Str .= '<option value="all">all</option>'."\n";
+    foreach my $field (@Fields) {
+        next if ($field->{'selectable'} ne 'yes');
+        $Str .= '    <option value="'.$field->{'name'}.'" ';
+        if ($field->{'selected'} eq 'yes') {
+            $Str .= 'selected ';
+        }
+        $Str .= '>'.$field->{'title'}.'</option>'."\n";
+    }
+    $Str .= "</select>\n";
+}
+
 ###############################################
 ###############################################
 
@@ -204,6 +602,10 @@ select sections, maps, and output.
 ###############################################
 ###############################################
 sub CreateInterface {
+    my ($r) = @_;
+    #
+    &parse_field_selection();
+    #
     my $Str = '';
     $Str .= &Apache::lonhtmlcommon::breadcrumbs
         (undef,'Overall Problem Statistics','Statistics_Overall_Key');
@@ -212,6 +614,7 @@ sub CreateInterface {
     $Str .= '<td align="center"><b>'.&mt('Sections').'</b></td>';
     $Str .= '<td align="center"><b>'.&mt('Enrollment Status').'</b></td>';
     $Str .= '<td align="center"><b>'.&mt('Sequences and Folders').'</b></td>';
+    $Str .= '<td align="center"><b>'.&mt('Statistics').'</b></td>';
     $Str .= '<td rowspan="2">'.
         &Apache::lonstathelpers::limit_by_time_form().'</td>';
     $Str .= '</tr>'."\n";
@@ -232,21 +635,21 @@ sub CreateInterface {
     };
     $Str .= &Apache::lonstatistics::MapSelect('Maps','multiple,all',5,
                                               $only_seq_with_assessments);
+    $Str .= '</td><td>'.&field_selection_input();
     $Str .= '</td></tr>'."\n";
     $Str .= '</table>'."\n";
+    #
+    $Str .= '<p>'.&mt('Status: [_1]',
+                         '<input type="text" '.
+                         'name="stats_status" size="60" value="" />'
+                         ).
+                         '</nobr></p>';
+    #
     $Str .= '<input type="submit" name="GenerateStatistics" value="'.
         &mt('Generate Statistics').'" />';
     $Str .= '&nbsp;'x5;
     $Str .= 'Plot '.&plot_dropdown().('&nbsp;'x10);
-    $Str .= '<input type="submit" name="ClearCache" value="'.
-        &mt('Clear Caches').'" />';
-    $Str .= '&nbsp;'x5;
-    $Str .= '<input type="submit" name="UpdateCache" value="'.
-        &mt('Update Student Data').'" />';
-    $Str .= '&nbsp;'x5;
-    $Str .= '<input type="submit" name="Excel" value="'.
-        &mt('Produce Excel Output').'" />';
-    $Str .= '&nbsp;'x5;
+    #
     return $Str;
 }
 
@@ -270,7 +673,8 @@ sub BuildProblemStatisticsPage {
                                'statsoutputmode' => 'scalar',
                                'Section' => 'array',
                                'StudentData' => 'array',
-                               'Maps' => 'array');
+                               'Maps' => 'array',
+                               'fieldselections'=> 'array');
     &Apache::loncommon::store_course_settings('statistics',
                                               \%Saveable_Parameters);
     &Apache::loncommon::restore_course_settings('statistics',
@@ -280,15 +684,23 @@ sub BuildProblemStatisticsPage {
     #
     # Clear the package variables
     undef(@StatsArray);
+    undef(%SeqStat);
     #
     # Finally let the user know we are here
-    my $interface = &CreateInterface();
+    my $interface = &CreateInterface($r);
     $r->print($interface);
     $r->print('<input type="hidden" name="sortby" value="'.$ENV{'form.sortby'}.
               '" />');
     #
-    if (! exists($ENV{'form.statsfirstcall'})) {
-        $r->print('<input type="hidden" name="statsfirstcall" value="yes" />');
+    my @CacheButtonHTML = 
+        &Apache::lonstathelpers::manage_caches($r,'Statistics','stats_status');
+    my $Str;
+    foreach my $html (@CacheButtonHTML) {
+        $Str.=$html.('&nbsp;'x5);
+    }
+    #
+    $r->print($Str);
+    if (! exists($ENV{'form.firstrun'})) {
         $r->print('<h3>'.
                   &mt('Press "Generate Statistics" when you are ready.').
                   '</h3><p>'.
@@ -297,39 +709,76 @@ sub BuildProblemStatisticsPage {
                       ' will not have this delay.').
                   '</p>');
         return;
-    } elsif ($ENV{'form.statsfirstcall'} eq 'yes' || 
-             exists($ENV{'form.UpdateCache'}) ||
-             exists($ENV{'form.ClearCache'}) ) {
-        $r->print('<input type="hidden" name="statsfirstcall" value="no" />');
-        &Apache::lonstatistics::Gather_Student_Data($r);
-    } else {
-        $r->print('<input type="hidden" name="statsfirstcall" value="no" />');
     }
     $r->rflush();
     #
     # This probably does not need to be done each time we are called, but
     # it does not slow things down noticably.
     &Apache::loncoursedata::populate_weight_table();
+    #
     if (exists($ENV{'form.Excel'})) {
         &Excel_output($r);
-    } else {
+    } else { 
+        $r->print('<input type="submit" name="Excel" value="'.
+                  &mt('Produce Excel Output').'" />'.'&nbsp;'x5);
+        $r->rflush();
+        my $count = 0;
+        foreach my $seq (&Apache::lonstatistics::Sequences_with_Assess()) {
+            $count += $seq->{'num_assess_parts'};
+        }
+        if ($count > 10) {
+            $r->print('<h2>'.
+                      &mt('Compiling statistics for [_1] problems',$count).
+                      '</h2>');
+            if ($count > 30) {
+                $r->print('<h3>'.&mt('This will take some time.').'</h3>');
+            }
+            $r->rflush();
+        }
+        #
         my $sortby = $ENV{'form.sortby'};
         $sortby = 'container' if (! defined($sortby) || $sortby =~ /^\s*$/);
         my $plot = $ENV{'form.plot'};
-        &Apache::lonnet::logthis('form.plot = '.$plot);
+        if ($plot eq '' || $plot eq 'none') {
+            undef($plot);
+        }
         if ($sortby eq 'container' && ! defined($plot)) {
+            &output_sequence_statistics($r);
             &output_html_by_sequence($r);
         } else {
             if (defined($plot)) {
-                &Apache::lonnet::logthis('calling plot routine');
                 &make_plot($r,$plot);
             }
             &output_html_stats($r);
+            &output_sequence_statistics($r);
         }
     }
     return;
 }
 
+sub output_sequence_statistics {
+    my ($r) = @_;
+    my $c=$r->connection();
+    $r->print('<h2>'.&mt('Sequence Statistics').
+	      &Apache::loncommon::help_open_topic('Statistics_Sequence').
+	      '</h2>');
+    $r->print('<table border="0"><tr><td bgcolor="#777777">'."\n".
+              '<table border="0" cellpadding="3">'."\n".
+              '<tr bgcolor="#FFFFE6">');
+    $r->print(&sequence_html_header());
+    foreach my $seq (&Apache::lonstatistics::Sequences_with_Assess()) {
+        last if ($c->aborted);
+        next if ($seq->{'num_assess'} < 1);
+        &compute_sequence_statistics($seq);
+        $r->print(&sequence_html_output($seq));
+    }
+    $r->print('</table>');
+    $r->print('</table>');
+    $r->rflush();
+    return;
+}
+
+
 ##########################################################
 ##########################################################
 ##
@@ -385,7 +834,6 @@ sub output_html_stats {
     return;
 }
 
-
 sub html_preamble {
     my $Str='';
     $Str .= "<h2>".
@@ -417,6 +865,7 @@ sub statistics_html_table_data {
     my $row = '';
     foreach my $field (@Fields) {
         next if ($options =~ /no $field->{'name'}/);
+        next if ($field->{'selected'} ne 'yes');
         $row .= '<td bgcolor="'.$field->{'color'}.'"';
         if (exists($field->{'align'})) {
             $row .= ' align="'.$field->{'align'}.'"';
@@ -425,7 +874,7 @@ sub statistics_html_table_data {
         if (exists($field->{'special'}) && $field->{'special'} eq 'link') {
             $row .= '<a href="'.$data->{$field->{'name'}.'.link'}.'">';
         }
-        if (exists($field->{'format'})) {
+        if (exists($field->{'format'}) && $data->{$field->{'name'}} !~ /[A-Z]/i) {
             $row .= sprintf($field->{'format'},$data->{$field->{'name'}});
         } else {
             $row .= $data->{$field->{'name'}};
@@ -443,6 +892,7 @@ sub statistics_table_header {
     my $header_row;
     foreach my $field (@Fields) {
         next if ($options =~ /no $field->{'name'}/);
+        next if ($field->{'selected'} ne 'yes');
         $header_row .= '<th>';
         if (exists($field->{'sortable'}) && $field->{'sortable'} eq 'yes') {
             $header_row .= '<a href="javascript:'.
@@ -467,6 +917,40 @@ sub statistics_table_header {
     return $header_row;
 }
 
+sub sequence_html_header {
+    my $Str .= '<tr>';
+    foreach my $field (@SeqFields) {
+#        next if ($field->{'selected'} ne 'yes');
+        $Str .= '<th bgcolor="'.$field->{'color'}.'"';
+        $Str .= '>'.$field->{'title'}.'</th>';
+    }
+    $Str .= '</tr>';
+    return $Str;
+}
+
+
+sub sequence_html_output {
+    my ($seq) = @_;
+    my $data = $SeqStat{$seq->{'symb'}};
+    my $row = '<tr>';
+    foreach my $field (@SeqFields) {
+        next if ($field->{'selected'} ne 'yes');
+        $row .= '<td bgcolor="'.$field->{'color'}.'"';
+        if (exists($field->{'align'})) {
+            $row .= ' align="'.$field->{'align'}.'"';
+        }
+        $row .= '>';
+        if (exists($field->{'format'})) {
+            $row .= sprintf($field->{'format'},$data->{$field->{'name'}});
+        } else {
+            $row .= $data->{$field->{'name'}};
+        }
+        $row .= '</td>';
+    }
+    $row .= '</tr>'."\n";
+    return $row;
+}
+
 ####################################################
 ####################################################
 ##
@@ -480,6 +964,8 @@ sub make_plot {
     &sort_data($ENV{'form.sortby'});
     if ($plot eq 'degrees') {
         &degrees_plot($r);
+    } elsif ($plot eq 'tries statistics') {
+        &tries_data_plot($r);
     } else {
         &make_single_stat_plot($r,$plot);
     }
@@ -592,6 +1078,11 @@ sub degrees_plot {
     my $diffdata .= '<data>'.join(',',@Labels).'</data>'.$/.
                     '<data>'.join(',',@Diff).'</data>'.$/;
     #
+    my $title = 'Degree of Discrimination\nand Degree of Difficulty';
+    if ($xmax > 50) {
+        $title = 'Degree of Discrimination and Degree of Difficulty';
+    }
+    #
     $plot=<<"END";
 <gnuplot 
     texfont="10"
@@ -602,7 +1093,7 @@ sub degrees_plot {
     align="center"
     border="on"
     transparent="on"
-    alttag="Sample Plot"
+    alttag="Degree of Discrimination and Degree of Difficulty Plot"
     samples="100"
     bgcolor="xffffff"
     height="$height"
@@ -611,7 +1102,7 @@ sub degrees_plot {
         pos="top right"
         title=""
         box="off" />
-    <title>Degree of Discrmination and Degree of Difficulty</title>
+    <title>$title</title>
     <axis xmin="0" ymin="$ymin" xmax="$xmax" ymax="$ymax" color="x000000" />
     <xlabel>Problem Number</xlabel>
     <curve 
@@ -636,6 +1127,91 @@ END
     return;
 }
 
+sub tries_data_plot {
+    my ($r)=@_;
+    my $count = scalar(@StatsArray);
+    my $width = 50 + 10*$count;
+    $width = 300 if ($width < 300);
+    my $height = 300;
+    my $plot = '';
+    my @STD;  my @Mean; my @Max; my @Min;
+    my @Labels;
+    my $ymax = 5;
+    foreach my $data (@StatsArray) {
+        my $max = $data->{'mean_tries'} + $data->{'std_tries'};
+        $ymax = $max if ($ymax < $max);
+        $ymax = $max if ($ymax < $max);
+        push(@Labels,$data->{'problem_num'});
+        push(@STD,$data->{'std_tries'});
+        push(@Mean,$data->{'mean_tries'});
+    }
+    #
+    # Make sure we show relevant information.
+    my $xmax = $Labels[-1];
+    if ($xmax > 50) {
+        if ($xmax % 10 != 0) {
+            $xmax = 10 * (int($xmax/10)+1);
+        }
+    } else {
+        if ($xmax % 5 != 0) {
+            $xmax = 5 * (int($xmax/5)+1);
+        }
+    }
+    $ymax = int($ymax)+1+2;
+    #
+    my $std_data .= '<data>'.join(',',@Labels).'</data>'.$/.
+                    '<data>'.join(',',@Mean).'</data>'.$/;
+    #
+    my $std_error_data .= '<data>'.join(',',@Labels).'</data>'.$/.
+                          '<data>'.join(',',@Mean).'</data>'.$/.
+                          '<data>'.join(',',@STD).'</data>'.$/;
+    #
+    my $title = 'Mean and S.D. of Tries';
+    if ($xmax > 25) {
+        $title = 'Mean and Standard Deviation of Tries';
+    }
+    #
+    $plot=<<"END";
+<gnuplot 
+    texfont="10"
+    fgcolor="x000000"
+    plottype="Cartesian"
+    font="large"
+    grid="on"
+    align="center"
+    border="on"
+    transparent="on"
+    alttag="Mean and S.D of Tries Plot"
+    samples="100"
+    bgcolor="xffffff"
+    height="$height"
+    width="$width">
+    <title>$title</title>
+    <axis xmin="0" ymin="0" xmax="$xmax" ymax="$ymax" color="x000000" />
+    <xlabel>Problem Number</xlabel>
+    <ylabel>Number of Tries</ylabel>
+    <curve 
+        linestyle="yerrorbars"
+        name="S.D. Tries" 
+        pointtype="1" 
+        color="x666666">
+        $std_error_data
+    </curve>
+    <curve 
+        linestyle="points"
+        name="Mean Tries" 
+        pointtype="1" 
+        color="xCC4444">
+        $std_data
+    </curve>
+</gnuplot>
+END
+    my $plotresult = 
+        '<p>'.&Apache::lonxml::xmlparse($r,'web',$plot).'</p>'.$/;
+    $r->print($plotresult);
+    return;
+}
+
 sub plot_dropdown {
     my $current = '';
     #
@@ -646,12 +1222,15 @@ sub plot_dropdown {
     my @Additional_Plots = (
                             { graphable=>'yes',
                               name => 'degrees',
-                              title => 'DoDisc and DoDiff' });
+                              title => 'Difficulty Indexes' },
+                            { graphable=>'yes',
+                              name => 'tries statistics',
+                              title => 'Tries Statistics' });
     #
     my $Str= "\n".'<select name="plot" size="1">';
     $Str .= '<option name="none"></option>'."\n";
     $Str .= '<option name="none2">none</option>'."\n";
-    foreach my $field (@Fields,@Additional_Plots) {
+    foreach my $field (@Additional_Plots,@Fields) {
         if (! exists($field->{'graphable'}) ||
             $field->{'graphable'} ne 'yes') {
             next;
@@ -713,6 +1292,8 @@ sub Excel_output {
     }
     my $excel_sheet = $excel_workbook->addworksheet(
         &Apache::loncommon::clean_excel_name($sheetname));
+    #
+    my $format = &Apache::loncommon::define_excel_formats($excel_workbook);
     ##
     ## Begin creating excel sheet
     ##
@@ -720,28 +1301,17 @@ sub Excel_output {
     #
     # Put the course description in the header
     $excel_sheet->write($rows_output,$cols_output++,
-                   $ENV{'course.'.$ENV{'request.course.id'}.'.description'});
+                   $ENV{'course.'.$ENV{'request.course.id'}.'.description'},
+                        $format->{'h1'});
     $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);
+    $excel_sheet->write($rows_output,$cols_output++,
+                        &Apache::lonstathelpers::sections_description
+                            (@Apache::lonstatistics::SelectedSections),
+                        $format->{'h3'});
+    $cols_output += scalar(@Apache::lonstatistics::SelectedSections);
     #
     # Time restrictions
     my $time_string;
@@ -757,6 +1327,10 @@ sub Excel_output {
         # See note above about lonlocal:locallocaltime
         $time_string .= 'Data collected before '.localtime($endtime).'.';
     }
+    if (defined($time_string)) {
+        $excel_sheet->write($rows_output,$cols_output++,$time_string);
+        $cols_output+= 5;
+    }
     #
     # Put the date in there too
     $excel_sheet->write($rows_output,$cols_output++,
@@ -764,35 +1338,49 @@ sub Excel_output {
     #
     $rows_output++; 
     $cols_output=0;
-    #
-    # Long Headers
-    foreach my $field (@Fields) {
-        next if ($field->{'name'} eq 'problem_num');
-        if (exists($field->{'long_title'})) {
+    ##
+    ## Sequence Statistics
+    ## 
+    &write_headers($excel_sheet,$format,\$rows_output,\$cols_output,
+                   \@SeqFields);
+    foreach my $seq (&Apache::lonstatistics::Sequences_with_Assess()) {
+        next if ($seq->{'num_assess'} < 1);
+        my $data = $SeqStat{$seq->{'symb'}};
+        $cols_output=0;
+        foreach my $field (@SeqFields) {
+            next if ($field->{'selected'} ne 'yes');
+            my $fieldformat = undef;
+            if (exists($field->{'excel_format'})) {
+                $fieldformat = $format->{$field->{'excel_format'}};
+            }
             $excel_sheet->write($rows_output,$cols_output++,
-                                $field->{'long_title'});
-        } else {
-            $excel_sheet->write($rows_output,$cols_output++,'');
+                                $data->{$field->{'name'}},$fieldformat);
         }
+        $rows_output++;
+        $cols_output=0;
     }
+    ##
+    ## Resource Statistics
+    ##
     $rows_output++;
     $cols_output=0;
-    # Brief headers
-    foreach my $field (@Fields) {
-        next if ($field->{'name'} eq 'problem_num');
-        # Use english for excel as I am not sure how well excel handles 
-        # other character sets....
-        $excel_sheet->write($rows_output,$cols_output++,$field->{'title'});
-    }
-    $rows_output++;
+    &write_headers($excel_sheet,$format,\$rows_output,\$cols_output,
+                   \@Fields);
+    #
     foreach my $data (@StatsArray) {
         $cols_output=0;
         foreach my $field (@Fields) {
+            next if ($field->{'selected'} ne 'yes');
             next if ($field->{'name'} eq 'problem_num');
+            my $fieldformat = undef;
+            if (exists($field->{'excel_format'})) {
+                $fieldformat = $format->{$field->{'excel_format'}};
+            }
             $excel_sheet->write($rows_output,$cols_output++,
-                                $data->{$field->{'name'}});
+                                $data->{$field->{'name'}},$fieldformat);
         }
         $rows_output++;
+        $cols_output=0;
     }
     #
     $excel_workbook->close();
@@ -805,6 +1393,44 @@ sub Excel_output {
     return;
 }
 
+##
+## &write_headers
+##
+sub write_headers {
+    my ($excel_sheet,$format,$rows_output,$cols_output,$Fields) = @_;
+    ##
+    ## First the long titles
+    foreach my $field (@{$Fields}) {
+        next if ($field->{'name'} eq 'problem_num');
+        next if ($field->{'selected'} ne 'yes');
+        if (exists($field->{'long_title'})) {
+            $excel_sheet->write($$rows_output,${$cols_output},
+                                $field->{'long_title'},
+                                $format->{'bold'});
+        } else {
+            $excel_sheet->write($$rows_output,${$cols_output},'');
+        }
+        ${$cols_output}+= 1;
+    }
+    ${$cols_output} =0;
+    ${$rows_output}+=1;
+    ##
+    ## Then the short titles
+    foreach my $field (@{$Fields}) {
+        next if ($field->{'selected'} ne 'yes');
+        next if ($field->{'name'} eq 'problem_num');
+        # Use english for excel as I am not sure how well excel handles 
+        # other character sets....
+        $excel_sheet->write($$rows_output,$$cols_output,
+                            $field->{'title'},
+                            $format->{'bold'});
+        $$cols_output+=1;
+    }
+    ${$cols_output} =0;
+    ${$rows_output}+=1;
+    return;
+}
+
 ##################################################
 ##################################################
 ##
@@ -818,6 +1444,7 @@ sub compute_statistics_on_sequence {
     foreach my $res (@{$seq->{'contents'}}) {
         next if ($res->{'type'} ne 'assessment');
         foreach my $part (@{$res->{'parts'}}) {
+            next if ($res->{'partdata'}->{$part}->{'Survey'});
             #
             # This is where all the work happens
             my $data = &get_statistics($seq,$res,$part,scalar(@StatsArray)+1);
@@ -838,6 +1465,7 @@ sub compute_all_statistics {
     foreach my $seq (&Apache::lonstatistics::Sequences_with_Assess()) {
         last if ($c->aborted);
         next if ($seq->{'num_assess'} < 1);
+        &compute_sequence_statistics($seq);
         &compute_statistics_on_sequence($seq);
     }
 }
@@ -930,6 +1558,7 @@ sub get_statistics {
                         (\@Apache::lonstatistics::SelectedSections,
                          $Apache::lonstatistics::enrollment_status,
                          $symb,$part,$courseid,$starttime,$endtime);
+    $data->{'symb'}        = $symb;
     $data->{'part'}        = $part;
     $data->{'problem_num'} = $problem_num;
     $data->{'container'}   = $sequence->{'title'};
@@ -937,11 +1566,44 @@ sub get_statistics {
     $data->{'title.link'}  = $resource->{'src'}.'?symb='.
         &Apache::lonnet::escape($resource->{'symb'});
     #
-    $data->{'deg_of_disc'} = &compute_discrimination_factor($resource,$part,$sequence);
+    if ($SelectedFields{'deg_of_disc'}) {
+        $data->{'deg_of_disc'} = 
+            &compute_discrimination_factor($resource,$part,$sequence);
+    }
+    #
+    # Store in metadata if computations were done for all students
+    if ($data->{'num_students'} > 1) {
+        my @Sections = @Apache::lonstatistics::SelectedSections;
+        my $sections = '"'.join(' ',@Sections).'"';
+        $sections =~ s/&+/_/g;  # Ensure no special characters
+        $data->{'sections'}=$sections;
+        $data->{'course'} = $ENV{'request.course.id'};
+        my $urlres=(&Apache::lonnet::decode_symb($resource->{'symb'}))[2];
+        $data->{'urlres'}=$urlres;
+        my %storestats = 
+            &LONCAPA::lonmetadata::dynamic_metadata_storage($data);
+        my ($dom,$user) = $urlres=~/^(\w+)\/(\w+)/; 
+        &Apache::lonnet::put('nohist_resevaldata',\%storestats,$dom,$user);
+    }
+    #
+    $data->{'tries_per_correct'} = $data->{'tries'} / 
+        ($data->{'num_solved'}+0.1);
+    #
+    # Get the due date for research purposes (commented out most of the time)
+#    $data->{'duedate'} = 
+#        &Apache::lonnet::EXT('resource.'.$part.'.duedate',$symb);
+#    $data->{'opendate'} = 
+#        &Apache::lonnet::EXT('resource.'.$part.'.opendate',$symb);
+#    $data->{'maxtries'} = 
+#        &Apache::lonnet::EXT('resource.'.$part.'.maxtries',$symb);
+#    $data->{'hinttries'} =
+#        &Apache::lonnet::EXT('resource.'.$part.'.hinttries',$symb);
+#    $data->{'weight'} =
+#        &Apache::lonnet::EXT('resource.'.$part.'.weight',$symb);
+#    $data->{'resptypes'} = join(',',@{$resource->{'partdata'}->{$part}->{'ResponseTypes'}});
     return $data;
 }
 
-
 ###############################################
 ###############################################
 
@@ -966,11 +1628,13 @@ sub compute_discrimination_factor {
     }
     #
     # rank
+    my ($starttime,$endtime) = &Apache::lonstathelpers::get_time_limits();
     my $ranking = 
         &Apache::loncoursedata::rank_students_by_scores_on_resources
         (\@Resources,
          \@Apache::lonstatistics::SelectedSections,
-         $Apache::lonstatistics::enrollment_status,undef);
+         $Apache::lonstatistics::enrollment_status,undef,
+         $starttime,$endtime);
     #
     # compute their percent scores on the problems in the sequence,
     my $number_to_grab = int(scalar(@{$ranking})/4);
@@ -981,10 +1645,16 @@ sub compute_discrimination_factor {
         map { 
             $_->[&Apache::loncoursedata::RNK_student()]; 
           } @{$ranking}[($num_students-$number_to_grab)..($num_students-1)];
+    if (! @BottomSet || (@BottomSet == 1 && $BottomSet[0] eq '') ||
+        ! @TopSet    || (@TopSet    == 1 && $TopSet[0]    eq '')) {
+        return 'nan';
+    }
     my ($bottom_sum,$bottom_max) = 
-        &Apache::loncoursedata::get_sum_of_scores($resource,$part,\@BottomSet);
+        &Apache::loncoursedata::get_sum_of_scores($resource,$part,\@BottomSet,
+                                                  undef,$starttime,$endtime);
     my ($top_sum,$top_max) = 
-        &Apache::loncoursedata::get_sum_of_scores($resource,$part,\@TopSet);
+        &Apache::loncoursedata::get_sum_of_scores($resource,$part,\@TopSet,
+                                                  undef,$starttime,$endtime);
     my $deg_of_disc;
     if ($top_max == 0 || $bottom_max==0) {
         $deg_of_disc = 'nan';
@@ -998,6 +1668,70 @@ sub compute_discrimination_factor {
 
 ###############################################
 ###############################################
+##
+## Compute KR-21
+##
+## To compute KR-21, you need the following information:
+##
+## K=the number of items in your test
+## M=the mean score on the test
+## s=the standard deviation of the scores on your test 
+##
+## then:
+## 
+## KR-21 rk= [K/(K-1)] * [1- (M*(K-M))/(K*s^2))]
+##
+###############################################
+###############################################
+sub compute_sequence_statistics {
+    my ($seq) = @_;
+    my $symb = $seq->{'symb'};
+    my @Resources;
+    foreach my $res (@{$seq->{'contents'}}) {
+        next if ($res->{'type'} ne 'assessment');
+        push (@Resources,$res->{'symb'});
+    }
+    my ($starttime,$endtime) = &Apache::lonstathelpers::get_time_limits();
+    #
+    # First compute statistics based on student scores
+    my ($smin,$smax,$sMean,$sSTD,$scount,$sMAX) = 
+        &Apache::loncoursedata::score_stats
+                    (\@Apache::lonstatistics::SelectedSections,
+                     $Apache::lonstatistics::enrollment_status,
+                     \@Resources,$starttime,$endtime,undef);
+    $SeqStat{$symb}->{'title'}  = $seq->{'title'};
+    $SeqStat{$symb}->{'scoremax'}  = $smax;
+    $SeqStat{$symb}->{'scoremin'}  = $smin;
+    $SeqStat{$symb}->{'scoremean'} = $sMean;
+    $SeqStat{$symb}->{'scorestd'}  = $sSTD;
+    $SeqStat{$symb}->{'scorecount'} = $scount;
+    $SeqStat{$symb}->{'max_possible'} = $sMAX;
+    #
+    # Compute statistics based on the number of correct problems
+    # 'correct' is taken to mean 
+    my ($cmin,$cmax,$cMean,$cSTD,$ccount)=
+        &Apache::loncoursedata::count_stats
+        (\@Apache::lonstatistics::SelectedSections,
+         $Apache::lonstatistics::enrollment_status,
+         \@Resources,$starttime,$endtime,undef);
+    my $K = $seq->{'num_assess_parts'};
+    my $kr_21;
+    if ($K > 1 && $cSTD > 0) {
+        $kr_21 =  ($K/($K-1)) * (1 - $cMean*($K-$cMean)/($K*$cSTD**2));
+    } else {
+        $kr_21 = 'nan';
+    }
+    $SeqStat{$symb}->{'countmax'} = $cmax;
+    $SeqStat{$symb}->{'countmin'} = $cmin;
+    $SeqStat{$symb}->{'countstd'} = $cSTD;
+    $SeqStat{$symb}->{'countmean'} = $cMean;
+    $SeqStat{$symb}->{'count'} = $ccount;
+    $SeqStat{$symb}->{'items'} = $K;
+    $SeqStat{$symb}->{'KR-21'}=$kr_21;
+    return;
+}
+
+
 
 =pod 
 
@@ -1058,111 +1792,6 @@ Number of Students had at least one disc
 
 =cut
 
-
-############################################################
-############################################################
-##
-##  How this all works:
-##     Statistics are computed by calling &get_statistics with the sequence,
-##     resource, and part id to run statistics on.  At various places within
-##     the loops which compute the statistics, as well as before and after 
-##     the entire process, subroutines can be called.  The subroutines are
-##     registered to the following hooks:
-##
-##         hook          subroutine inputs
-##     ----------------------------------------------------------
-##         pre           $r,$count
-##         pre_seq       $r,$count,$seq
-##         pre_res       $r,$count,$seq,$res
-##         calc          $r,$count,$seq,$res,$data
-##         post_res      $r,$count,$seq,$res
-##         post_seq      $r,$count,$seq
-##         post          $r,$count
-##
-##         abort         $r
-##
-##     subroutines will be called in the order in which they are registered.
-##   
-############################################################
-############################################################
-{
-
-my %hooks;
-my $aborted = 0;
-
-sub abort_computation {
-    $aborted = 1;
-}
-
-sub clear_hooks {
-    $aborted = 0;
-    undef(%hooks);
-}
-
-sub register_hook {
-    my ($hookname,$subref)=@_;
-    if ($hookname !~ /^(pre|pre_seq|pre_res|post|post_seq|post_res|calc)$/){
-        return;
-    }
-    if (ref($subref) ne 'CODE') {
-        &Apache::lonnet::logthis('attempt to register hook to non-code: '.
-                                 $hookname,' = '.$subref);
-    } else {
-        if (exists($hooks{$hookname})) {
-            push(@{$hooks{$hookname}},$subref);
-        } else {
-            $hooks{$hookname} = [$subref];
-        }
-    }
-    return;
-}
-
-sub run_hooks {
-    my $context = shift();
-    foreach my $hook (@{$hooks{$context}}) { 
-        if ($aborted && $context ne 'abort') {
-            last;
-        }
-        my $retvalue = $hook->(@_);
-        if (defined($retvalue) && $retvalue eq '0') {
-            $aborted = 1 if (! $aborted);
-        }
-    }
-}
-
-sub run_statistics {
-    my ($r) = @_;
-    my $count = 0;
-    &run_hooks('pre',$r,$count);
-    foreach my $seq (&Apache::lonstatistics::Sequences_with_Assess()) {
-        last if ($aborted);
-        next if ($seq->{'num_assess'}<1);
-        &run_hooks('pre_seq',$r,$count,$seq);
-        foreach my $res (@{$seq->{'contents'}}) {
-            last if ($aborted);
-            next if ($res->{'type'} ne 'assessment');
-            &run_hooks('pre_res',$r,$count,$seq,$res);            
-            foreach my $part (@{$res->{'parts'}}) {
-                last if ($aborted);
-                #
-                # This is where all the work happens
-                my $data = &get_statistics($seq,$res,$part,++$count);
-                &run_hooks('calc',$r,$count,$seq,$res,$part,$data); 
-            }
-            &run_hooks('post_res',$r,$count,$seq,$res);
-        }
-        &run_hooks('post_seq',$r,$count,$seq);
-    }
-    if ($aborted) {
-        &run_hooks('abort',$r);
-    } else {
-        &run_hooks('post',$r,$count);
-    }
-    return;
-}
-
-} # End of %hooks scope
-
 ############################################################
 ############################################################