--- loncom/interface/statistics/lonproblemstatistics.pm 2002/08/14 16:18:55 1.25
+++ loncom/interface/statistics/lonproblemstatistics.pm 2003/02/25 20:47:47 1.40
@@ -1,7 +1,6 @@
# The LearningOnline Network with CAPA
-# (Publication Handler
#
-# $Id: lonproblemstatistics.pm,v 1.25 2002/08/14 16:18:55 stredwic Exp $
+# $Id: lonproblemstatistics.pm,v 1.40 2003/02/25 20:47:47 matthew Exp $
#
# Copyright Michigan State University Board of Trustees
#
@@ -30,12 +29,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,17 +42,14 @@ 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);
}
# Remove students who don't have the proper section.
@@ -80,84 +76,58 @@ sub BuildProblemStatisticsPage {
}
}
- 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 .= 'Select Sections';
- $Ptr .= ' | '."\n";
- $Ptr .= ''."\n";
- my @sections = split(':',$cache{'sectionList'});
- $Ptr .= &Apache::lonhtmlcommon::MultipleSectionSelect(\@sections,
- \@sectionsSelected,
- 'Statistics');
- $Ptr .= ' |
'."\n";
- $Ptr .= &ProblemStatisticsButtons($cache{'DisplayFormat'},
- $cache{'DisplayLegend'});
- $Ptr .= '
';
- if($cache{'DisplayLegend'} eq 'Show Legend') {
- $Ptr .= &ProblemStatisticsLegend();
- }
- $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 %Discuss=&Apache::loncoursedata::LoadDiscussion($courseID);
+ my $isNotCached = 0;
my $lastStatus = (defined($cache{'StatisticsLastStatus'})) ?
$cache{'StatisticsLastStatus'} : 'Nothing';
my $whichStudents = join(':::',sort(@$students));
- if(!defined($cache{'StatisticsCached'}) ||
+ if(!defined($cache{'StatisticsCached'}) ||
$lastStatus ne $cache{'Status'} ||
$whichStudents ne $cache{'StatisticsWhichStudents'}) {
- 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{$_};
- }
+ $isNotCached = 1;
+ }
+
+ 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->{$_};
}
@@ -165,18 +135,80 @@ sub BuildProblemStatisticsPage {
$cache{'StatisticsCached'} = 'true';
$cache{'StatisticsLastStatus'} = $cache{'Status'};
$cache{'StatisticsWhichStudents'} = $whichStudents;
- untie(%cache);
+ }
+ 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::lonstatistics::SectionSelect('Section','multiple',5);
+ $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);
@@ -184,17 +216,53 @@ 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.';
+ 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) {
+
+ unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER(),0640)) {
+ return 'Unable to tie database.7';
+ }
+
+ 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'}),
@@ -208,23 +276,35 @@ sub BuildGraphicChart {
}
untie(%cache);
- my $sendValues = join(',', @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('