--- loncom/interface/statistics/lonproblemstatistics.pm 2002/08/14 13:13:37 1.24
+++ loncom/interface/statistics/lonproblemstatistics.pm 2002/11/26 16:59:03 1.37
@@ -1,7 +1,7 @@
# The LearningOnline Network with CAPA
# (Publication Handler
#
-# $Id: lonproblemstatistics.pm,v 1.24 2002/08/14 13:13:37 stredwic Exp $
+# $Id: lonproblemstatistics.pm,v 1.37 2002/11/26 16:59:03 minaeibi Exp $
#
# Copyright Michigan State University Board of Trustees
#
@@ -30,12 +30,12 @@
# 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,7/25,7/29,8/5 Behrouz Minaei
+# 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;
+package Apache::lonproblemstatistics;
use strict;
use Apache::lonnet();
@@ -43,101 +43,175 @@ use Apache::lonhtmlcommon;
use Apache::loncoursedata;
use GDBM_File;
-my $jr;
-sub BuildProblemStatisticsPage {
+sub InitializeProblemStatistics {
my ($cacheDB, $students, $courseID, $c, $r)=@_;
my %cache;
- $jr = $r;
-
unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER(),0640)) {
- $r->print('Unable to tie database.');
- return;
+ $r->print('Unable to tie database1.');
+ return ('ERROR', undef);
}
- my $Ptr = '';
- $Ptr .= '
';
- $Ptr .= 'Select Map | '."\n";
- $Ptr .= '';
- $Ptr .= &Apache::lonhtmlcommon::MapOptions(\%cache, 'ProblemStatistics',
- 'Statistics');
- $Ptr .= ' |
'."\n";
- $Ptr .= 'Sorting Type: | '."\n";
- $Ptr .= ''."\n";
- $Ptr .= &Apache::lonhtmlcommon::AscendOrderOptions(
- $cache{'ProblemStatisticsAscend'},
- 'ProblemStatistics',
- 'Statistics');
- $Ptr .= ' |
'."\n";
- $Ptr .= &ProblemStatisticsButtons($cache{'DisplayFormat'},
- $cache{'DisplayLegend'});
- $Ptr .= '
';
- if($cache{'DisplayLegend'} eq 'Show Legend') {
- $Ptr .= &ProblemStatisticsLegend();
+ # 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);
+ }
}
- $r->print($Ptr);
- $r->rflush();
- my @Header = ("Homework Sets Order","#Stdnts","Tries","Mod",
- "Mean","#YES","#yes","%Wrng","DoDiff",
- "S.D.","Skew.","D.F.1st","D.F.2nd","Disc.");
- my $color=&setbgcolor(0);
+ 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;
+ }
-# my %Discuss=&Apache::loncoursedata::LoadDiscussion($courseID);
-# my ($upper, $lower) = &Discriminant(\%discriminant,$r);
- if(!defined($cache{'StatisticsCached'})) {
- if(defined($cache{'StatisticsCached'})) {
- untie(%cache);
- unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_WRCREAT(),0640)) {
- $r->print('Unable to tie database.');
- return;
- }
- my @statkeys = split(':::', $cache{'StatisticsKeys'});
- delete $cache{'StatisticsKeys'};
- delete $cache{'StatisticsCached'};
- foreach(@statkeys) {
- delete $cache{$_};
- }
+ 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{$_};
}
- untie(%cache);
+ }
+
+ untie(%cache);
+ if($isNotCached) {
&Apache::loncoursedata::DownloadStudentCourseDataSeparate($students,
'true',
$cacheDB,
- 'true',
+ 'true',
'true',
$courseID,
$r, $c);
- if($c->aborted()) { return; }
+ }
+ if($c->aborted()) { return ('ERROR', undef); }
- unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER(),0640)) {
- $r->print('Unable to tie database.');
- return;
- }
- my ($problemData) = &ExtractStudentData(\%cache, $students);
- &CalculateStatistics($problemData, \%cache);
- untie(%cache);
-
- unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_WRCREAT(),0640)) {
- $r->print('Unable to tie database.');
- return;
- }
+ 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);
+
+ 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';
- untie(%cache);
+ $cache{'StatisticsLastStatus'} = $cache{'Status'};
+ $cache{'StatisticsWhichStudents'} = $whichStudents;
+ }
+ untie(%cache);
- unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER(),0640)) {
- $r->print('Unable to tie database.');
- return;
- }
+ unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER(),0640)) {
+ $r->print('Unable to tie database.5');
+ return ('ERROR', undef);
}
- my $orderedProblems = &SortProblems(\%cache,
+
+ my $orderedProblems = &SortProblems(\%cache,
$cache{'ProblemStatisticsSort'},
+ $cache{'SortProblems'},
$cache{'ProblemStatisticsAscend'});
- &BuildStatisticsTable(\%cache, $cache{'DisplayFormat'}, $orderedProblems,
+ untie(%cache);
+
+ return ('OK', $orderedProblems);
+}
+
+sub BuildProblemStatisticsPage {
+ my ($cacheDB, $students, $courseID, $c, $r)=@_;
+
+ 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;
+
+ unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER(),0640)) {
+ $r->print('Unable to tie database.6');
+ return;
+ }
+ my $Ptr = '';
+ $Ptr .= '';
+ $Ptr .= 'Select Map | '."\n";
+ $Ptr .= '';
+ $Ptr .= &Apache::lonhtmlcommon::MapOptions(\%cache, 'Statistics',
+ 'Statistics');
+ $Ptr .= ' |
'."\n";
+ $Ptr .= 'Sorting Type: | '."\n";
+ $Ptr .= ''."\n";
+ $Ptr .= &Apache::lonhtmlcommon::AscendOrderOptions(
+ $cache{'ProblemStatisticsAscend'},
+ 'ProblemStatistics',
+ 'Statistics');
+ $Ptr .= ' |
'."\n";
+ $Ptr .= 'Select Sections';
+ $Ptr .= ' | '."\n";
+ $Ptr .= ''."\n";
+ my @sections = split(':',$cache{'sectionList'});
+ my @sectionsSelected = split(':',$cache{'sectionsSelected'});
+ $Ptr .= &Apache::lonhtmlcommon::MultipleSectionSelect(\@sections,
+ \@sectionsSelected,
+ 'Statistics');
+ $Ptr .= ' |
'."\n";
+ $Ptr .= &ProblemStatisticsButtons($cache{'DisplayFormat'},
+ $cache{'DisplayLegend'},
+ $cache{'SortProblems'});
+ $Ptr .= '
';
+ if($cache{'DisplayLegend'} eq 'Show Legend') {
+ $Ptr .= &ProblemStatisticsLegend();
+ }
+ $r->print($Ptr);
+ $r->rflush();
+ untie(%cache);
+
+ my ($result, $orderedProblems) =
+ &InitializeProblemStatistics($cacheDB, $students, $courseID, $c, $r);
+ if($result ne 'OK') {
+ return;
+ }
+
+ unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER(),0640)) {
+ $r->print('Unable to tie database.6');
+ return;
+ }
+ &BuildStatisticsTable(\%cache, $cache{'DisplayFormat'},
+ $cache{'SortProblems'}, $orderedProblems,
\@Header, $r, $color);
untie(%cache);
@@ -145,17 +219,52 @@ sub BuildProblemStatisticsPage {
}
sub BuildGraphicChart {
- my ($graph,$cacheDB,$courseDescription,$r)=@_;
+ my ($graph,$cacheDB,$courseDescription,$students,$courseID,$r,$c)=@_;
my %cache;
- my $max = 0;
-
+ my $max;
unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER(),0640)) {
- return 'Unable to tie database.';
+ return 'Unable to tie database.7';
+ }
+
+ my $title = '';
+ if($graph eq 'DoDiffGraph') {
+ $title = 'Degree-of-Difficulty';
+ } else {
+ $title = 'Wrong-Percentage';
+ }
+
+ my $currentSequence = -1;
+ my $sortProblems = 'Sort Within Sequence';
+
+ my ($result, $orderedProblems) =
+ &InitializeProblemStatistics($cacheDB, $students, $courseID, $c, $r);
+ if($result ne 'OK') {
+ return;
}
-
- my @problems = split(':::', $cache{'problemList'});
+
my @values = ();
- foreach (@problems) {
+
+ 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('
'.$cache{$sequence.':title'}.''."\n");
+ }
+
+ $currentSequence = $sequence;
+ @values = ();
+ $max=0;
+ }
my $data = 0;
if($graph eq 'DoDiffGraph') {
$data = sprintf("%.2f", $cache{$_.':degreeOfDifficulty'}),
@@ -169,24 +278,35 @@ sub BuildGraphicChart {
}
untie(%cache);
- my $sendValues = join(',', @values);
-# my $sendCount = $#values;
- my $sendCount = scalar(@values);
+ &DrawGraph(\@values,$courseDescription,$title,$max,$r);
- my $title = '';
- if($graph eq 'DoDiffGraph') {
- $title = 'Degree-of-Difficulty';
- } else {
- $title = 'Wrong-Percentage';
- }
- my @GData = ($courseDescription, 'Problems', $title, $max, $sendCount,
- $sendValues);
+ return;
+}
- $r->print(''."\n");
- $r->print('');
- $r->print(''."\n");
+ $r->print('
'."\n");
+ $r->print('');
+# $r->print('