--- loncom/interface/statistics/lonproblemstatistics.pm 2002/07/29 21:23:22 1.5 +++ loncom/interface/statistics/lonproblemstatistics.pm 2002/07/30 22:09:01 1.7 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # (Publication Handler # -# $Id: lonproblemstatistics.pm,v 1.5 2002/07/29 21:23:22 minaeibi Exp $ +# $Id: lonproblemstatistics.pm,v 1.7 2002/07/30 22:09:01 minaeibi Exp $ # # Copyright Michigan State University Board of Trustees # @@ -31,7 +31,7 @@ # 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 Behrouz Minaei +# 5/12,5/14,5/15,5/19,5/26,7/16,7/25,7/29 Behrouz Minaei # ### @@ -80,24 +80,23 @@ sub BuildProblemStatisticsPage { unless(tie(%cache,'GDBM_File',$cacheDB,&GDBM_READER,0640)) { return 'Unable to tie database.'; } - my $discriminantFactor; + my %discriminant=(); my @list=(); my %Discuss=&Apache::loncoursedata::LoadDiscussion($courseID); my $index=0; foreach (@$students) { $index++; #&Apache::lonstatistics::Update_PrgWin(++$index,$r); - $discriminantFactor = &ExtractStudentData(\%cache, $_, \@list, - \%Discuss, $r); + &ExtractStudentData(\%cache, $_, \@list,\%Discuss, $r, \%discriminant); } #&Apache::lonstatistics::Close_PrgWin($r); - my ($upper, $lower) = &Discriminant($discriminantFactor); + my ($upper, $lower) = &Discriminant(\%discriminant,$r); my %Header = (0,"Homework Sets Order",1,"#Stdnts",2,"Tries",3,"Mod", 4,"Mean",5,"#YES",6,"#yes",7,"%Wrng",8,"DoDiff", 9,"S.D.",10,"Skew.",11,"D.F.1st",12,"D.F.2nd", 13, "Disc."); my $color=&setbgcolor(0); - my %GraphDat= &BuildStatisticsTable(\%cache, $discriminantFactor, \@list, + my %GraphDat= &BuildStatisticsTable(\%cache, $upper, $lower, \@list, \%Header, $students, $r, $color); untie(%cache); } @@ -146,9 +145,7 @@ sub CloseTable { # ------ Dump the Student's DB file and handling the data for statistics table sub ExtractStudentData { - my ($cache,$name,$list,$Discuss,$r)=@_; - my %discriminantFactor; - + my ($cache,$name,$list,$Discuss,$r,$discriminant)=@_; my $totalTries = 0; my $totalAwarded = 0; my $spent=0; @@ -160,6 +157,7 @@ sub ExtractStudentData { my $TotFirst = 0; my $TimeTot = 0; my $Discussed=0; + my $discrim=''; my $tempSequenceOrder=100; #$Apache::lonxml::debug=1; @@ -176,7 +174,7 @@ sub ExtractStudentData { # next; # } - my $Dis = ''; + #$discrim .= '&'; foreach my $problemID (split(':', $cache->{$sequence.':problems'})) { # $r->print("
problems=$cache->{$sequence.':problems'}"); my $problem = $cache->{$problemID.':problem'}; @@ -195,6 +193,7 @@ sub ExtractStudentData { #} my %partData=undef; + $partData{'count'}=0; # Initialize part data, display skips correctly # Skip refers to when a student made no submissions on that # part/problem. @@ -202,6 +201,7 @@ sub ExtractStudentData { $problemID. ':parts'})) { $tempProblemOrder++; + $partData{'count'}++; # $r->print("
parts=$cache->{$sequence.':'.$problemID.':parts'}"); $partData{$part.':order'}=$tempProblemOrder; $partData{$part.':tries'}=0; @@ -228,13 +228,12 @@ sub ExtractStudentData { $Discussed=0; my $code='-'; - $awarded = $cache->{$name. - "$Version:$problem:resource.". + $awarded = $cache->{"$name:$Version:$problem:resource.". "$part.awarded"}; $partData{$part.':awarded'} = ($awarded) ? $awarded : 0; $totalAwarded += $awarded; - $tries = $cache->{$name.":$Version:$problem". + $tries = $cache->{"$name:$Version:$problem". ":resource.$part.tries"}; $partData{$part.':tries'} = ($tries) ? $tries : 0; $partData{$part.':wrong'} = $partData{$part.':tries'}; @@ -261,10 +260,10 @@ sub ExtractStudentData { # Loop through all the parts for the current problem in the # correct order and prepare the output + my $partCounter=0; foreach (split(/\:/,$cache->{$sequence.':'.$problemID. ':parts'})) { - #$r->print("
".$cache->{$sequence.':title'}); - + $partCounter++; my $Yes = 0; if($partData{$_.':code'} eq 'C' || $partData{$_.':code'} eq 'O') { @@ -273,10 +272,12 @@ sub ExtractStudentData { my $pOrder=$partData{$_.':order'}; my $ptr = $tempSequenceOrder.':'.$pOrder.':'.$problemID; - if($_ > 1) { + if($partData{'count'} > 1) { $ptr .= "*(part $_)"; - $Dis .= '&'; } + #if($partCounter > 1) { + $discrim .= '&'; + #} my ($pr_no,$dod)=split('&',$ptr); # my $DoDiff=$DoDiff->{$dod}; @@ -296,7 +297,7 @@ sub ExtractStudentData { $Discussed=1; } my $time = $cache->{"$name:$LatestVersion:$problem:timestamp"}; - $Dis .= $tempProblemOrder.'='.$DisF.'+'.$Yes; + $discrim .= $tempSequenceOrder.'@'.$pOrder.'='.$DisF.'+'.$Yes; $ptr .= '&'.$partData{$_.':tries'}. '&'.$partData{$_.':wrong'}. '&'.$partData{$_.':code'}; @@ -326,21 +327,18 @@ sub ExtractStudentData { $tempProblemOrder++; } } - my $pstr; - if($totalTries) { - my $DisFac = ($totalAwarded/$totalTries); - my $DisFactor = sprintf( "%.4f", $DisFac ); - my $TS = sprintf( "%.2f", $spent ); - my $TS_yes = sprintf( "%.2f", $spent_yes ); - # $DiscFac{$DisFactor}=$Dis; - $pstr=$DisFactor.':'.$name.':'.$ProbTot.':'.$TotalOpend.':'. - $totalTries.':'.$ProbSolved.':'.$TotFirst.':'. - $TS_yes.':'.$TS.':'.$TotDiscuss; - $discriminantFactor{$pstr}=$Dis; - } } - - return (\%discriminantFactor); + my $pstr; + if($totalTries) { + my $DisFac = ($totalAwarded/$totalTries); + my $DisFactor = sprintf( "%.4f", $DisFac ); + my $TS = sprintf( "%.2f", $spent ); + my $TS_yes = sprintf( "%.2f", $spent_yes ); + $pstr=$DisFactor.':'.$name.':'.$ProbTot.':'.$TotalOpend.':'. + $totalTries.':'.$ProbSolved.':'.$TotFirst.':'. + $TS_yes.':'.$TS.':'.$TotDiscuss; + (%$discriminant)->{$pstr}=$discrim; + } } @@ -359,16 +357,10 @@ sub MySort { sub BuildStatisticsTable { - my ($cache,$discriminantFactor,$list,$headings,$students,$r,$color)=@_; - + my ($cache,$upper,$lower,$list,$headings,$students,$r,$color)=@_; my $NoElements = scalar @$list; my @List=sort(@$list); -# for (my $c=0;$c<$NoElements;$c++) { -## $r->print('
'.$c.')'.$List[$c]); -# $r->print("
$List[$c]"); -# } - #6666666 # my $file="/home/httpd/perl/tmp/183d.txt"; # open(OUT, ">$file"); @@ -395,10 +387,8 @@ sub BuildStatisticsTable { &CreateProblemStatisticsTableHeading(1,$cache->{$sequence.':source'}, $cache->{$sequence.':title'}, $headings,$r); - my ($tar,$Tries,$Wrongs,$Code,$Disc)=split(/\&/, $List[$cIdx]); - my ($SqOrd,$PrOrd,$Prob)=split(/\:/,$tar); $sequence+=100; while ($SqOrd==$sequence && $cIdx<$NoElements) { @@ -427,15 +417,8 @@ sub BuildStatisticsTable { ($tar,$Tries,$Wrongs,$Code,$Disc)=split(/\&/, $List[$cIdx]); ($SqOrd,$PrOrd,$Prob)=split(/\:/,$tar); - -#$r->print("
$cIdx ... $SqOrd, $PrOrd, $Tries, $Wrongs, $Code, $Disc"); -#$r->rflush(); - } -#$r->print("
$cIdx ... $SqOrd , $Temp , $TotalTries"); -#$r->rflush(); - $p_count++; my $Dummy; ($ResId,$Dummy)=split(/\*/,$Temp); @@ -498,8 +481,9 @@ sub BuildStatisticsTable { } #--------------------- Compute the Discrimination Factors - my ($Up1,$Up2)=split(/\:/,':');#jason$DisUp->{$lpr[$i]}); - my ($Lw1,$Lw2)=split(/\:/,':');#jason$DisLow->{$lpr[$i]}); + my ($Up1,$Up2)=split(/\:/,$upper->{$sequence.'@'.$pOrd}); + my ($Lw1,$Lw2)=split(/\:/,$lower->{$sequence.'@'.$pOrd}); + my $Dis1 = $Up1 - $Lw1; my $Dis2 = $Up2 - $Lw2; my $_D1 = sprintf("%.2f", $Dis1); @@ -736,42 +720,42 @@ sub ProblemStatisticsLegend { my $Ptr = ''; $Ptr = ''; $Ptr .= ''; + $Ptr .= '#Stdnts'; $Ptr .= ''; + $Ptr .= 'Tries'; $Ptr .= ''; + $Ptr .= 'Mod'; $Ptr .= ''; + $Ptr .= 'Mean'; $Ptr .= ''; + $Ptr .= '#YES'; $Ptr .= ''; + $Ptr .= '#yes'; $Ptr .= ''; + $Ptr .= '%Wrng'; $Ptr .= ''; + $Ptr .= 'DoDiff'; $Ptr .= ''; + $Ptr .= 'S.D.'; $Ptr .= ''; + $Ptr .= 'Skew.'; $Ptr .= ''; + $Ptr .= 'Dis.F.'; $Ptr .= '
'; - $Ptr .= '#Stdnts:Total Number of Students opened the problem.'; $Ptr .= '
'; - $Ptr .= 'Tries:Total Number of Tries for solving the problem.'; $Ptr .= '
'; - $Ptr .= 'Mod:Maximunm Number of Tries for solving the problem.'; $Ptr .= '
'; - $Ptr .= 'Mean:Average Number of the tries. [ Tries / #Stdnts ]'; $Ptr .= '
'; - $Ptr .= '#YES:Number of students solved the problem correctly.'; $Ptr .= '
'; - $Ptr .= '#yes:Number of students solved the problem by override.'; $Ptr .= '
'; - $Ptr .= '%Wrng:Percentage of students tried to solve the problem '; $Ptr .= 'but still incorrect. [ 100*((#Stdnts-(#YES+#yes))/#Stdnts) ]'; $Ptr .= '
'; - $Ptr .= 'DoDiff:Degree of Difficulty of the problem. '; $Ptr .= '[ 1 - ((#YES+#yes) / Tries) ]'; $Ptr .= '
'; - $Ptr .= 'S.D.:Standard Deviation of the tries. '; $Ptr .= '[ sqrt(sum((Xi - Mean)^2)) / (#Stdnts-1) '; $Ptr .= 'where Xi denotes every student\'s tries ]'; $Ptr .= '
'; - $Ptr .= 'Skew.:Skewness of the students tries.'; $Ptr .= '[(sqrt( sum((Xi - Mean)^3) / #Stdnts)) / (S.D.^3)]'; $Ptr .= '
'; - $Ptr .= 'Dis.F.:Discrimination Factor: A Standard for evaluating the '; $Ptr .= 'problem according to a Criterion
'; $Ptr .= '[Applied Criterion in %27 Upper Students - '; @@ -790,7 +774,7 @@ sub ProblemStatisticsLegend { #------- Processing upperlist and lowerlist according to each problem sub ProcessDiscriminant { - my ($List) = @_; + my ($List,$r) = @_; my @sortedList = sort (@$List); my $Count = scalar @sortedList; my $Problem; @@ -803,6 +787,7 @@ sub ProcessDiscriminant { my $nStudent=0; my %Proc=undef; while ($nIndex<$Count) { +# $r->print("
$nIndex) $sortedList[$nIndex]"); ($Problem,$tmp)=split(/\=/,$sortedList[$nIndex]); @Dis=split(/\+/,$tmp); my $Temp = $Problem; @@ -814,9 +799,8 @@ sub ProcessDiscriminant { ($Problem,$tmp)=split(/\=/,$sortedList[$nIndex]); @Dis=split(/\+/,$tmp); } while ( $Problem eq $Temp && $nIndex < $Count ); -# $Proc{$Temp}=($Sum1/$nStudent).':'.$nStudent; $Proc{$Temp}=($Sum1/$nStudent).':'.($Sum2/$nStudent); -# $r->print("$nIndex) $Temp --> ($nStudent) $Proc{$Temp}
"); +# $r->print("
$nIndex) $Temp --> ($nStudent) $Proc{$Temp}"); $Sum1=0; $Sum2=0; $nStudent=0; @@ -827,8 +811,11 @@ sub ProcessDiscriminant { #------- Creating Discimination factor sub Discriminant { - my ($discriminantFactor)=@_; - my @discriminantKeys=keys(%$discriminantFactor); + my ($discriminant,$r)=@_; +#$Apache::lonxml::debug=1; +#&Apache::lonhomework::showhash(%$discriminant); +#$Apache::lonxml::debug=0; + my @discriminantKeys=keys(%$discriminant); my $Count = scalar @discriminantKeys; my $UpCnt = int(0.27*$Count); @@ -842,8 +829,8 @@ sub Discriminant { $Count++; if($low < $UpCnt || $Count > $up) { $low++; - my $str=$discriminantFactor->{$key}; - foreach(split(/\:/,$str)){ + my $str=$discriminant->{$key}; + foreach(split(/\&/,$str)){ if($_) { if($low<$UpCnt) { push(@LowList,$_); } else { push(@UpList,$_); } @@ -851,8 +838,8 @@ sub Discriminant { } } } - my %DisUp = &ProcessDiscriminant(\@UpList); - my %DisLow = &ProcessDiscriminant(\@LowList); + my %DisUp = &ProcessDiscriminant(\@UpList,$r); + my %DisLow = &ProcessDiscriminant(\@LowList,$r); return (\%DisUp, \%DisLow); }