--- loncom/interface/statistics/lonstudentassessment.pm 2003/01/09 15:59:17 1.25
+++ loncom/interface/statistics/lonstudentassessment.pm 2003/02/28 21:19:00 1.30
@@ -1,7 +1,6 @@
# The LearningOnline Network with CAPA
-# (Publication Handler
#
-# $Id: lonstudentassessment.pm,v 1.25 2003/01/09 15:59:17 matthew Exp $
+# $Id: lonstudentassessment.pm,v 1.30 2003/02/28 21:19:00 matthew Exp $
#
# Copyright Michigan State University Board of Trustees
#
@@ -25,389 +24,432 @@
# 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/6,3/17,3/21,3/22,3/26,4/7,5/6 Behrouz Minaei
-# 5/12,5/14,5/15,5/19,5/26,7/16,12/24 Behrouz Minaei
#
-###
+#######################################################
+#######################################################
+
+=pod
+
+=head1 NAME
+
+lonstudentassessment
+
+=head1 SYNOPSIS
+
+Presents assessment data about a student or a group of students.
+
+=head1 Subroutines
+
+=over 4
+
+=cut
+
+#######################################################
+#######################################################
package Apache::lonstudentassessment;
use strict;
+use Apache::lonstatistics;
use Apache::lonhtmlcommon;
use Apache::loncoursedata;
+use Apache::lonnet; # for logging porpoises
use GDBM_File;
-#my $jr;
+my $Statistics;
-sub BuildStudentAssessmentPage {
- my ($cacheDB,$students,$courseID,$formName,$headings,$spacing,
- $studentInformation,$r,$c)=@_;
-# $jr = $r;
- my %cache;
- unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER(),0640)) {
- $r->print('Unable to tie database.');
- return;
- }
-
- # 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);
- }
- }
- my ($infoHeadings, $infoKeys, $sequenceHeadings, $sequenceKeys,
- $doNotShow) =
- &ShouldShowColumns(\%cache, $headings, $studentInformation);
-
- my $selectedName = &FindSelectedStudent(\%cache,
- $cache{'StudentAssessmentStudent'},
- $students);
- $r->print(&CreateInterface(\%cache, $selectedName, $students, $formName,
- $doNotShow));
- $r->rflush();
+#######################################################
+#######################################################
- my $Str = '';
- if($selectedName eq 'No Student Selected') {
- $Str .= '
WARNING: ';
- $Str .= 'Please select a student
';
- $r->print($Str);
- return;
- }
+=pod
+
+=item &BuildStudentAssessmentPage()
+
+Inputs:
+
+=over 4
+
+=item $cacheDB The name of the cache file used to store student data
+
+=item $students Array ref containing the name(s) of the students
+selected for display
- $r->print(&CreateTableHeadings(\%cache, $spacing, $infoKeys, $infoHeadings,
- $sequenceKeys, $sequenceHeadings));
- untie(%cache);
- if($c->aborted()) { return $Str; }
+=item $courseID The ID of the course
- my $selected=0;
+=item $formName The name of the html form - 'Statistics'
+
+=item $headings Array ref of headings to show
+
+=item $spacing A string of spaces
+
+=item $studentInformation Array ref of possible headings for student info
+('fullname','section',...)
+
+=item $r Apache Request
+
+=item $c Apache Connection
+
+=back
+
+=cut
+
+#######################################################
+#######################################################
+sub BuildStudentAssessmentPage {
+ my ($r,$c)=@_;
+ undef($Statistics);
+
+ #
+ $r->print(&CreateInterface());
+ $r->rflush();
+ #
+ $r->print(&CreateTableHeadings());
+ if($c->aborted()) { return ; }
+
my $Count = 0;
$r->print('
';
- $r->print($Str);
+ $r->rflush();
}
-
+ $r->print(''."\n");
+ my $Str;
return;
}
+#######################################################
+#######################################################
+
+=pod
+=item &CreateInterface()
+Called by &BuildStudentAssessmentPage to create the top part of the
+page which displays the chart.
-#---- Student Assessment Web Page --------------------------------------------
+Inputs: None
-sub CreateInterface {
- my($cache,$selectedName,$students,$formName,$doNotShow)=@_;
+Returns: A string containing the HTML for the headers and top table for
+the chart page.
- my $Str = '';
- $Str .= &CreateLegend();
- $Str .= '
'."\n";
+=item Table Output Routines
- return $Str;
-}
+=over 4
+
+=cut
+
+#######################################################
+#######################################################
+{
+ my $padding;
+
+#######################################################
+#######################################################
=pod
-=item &FormatStudentData()
+=item &CreateTableHeadings()
+
+Create HTML for the columns of student data to show.
+Called by &BuildStudentAssessmentPage(). Calls
+&Apache::lonhtmlcommon::CreateHeadings().
-First, FormatStudentInformation is called and prefixes the course information.
-This function produces a formatted string of the student\'s course information.
-Each column of data represents all the problems for a given sequence. For
-valid grade data, a link is created for that problem to a submission record
-for that problem.
+Inputs:
=over 4
-Input: $name, $studentInformation, $ChartDB
+=item $cache The ubiquitous cache
-$name: The name and domain of the current student in name:domain format
+=item $spacing A string of spaces
-$studentInformation: A pointer to an array holding the names used to
-remove data from the hash. They represent
-the name of the data to be removed.
+=item $infoKeys Array ref to names of keys to display from the cache
+which describe students
-$ChartDB: The name of the cached data database which will be tied to that
-database.
+=item $infoHeadings Array ref to headings of columns for student info
-Output: $Str
+=item $sequenceKeys Array ref of names of keys to use to retrieve sequence
+data from the cache
-$Str: Formatted string that is an entire row of the chart. It is a
-concatenation of student information and student course information.
+=item $sequenceHeadings Array ref of names of sequences used for output.
=back
+Returns: A string containing the HTML of the table headings.
+
=cut
-sub StudentReport {
- my ($cache,$name,$spacing,$showSequences)=@_;
- my ($username,$domain)=split(':',$name);
+#######################################################
+#######################################################
+sub CreateTableHeadings {
+ #
+ $padding = ' 'x3;
+ #
+ my $Str = '
';
+ # First, the @StudentData fields need to be listed
+ my @to_show = @Apache::lonstatistics::SelectedStudentData;
+ foreach (@to_show) {
+ if ($_ eq 'all') {
+ @to_show = @Apache::lonstatistics::StudentDataOrder;
+ last;
+ }
+ }
+ foreach my $field (@to_show) {
+ my $title=$Apache::lonstatistics::StudentData{$field}->{'title'};
+ my $base =$Apache::lonstatistics::StudentData{$field}->{'base_width'};
+ my $width=$Apache::lonstatistics::StudentData{$field}->{'width'};
+ $Str .= $title.' 'x($width-$base).$padding;
+ }
+ # Now the selected sequences need to be listed
+ foreach my $map_symb (@Apache::lonstatistics::SelectedMaps) {
+ foreach my $sequence (@Apache::lonstatistics::Sequences) {
+ next if ($sequence->{'symb'} ne $map_symb && $map_symb ne 'all');
+ next if ($sequence->{'num_assess'} < 1);
+ my $title = $sequence->{'title'};
+ my $base = $sequence->{'base_width'};
+ my $width = $sequence->{'width'};
+ $Str .= $title.' 'x($width-$base).$padding;
+ }
+ }
+ $Str .= 'total';
+ $Str .= "
\n";
+ return $Str;
+}
+
+#######################################################
+#######################################################
+
+=pod
+
+=item &ChartOutputStudent($student)
+
+Return a line of the chart for a student.
+=cut
+
+#######################################################
+#######################################################
+sub ChartOutputStudent {
+ my $student = shift;
my $Str = '';
- if(defined($cache->{$name.':error'})) {
- return $Str;
+ # First, the @StudentData fields need to be listed
+ my @to_show = @Apache::lonstatistics::SelectedStudentData;
+ foreach (@to_show) {
+ if ($_ eq 'all') {
+ @to_show = @Apache::lonstatistics::StudentDataOrder;
+ last;
+ }
+ }
+ foreach my $field (@to_show) {
+ my $title=$student->{$field};
+ my $base =scalar(my @Tmp = split(//,$title));
+ my $width=$Apache::lonstatistics::StudentData{$field}->{'width'};
+ $Str .= $title.' 'x($width-$base).$padding;
+ }
+ # Get ALL the students data
+ my %StudentsData;
+ my @tmp = &Apache::loncoursedata::get_current_state
+ ($student->{'username'},$student->{'domain'},undef,
+ $ENV{'request.course.id'});
+ if ((scalar @tmp > 0) && ($tmp[0] !~ /^error:/)) {
+ %StudentsData = @tmp;
}
- if($cache->{$name.':error'} =~ /course/) {
- $Str .= 'No course data for student ';
- $Str .= ''.$username.'. ';
+ if (scalar(@tmp) < 1) {
+ $Str .= 'No Course Data'."\n";
return $Str;
}
+ #
+ # By sequence build up the data
+ my $studentstats;
+ foreach my $map_symb (@Apache::lonstatistics::SelectedMaps) {
+ foreach my $seq (@Apache::lonstatistics::Sequences) {
+ next if ($map_symb ne $seq->{'symb'} && $map_symb ne 'all');
+ next if ($seq->{'num_assess'} < 1);
+ my ($performance,$score,$seq_max) =
+ &StudentPerformanceOnSequence($student,\%StudentsData,
+ $seq,'linkify');
+ $Str .= $performance.$padding;
+ $studentstats->{$seq->{'symb'}}->{'score'}= $score;
+ $studentstats->{$seq->{'symb'}}->{'max'} = $seq_max;
+ }
+ }
+ #
+ # Total it up and store the statistics info.
+ my ($score,$max) = (0,0);
+ while (my ($symb,$seq_stats) = each (%{$studentstats})) {
+ $Statistics->{$symb}->{'score'} += $seq_stats->{'score'};
+ $Statistics->{$symb}->{'max'} += $seq_stats->{'max'};
+ $score += $seq_stats->{'score'};
+ $max += $seq_stats->{'max'};
+ }
+ my $scorelength = scalar(my @tmp1 = split(//,$score));
+ my $maxlength = scalar(my @tmp2 = split(//,$max));
+ $Str .= ' '.' 'x($maxlength-$scorelength).$score.'/'.$max;
+ $Str .= " \n";
+ return $Str;
+}
- my $hasVersion = 'false';
- my $hasFinalData = 'false';
- foreach my $sequence (@$showSequences) {
- my $hasData = 'false';
- my $characterCount=0;
- foreach my $problemID (split(':', $cache->{$sequence.':problems'})) {
- my $problem = $cache->{$problemID.':problem'};
- # All grades (except for versionless parts) are displayed as links
- # to their submission record. Loop through all the parts for the
- # current problem in the correct order and prepare the output links
- foreach(split(/\:/,$cache->{$sequence.':'.$problemID.
- ':parts'})) {
- if($cache->{$name.':'.$problemID.':NoVersion'} eq 'true' ||
- $cache->{$name.':'.$problemID.':'.$_.':code'} eq ' ' ||
- $cache->{$name.':'.$problemID.':'.$_.':code'} eq '') {
- $Str .= ' ';
- $characterCount++;
- next;
- }
- $hasVersion = 'true';
- $hasData = 'true';
- if (lc($ENV{'form.displaymode'}) ne 'display without links') {
- $Str .= '';
- }
- my $code = $cache->{$name.':'.$problemID.':'.$_.':code'};
- my $tries = $cache->{$name.':'.$problemID.':'.$_.':tries'};
- if($code eq '*' && $tries < 10 && $tries ne '') {
- $code = $tries;
- }
- $Str .= $code;
- if (lc($ENV{'form.displaymode'}) ne 'display without links') {
- $Str .= '';
- }
- $characterCount++;
- }
- }
+#######################################################
+#######################################################
- # Output the number of correct answers for the current sequence.
- # This part takes up 6 character slots, but is formated right
- # justified.
- my $spacesNeeded=$cache->{$sequence.':columnWidth'}-$characterCount;
- $spacesNeeded -= 3;
- $Str .= (' 'x$spacesNeeded);
-
-# my $outputProblemsCorrect = sprintf("%3d", $cache->{$name.':'.$sequence.
-# ':problemsCorrect'});
-
- my $outputProblemsCorrect = sprintf("%2d/%2d", $cache->{$name.':'.$sequence.
- ':problemsCorrect'},
- $characterCount);
- if($hasData eq 'true') {
- $Str .= ''.$outputProblemsCorrect.'';
- $hasFinalData = 'true';
- } else {
- $Str .= ' ';
- }
- $Str .= $spacing;
- }
+=pod
- # Output the total correct problems over the total number of problems.
- # I don't like this type of formatting, but it is a solution. Need
- # a way to dynamically determine the space requirements.
- my $outputProblemsSolved = sprintf("%4d", $cache->{$name.':problemsSolved'});
- my $outputTotalProblems = sprintf("%4d", $cache->{$name.':totalProblems'});
- if($hasFinalData eq 'true') {
- $Str .= ''.$outputProblemsSolved.
- ' / '.$outputTotalProblems.'';
- } else {
- $Str .= ' ';
- }
+=back
- if($hasVersion eq 'false') {
- $Str = 'No course data.';
- }
+=cut
+
+#######################################################
+#######################################################
- return $Str;
}
+#######################################################
+#######################################################
+
+=pod
+
+=item &StudentPerformanceOnSequence()
+
+Inputs:
+
+=over 4
+
+=item $student
+
+=item $studentdata Hash ref to all student data
+
+=item $seq Hash ref, the sequence we are working on
+=item $links if defined we will output links to each resource.
+
+=back
+
+=cut
+
+#######################################################
+#######################################################
+sub StudentPerformanceOnSequence {
+ my ($student,$studentdata,$seq,$links) = @_;
+ my $Str = '';
+ my $output_width = 0;
+ my ($sum,$max) = (0,0);
+ foreach my $resource (@{$seq->{'contents'}}) {
+ next if ($resource->{'type'} ne 'assessment');
+ my $resource_data = $studentdata->{$resource->{'symb'}};
+ my $value = '';
+ foreach my $partnum (@{$resource->{'parts'}}) {
+ $max++;
+ my $symbol = ' '; # default to space
+ #
+ if (exists($resource_data->{'resource.'.$partnum.'.solved'})) {
+ my $status = $resource_data->{'resource.'.$partnum.'.solved'};
+ if ($status eq 'correct_by_override') {
+ $symbol = '+';
+ $sum++;
+ } elsif ($status eq 'incorrect_by_override') {
+ $symbol = '-';
+ } elsif ($status eq 'ungraded_attempted') {
+ $symbol = '#';
+ } elsif ($status eq 'incorrect_attempted') {
+ $symbol = '.';
+ } elsif ($status eq 'excused') {
+ $symbol = 'x';
+ $max--;
+ } elsif ($status eq 'correct_by_student' &&
+ exists($resource_data->{'resource.'.$partnum.'.tries'})){
+ my $num = $resource_data->{'resource.'.$partnum.'.tries'};
+ if ($num > 9) {
+ $symbol = '*';
+ } elsif ($num > 0) {
+ $symbol = $num;
+ } else {
+ $symbol = ' ';
+ }
+ $sum++;
+ } else {
+ $symbol = ' ';
+ }
+ } else {
+ # Unsolved. Did they try?
+ if (exists($resource_data->{'resource.'.$partnum.'.tries'})){
+ $symbol = '.';
+ } else {
+ $symbol = ' ';
+ }
+ }
+ #
+ $output_width++;
+ if (defined($links) && $symbol ne ' ') {
+ $symbol = ''.$symbol.'';
+ }
+ $value .= $symbol;
+ }
+ $Str .= $value;
+ }
+ # Put on the totals
+ my $ratio = $sum.'/'.$max;
+ my $ratio_length = scalar(my @tmp1 = split(//,$ratio));
+ # Pad with extra spaces
+ my $width = $seq->{'width'};
+ $Str .= ' 'x($width-$output_width-$ratio_length).$ratio;
+ #
+ return ($Str,$sum,$max);
+}
+
+#######################################################
+#######################################################
sub StudentAverageTotal {
my ($cache, $students, $sequenceKeys)=@_;
my $Str = "\nSummary Tables:\n";
@@ -426,12 +468,14 @@ sub StudentAverageTotal {
$TotalProblems += $cache->{$name.':totalProblems'};
}
if ($StudentCount) {
- $ProblemsSolved /= $StudentCount;
+ $ProblemsSolved = sprintf( "%.2f",
+ $ProblemsSolved/$StudentCount);
$TotalProblems /= $StudentCount;
} else {
$ProblemsSolved = 0;
$TotalProblems = 0;
}
+
$Str .= '
".
" 1 correct by student in 1 try\n".
@@ -516,210 +541,18 @@ sub CreateLegend {
return $Str;
}
-=pod
-
-=item &CreateColumnSelectionBox()
-
-If there are columns not being displayed then this selection box is created
-with a list of those columns. When selections are made and the page
-refreshed, the columns will be removed from this box and the column is
-put back in the chart. If there is no columns to select, no row is added
-to the interface table.
-
-=over 4
-Input: $CacheData, $headings
-
-
-$CacheData: A pointer to a hash tied to the cached data
-
-$headings: An array of the names of the columns for the student information.
-They are used for displaying which columns are missing.
-
-Output: $notThere
-
-$notThere: The string contains one row of a table. The first column has the
-name of the selection box. The second contains the selection box
-which has a size of four.
-
-=back
-
-=cut
-
-sub CreateColumnSelectionBox {
- my ($doNotShow)=@_;
-
- my $notThere = '';
- $notThere .= '';
-
- return $notThere;
-}
-
-=pod
-
-=item &CreateColumnSelectors()
-
-This function generates the checkboxes above the column headings. The
-column will be removed if the checkbox is unchecked.
-
-=over 4
-
-Input: $CacheData, $headings
-
-$CacheData: A pointer to a hash tied to the cached data
-
-$headings: An array of the names of the columns for the student
-information. They are used to know what are the student information columns
-
-Output: $present
-
-$present: The string contains the first row of a table. Each column contains
-a checkbox which is left justified. Currently left justification is used
-for consistency of location over the column in which it presides.
-
-=back
-
-=cut
-
-sub CreateColumnSelectors {
- my ($infoHeadings, $sequenceHeadings, $sequenceKeys)=@_;
-
- my $present = '';
- for(my $index=0; $index<(scalar @$infoHeadings); $index++) {
- $present .= '