--- loncom/interface/statistics/lonstudentassessment.pm 2002/07/26 19:49:26 1.4
+++ loncom/interface/statistics/lonstudentassessment.pm 2003/02/28 21:19:00 1.30
@@ -1,12 +1,10 @@
# The LearningOnline Network with CAPA
-# (Publication Handler
#
-# $Id: lonstudentassessment.pm,v 1.4 2002/07/26 19:49:26 stredwic Exp $
+# $Id: lonstudentassessment.pm,v 1.30 2003/02/28 21:19:00 matthew Exp $
#
# Copyright Michigan State University Board of Trustees
#
# This file is part of the LearningOnline Network with CAPA (LON-CAPA).
-#
# LON-CAPA is free software; you can redistribute it and/or modify
# it under the terms of the GNU General Public License as published by
# the Free Software Foundation; either version 2 of the License, or
@@ -26,605 +24,535 @@
# 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 Behrouz Minaei
#
-###
+#######################################################
+#######################################################
-package Apache::lonstudentassessment;
+=pod
-use strict;
-use Apache::lonhtmlcommon;
-use Apache::loncoursedata;
-use GDBM_File;
+=head1 NAME
-#my $jr;
+lonstudentassessment
-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('
';
- $r->print($Str);
- return;
- }
+Presents assessment data about a student or a group of students.
- $r->print(&CreateTableHeadings(\%cache, $spacing, $infoKeys, $infoHeadings,
- $sequenceKeys, $sequenceHeadings));
- untie(%cache);
+=head1 Subroutines
- my $selected=0;
- $r->print('
'."\n";
+=item &BuildStudentAssessmentPage()
- return $Str;
-}
+Inputs:
-=pod
+=over 4
-=item &FormatStudentData()
+=item $cacheDB The name of the cache file used to store student data
-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.
+=item $students Array ref containing the name(s) of the students
+selected for display
-=over 4
+=item $courseID The ID of the course
-Input: $name, $studentInformation, $ChartDB
+=item $formName The name of the html form - 'Statistics'
-$name: The name and domain of the current student in name:domain format
+=item $headings Array ref of headings to show
-$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 $spacing A string of spaces
-$ChartDB: The name of the cached data database which will be tied to that
-database.
+=item $studentInformation Array ref of possible headings for student info
+('fullname','section',...)
-Output: $Str
+=item $r Apache Request
-$Str: Formatted string that is an entire row of the chart. It is a
-concatenation of student information and student course information.
+=item $c Apache Connection
=back
=cut
-sub StudentReport {
- my ($cache,$name,$spacing,$showSequences)=@_;
- my ($username,$domain)=split(':',$name);
+#######################################################
+#######################################################
+sub BuildStudentAssessmentPage {
+ my ($r,$c)=@_;
+ undef($Statistics);
+
+ #
+ $r->print(&CreateInterface());
+ $r->rflush();
+ #
+ $r->print(&CreateTableHeadings());
+ if($c->aborted()) { return ; }
+
+ my $Count = 0;
+ $r->print('
'."\n");
+ my $Str;
+ return;
+}
- my $Version;
- my $problemsCorrect = 0;
- my $totalProblems = 0;
- my $problemsSolved = 0;
- my $numberOfParts = 0;
-# foreach my $sequence (split(':', $cache->{'orderedSequences'})) {
- foreach my $sequence (@$showSequences) {
- my $characterCount=0;
- foreach my $problemID (split(':', $cache->{$sequence.':problems'})) {
- my $problem = $cache->{$problemID.':problem'};
- my $LatestVersion = $cache->{$name.':version:'.$problem};
-
- # Output dashes for all the parts of this problem if there
- # is no version information about the current problem.
- if(!$LatestVersion) {
- foreach my $part (split(/\:/,$cache->{$sequence.':'.
- $problemID.
- ':parts'})) {
- $Str .= ' ';
- $totalProblems++;
- $characterCount++;
- }
- next;
- }
+#######################################################
+#######################################################
- my %partData=undef;
- # Initialize part data, display skips correctly
- # Skip refers to when a student made no submissions on that
- # part/problem.
- foreach my $part (split(/\:/,$cache->{$sequence.':'.
- $problemID.
- ':parts'})) {
- $partData{$part.':tries'}=0;
- $partData{$part.':code'}=' ';
- }
+=pod
- # Looping through all the versions of each part, starting with the
- # oldest version. Basically, it gets the most recent
- # set of grade data for each part.
- for(my $Version=1; $Version<=$LatestVersion; $Version++) {
- foreach my $part (split(/\:/,$cache->{$sequence.':'.
- $problemID.
- ':parts'})) {
-
- if(!defined($cache->{$name.":$Version:$problem".
- ":resource.$part.solved"})) {
- # No grade for this submission, so skip
- next;
- }
+=item &CreateInterface()
- my $tries=0;
- my $code=' ';
+Called by &BuildStudentAssessmentPage to create the top part of the
+page which displays the chart.
- $tries = $cache->{$name.':'.$Version.':'.$problem.
- ':resource.'.$part.'.tries'};
- $partData{$part.':tries'}=($tries) ? $tries : 0;
-
- my $val = $cache->{$name.':'.$Version.':'.$problem.
- ':resource.'.$part.'.solved'};
- if ($val eq 'correct_by_student') {$code = '*';}
- elsif ($val eq 'correct_by_override') {$code = '+';}
- elsif ($val eq 'incorrect_attempted') {$code = '.';}
- elsif ($val eq 'incorrect_by_override'){$code = '-';}
- elsif ($val eq 'excused') {$code = 'x';}
- elsif ($val eq 'ungraded_attempted') {$code = '#';}
- else {$code = ' ';}
- $partData{$part.':code'}=$code;
- }
- }
+Inputs: None
- # 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
- $Str .= '';
- foreach(split(/\:/,$cache->{$sequence.':'.$problemID.
- ':parts'})) {
- if($partData{$_.':code'} eq '*') {
- $problemsCorrect++;
- if (($partData{$_.':tries'}<10) &&
- ($partData{$_.':tries'} ne '')) {
- $partData{$_.':code'}=$partData{$_.':tries'};
- }
- } elsif($partData{$_.':code'} eq '+') {
- $problemsCorrect++;
- }
+Returns: A string containing the HTML for the headers and top table for
+the chart page.
- $Str .= $partData{$_.':code'};
- $characterCount++;
+=cut
- if($partData{$_.':code'} ne 'x') {
- $totalProblems++;
- }
- }
- $Str.='';
+#######################################################
+#######################################################
+sub CreateInterface {
+ my $Str = '';
+# $Str .= &CreateLegend();
+ $Str .= '
';
+ my $only_seq_with_assessments = sub {
+ my $s=shift;
+ if ($s->{'num_assess'} < 1) {
+ return 0;
+ } else {
+ return 1;
}
-
- # 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", $problemsCorrect );
- $Str .= ''.$outputProblemsCorrect.'';
- $problemsSolved += $problemsCorrect;
- $problemsCorrect=0;
-
- $Str .= $spacing;
- }
-
- # 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", $problemsSolved );
- my $outputTotalProblems = sprintf( "%4d", $totalProblems );
- $Str .= ''.$outputProblemsSolved.
- ' / '.$outputTotalProblems.'';
-
+ };
+ $Str .= &Apache::lonstatistics::StudentDataSelect('StudentData','multiple',
+ 5,undef);
+ $Str .= '
'."\n";
return $Str;
}
+
+#######################################################
+#######################################################
+
=pod
-=item &CreateLegend()
+=item Table Output Routines
-This function returns a formatted string containing the legend for the
-chart. The legend describes the symbols used to represent grades for
-problems.
+=over 4
=cut
-sub CreateLegend {
- my $Str = "
".
- "1..9: correct by student in 1..9 tries\n".
- " *: correct by student in more than 9 tries\n".
- " +: correct by override\n".
- " -: incorrect by override\n".
- " .: incorrect attempted\n".
- " #: ungraded attempted\n".
- " : not attempted\n".
- " x: excused".
- "
";
- return $Str;
-}
+#######################################################
+#######################################################
+{
+ my $padding;
+
+#######################################################
+#######################################################
=pod
-=item &CreateColumnSelectionBox()
+=item &CreateTableHeadings()
+
+Create HTML for the columns of student data to show.
+Called by &BuildStudentAssessmentPage(). Calls
+&Apache::lonhtmlcommon::CreateHeadings().
-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.
+Inputs:
=over 4
-Input: $CacheData, $headings
+=item $cache The ubiquitous cache
+
+=item $spacing A string of spaces
-$CacheData: A pointer to a hash tied to the cached data
+=item $infoKeys Array ref to names of keys to display from the cache
+which describe students
-$headings: An array of the names of the columns for the student information.
-They are used for displaying which columns are missing.
+=item $infoHeadings Array ref to headings of columns for student info
-Output: $notThere
+=item $sequenceKeys Array ref of names of keys to use to retrieve sequence
+data from the cache
-$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.
+=item $sequenceHeadings Array ref of names of sequences used for output.
=back
+Returns: A string containing the HTML of the table headings.
+
=cut
-sub CreateColumnSelectionBox {
- my ($doNotShow)=@_;
+#######################################################
+#######################################################
+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;
+}
- my $notThere = '';
- $notThere .= '';
+=item &ChartOutputStudent($student)
- return $notThere;
-}
+Return a line of the chart for a student.
-=pod
+=cut
-=item &CreateColumnSelectors()
+#######################################################
+#######################################################
+sub ChartOutputStudent {
+ my $student = shift;
+ 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=$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 (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;
+}
-This function generates the checkboxes above the column headings. The
-column will be removed if the checkbox is unchecked.
+#######################################################
+#######################################################
-=over 4
+=pod
-Input: $CacheData, $headings
+=back
-$CacheData: A pointer to a hash tied to the cached data
+=cut
-$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
+=pod
-=cut
+=item &StudentPerformanceOnSequence()
-sub CreateColumnSelectors {
- my ($infoHeadings, $sequenceHeadings, $sequenceKeys)=@_;
+Inputs:
- my $present = '';
- for(my $index=0; $index<(scalar @$infoHeadings); $index++) {
- $present .= '
'."\n";
- }
+=item $student
- return $present;
-}
+=item $studentdata Hash ref to all student data
-#---- END Student Assessment Web Page ----------------------------------------
+=item $seq Hash ref, the sequence we are working on
-#---- Student Assessment Worker Functions ------------------------------------
+=item $links if defined we will output links to each resource.
-sub FindSelectedStudent {
- my($cache, $selectedName, $students)=@_;
+=back
- if($selectedName eq 'All Students' ||
- $selectedName eq 'No Student Selected') {
- return $selectedName;
- }
+=cut
- for(my $index=0; $index<(scalar @$students); $index++) {
- my $fullname = $cache->{$students->[$index].':fullname'};
- if($fullname eq $selectedName) {
- if($cache->{'StudentAssessmentMove'} eq 'next') {
- if($index == ((scalar @$students) - 1)) {
- $selectedName = $students->[0];
- return $selectedName;
+#######################################################
+#######################################################
+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 {
- $selectedName = $students->[$index+1];
- return $selectedName;
+ $symbol = ' ';
}
- } elsif($cache->{'StudentAssessmentMove'} eq 'previous') {
- if($index == 0) {
- $selectedName = $students->[-1];
- return $selectedName;
+ } else {
+ # Unsolved. Did they try?
+ if (exists($resource_data->{'resource.'.$partnum.'.tries'})){
+ $symbol = '.';
} else {
- $selectedName = $students->[$index-1];
- return $selectedName;
+ $symbol = ' ';
}
- } else {
- $selectedName = $students->[$index];
- return $selectedName;
}
- last;
+ #
+ $output_width++;
+ if (defined($links) && $symbol ne ' ') {
+ $symbol = ''.$symbol.'';
+ }
+ $value .= $symbol;
}
+ $Str .= $value;
}
-
- return 'No Student Selected';
+ # 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);
}
-=pod
-
-=item &ShouldShowColumn()
-
-Determine if a specified column should be shown on the chart.
+#######################################################
+#######################################################
+sub StudentAverageTotal {
+ my ($cache, $students, $sequenceKeys)=@_;
+ my $Str = "\nSummary Tables:\n";
+ my %Correct = ();
+ my $ProblemsSolved = 0;
+ my $TotalProblems = 0;
+ my $StudentCount = 0;
+
+ foreach my $name (@$students) {
+ $StudentCount++;
+ foreach my $sequence (@$sequenceKeys) {
+ $Correct{$sequence} +=
+ $cache->{$name.':'.$sequence.':problemsCorrect'};
+ }
+ $ProblemsSolved += $cache->{$name.':problemsSolved'};
+ $TotalProblems += $cache->{$name.':totalProblems'};
+ }
+ if ($StudentCount) {
+ $ProblemsSolved = sprintf( "%.2f",
+ $ProblemsSolved/$StudentCount);
+ $TotalProblems /= $StudentCount;
+ } else {
+ $ProblemsSolved = 0;
+ $TotalProblems = 0;
+ }
+
+ $Str .= '
'."\n";
+ $Str .= '
Students Count
'.
+ $StudentCount.'
'."\n";
+ $Str .= '
Total Problems
'.
+ $TotalProblems.'
'."\n";
+ $Str .= '
Average Correct
'.
+ $ProblemsSolved.'
'."\n";
+ $Str .= '
'."\n";
+
+ $Str .= '
'."\n";
+ $Str .= '
Title
Total Problems
'.
+ '
Average Correct
'."\n";
+ foreach my $S(@$sequenceKeys) {
+ my $title=$cache->{$S.':title'};
+ #$Str .= $cache->{$S.':problems'};
+ #my @problems=split(':', $cache->{$S.':problems'});
+ #my $pCount=scalar @problems;
+ my $pCount=MaxSeqPr($cache,@$students[0],$S);
+ my $crr;
+ if ($StudentCount) {
+ $crr=sprintf( "%.2f", $Correct{$S}/$StudentCount );
+ } else {
+ $crr="0.00";
+ }
+ $Str .= '
'.$title.
+ '
'.$pCount.
+ '
'.$crr.
+ '
'."\n";
+ }
-=over 4
+ $Str .= '
'."\n";
-Input: $cache, $test
+ return $Str;
+}
-$cache: A pointer to the hash tied to the cached data
+#######################################################
+#######################################################
-$test: The form name of the column (heading.$headingIndex) or
-(sequence.$sequenceIndex)
+=pod
-Output: 0 (false), 1 (true)
+=item &CreateLegend()
-=back
+This function returns a formatted string containing the legend for the
+chart. The legend describes the symbols used to represent grades for
+problems.
=cut
-sub ShouldShowColumns {
- my ($cache,$headings,$cacheKey)=@_;
+#######################################################
+#######################################################
+sub CreateLegend {
+ my $Str = "
".
+ " 1 correct by student in 1 try\n".
+ " 7 correct by student in 7 tries\n".
+ " * correct by student in more than 9 tries\n".
+ " + correct by hand grading or override\n".
+ " - incorrect by override\n".
+ " . incorrect attempted\n".
+ " # ungraded attempted\n".
+ " not attempted (blank field)\n".
+ " x excused".
+ "