--- loncom/interface/statistics/lonproblemanalysis.pm 2003/10/13 21:12:56 1.34 +++ loncom/interface/statistics/lonproblemanalysis.pm 2003/10/16 15:24:49 1.41 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # -# $Id: lonproblemanalysis.pm,v 1.34 2003/10/13 21:12:56 matthew Exp $ +# $Id: lonproblemanalysis.pm,v 1.41 2003/10/16 15:24:49 matthew Exp $ # # Copyright Michigan State University Board of Trustees # @@ -35,6 +35,12 @@ use Apache::lonhtmlcommon(); use Apache::loncoursedata(); use Apache::lonstatistics; use Apache::lonlocal; +use HTML::Entities(); + +my $plotcolors = ['#33ff00', + '#0033cc', '#990000', '#aaaa66', '#663399', '#ff9933', + '#66ccff', '#ff9999', '#cccc33', '#660000', '#33cc66', + ]; sub BuildProblemAnalysisPage { my ($r,$c)=@_; @@ -43,7 +49,8 @@ sub BuildProblemAnalysisPage { # my @Students = @Apache::lonstatistics::Students; # - if (exists($ENV{'form.updatecaches'}) || + if (exists($ENV{'form.ClearCache'}) || + exists($ENV{'form.updatecaches'}) || (exists($ENV{'form.firstanalysis'}) && $ENV{'form.firstanalysis'} ne 'no')) { &Apache::lonstatistics::Gather_Full_Student_Data($r); @@ -53,6 +60,7 @@ sub BuildProblemAnalysisPage { } else { $r->print(''); } + $r->rflush(); if (exists($ENV{'form.problemchoice'}) && ! exists($ENV{'form.SelectAnother'})) { $r->print(''."\n"; - $table .= ''.&mt('Foil Number').''. - ''.&mt('Foil Text')."\n"; - my $index = 1; - foreach my $foilid (@Foils) { - $table .= ''.$index.''. - ''.$Row_Label{$foilid}."\n"; + if (@Concepts > 1) { + $table .= ''. + ''.&mt('Concept Number').''. + ''.&mt('Concept').''. + ''.&mt('Foil Number').''. + ''.&mt('Foil Name').''. + ''.&mt('Foil Text').''. + ''.&mt('Correct Value').''. + "\n"; + } else { + $table .= ''. + ''.&mt('Foil Number').''. + ''.&mt('Foil Name').''. + ''.&mt('Foil Text').''. + ''.&mt('Correct Value').''. + "\n"; + } + my $conceptindex = 1; + my $foilindex = 1; + foreach my $concept (@Concepts) { + my @FoilsInConcept = @{$concept->{'foils'}}; + my $firstfoil = shift(@FoilsInConcept); + if (@Concepts > 1) { + $table .= ''. + ''.$conceptindex.''. + ''.$concept->{'name'}.''. + ''.$foilindex++.''. + ''.$Foildata{$firstfoil}->{'name'}.''. + ''.$Foildata{$firstfoil}->{'text'}.''. + ''.$Foildata{$firstfoil}->{'value'}.''. + "\n"; + } else { + $table .= ''. + ''.$foilindex++.''. + ''.$Foildata{$firstfoil}->{'name'}.''. + ''.$Foildata{$firstfoil}->{'text'}.''. + ''.$Foildata{$firstfoil}->{'value'}.''. + "\n"; + } + foreach my $foilid (@FoilsInConcept) { + if (@Concepts > 1) { + $table .= ''. + ''. + ''. + ''.$foilindex.''. + ''.$Foildata{$foilid}->{'name'}.''. + ''.$Foildata{$foilid}->{'text'}.''. + ''.$Foildata{$foilid}->{'value'}.''. + "\n"; + } else { + $table .= ''. + ''.$foilindex.''. + ''.$Foildata{$foilid}->{'name'}.''. + ''.$Foildata{$foilid}->{'text'}.''. + ''.$Foildata{$foilid}->{'value'}.''. + "\n"; + } + } continue { + $foilindex++; + } } continue { - $index++; + $conceptindex++; + } + $table .= "\n"; + # + # Build option index with color stuff + return ($table,\@Foils,\@Concepts); +} + +sub build_option_index { + my ($ORdata)= @_; + my $table = "\n"; + my $optionindex = 0; + my @Rows; + foreach my $option (&mt('correct option chosen'),@{$ORdata->{'Options'}}) { + push (@Rows, + ''. + ''. + ''. + "\n"); } + $table .= join('',reverse(@Rows)); $table .= "
'. + (' 'x4).''.$option.'
\n"; - return ($table,@Foils); } ######################################################### @@ -175,83 +290,113 @@ sub tries_analysis { my ($PerformanceData,$ORdata) = @_; my $mintries = 1; my $maxtries = $ENV{'form.NumPlots'}; + my ($table,$Foils,$Concepts) = &build_foil_index($ORdata); + if ((@$Concepts < 2) && ($ENV{'form.AnalyzeAs'} ne 'Foils')) { + $table = '

'. + &mt('Not enough data for concept analysis. '. + 'Performing Foil Analysis'). + '

'.$table; + $ENV{'form.AnalyzeAs'} = 'Foils'; + } my %ResponseData = &analyze_option_data_by_tries($PerformanceData, - $mintries,$maxtries); - my ($table,@Foils) = &build_foil_index($ORdata); + $mintries,$maxtries); # # Compute the data neccessary to make the plots - my @PlotData; - my @CumulativePlotData; - foreach my $foilid (@Foils) { - for (my $i=$mintries;$i<=$maxtries;$i++) { - # - # Gather the per-attempt data - push (@{$PlotData[$i]->{'good'}}, - $ResponseData{$foilid}->[$i]->{'percent_corr'}); - push (@{$PlotData[$i]->{'bad'}}, - 100-$ResponseData{$foilid}->[$i]->{'percent_corr'}); - # - # Someday we may need the cumulative data and I think - # this is a neat way of computing it as we go along. - push (@{$CumulativePlotData[$i]->{'good'}}, - $CumulativePlotData[-1]->{'good'}+ - $ResponseData{$foilid}->[$i]->{'correct'}); - push (@{$CumulativePlotData[$i]->{'bad'}}, - $CumulativePlotData[-1]->{'bad'}+ - $ResponseData{$foilid}->[$i]->{'incorrect'}); + my @PlotData; # Array which holds the data for each plot + # @{$PlotData[$try]->{'datasetname'}} holds the data for + # try $try with respect to 'datasetname'. The array is + # filled either with per-foil or per-concept data. + my ($extrakey,$xlabel,$ylabel); + if ($ENV{'form.AnalyzeAs'} eq 'Foils') { + $extrakey = &build_option_index($ORdata); + $xlabel = 'Foil Number'; + $ylabel = 'Percent Choosing Option'; + foreach my $foilid (@$Foils) { + for (my $i=$mintries;$i<=$maxtries;$i++) { + foreach my $option ('_correct',@{$ORdata->{'Options'}}) { + push(@{$PlotData[$i]->{'_total'}}, + $ResponseData{$foilid}->[$i]->{'_total'}); + if ($ResponseData{$foilid}->[$i]->{'_total'} == 0) { + push (@{$PlotData[$i]->{$option}},0); + } else { + push (@{$PlotData[$i]->{$option}}, + 100 * $ResponseData{$foilid}->[$i]->{$option} / + $ResponseData{$foilid}->[$i]->{'_total'}); + } + } + } } - } + } else { + # Concept analysis + # + # Note: we do not bother with characterizing the students incorrect + # answers at the concept level because an incorrect answer for one foil + # may be a correct answer for another foil. + $extrakey = ''; + $xlabel = 'Concept Number'; + $ylabel = 'Percent Correct'; + my %ConceptData; + foreach my $concept (@{$Concepts}) { + for (my $i=$mintries;$i<=$maxtries;$i++) { + # + # Gather the per-attempt data + my $cdata = $ConceptData{$concept}->[$i]; + foreach my $foilid (@{$concept->{'foils'}}) { + $cdata->{'_correct'} += + $ResponseData{$foilid}->[$i]->{'_correct'}; + $cdata->{'_total'} += + $ResponseData{$foilid}->[$i]->{'_total'}; + } + push (@{$PlotData[$i]->{'_total'}},$cdata->{'_total'}); + if ($cdata->{'_total'} == 0) { + push (@{$PlotData[$i]->{'_correct'}},0); + } else { + push (@{$PlotData[$i]->{'_correct'}}, + 100*$cdata->{'_correct'}/$cdata->{'_total'}); + } + } + } + } # End of work to fill @PlotData # # Build a table for the plots $table .= "\n"; my @Plots; for (my $i=$mintries;$i<=$maxtries;$i++) { - my $minstu = $ResponseData{$Foils[0]}->[$i]->{'total'}; - my $maxstu = $ResponseData{$Foils[0]}->[$i]->{'total'}; - foreach my $foilid (@Foils) { - if ($minstu > $ResponseData{$foilid}->[$i]->{'total'}) { - $minstu = $ResponseData{$foilid}->[$i]->{'total'}; - } - if ($maxstu < $ResponseData{$foilid}->[$i]->{'total'}) { - $maxstu = $ResponseData{$foilid}->[$i]->{'total'}; + my $minstu = $PlotData[$i]->{'_total'}->[0]; + my $maxstu = $PlotData[$i]->{'_total'}->[0]; + foreach my $count (@{$PlotData[$i]->{'_total'}}) { + if ($minstu > $count) { + $minstu = $count; + } + if ($maxstu < $count) { + $maxstu = $count; } } - $maxstu = 0 if (! $maxstu); - $minstu = 0 if (! $minstu); - my $graphlink; + $maxstu = 0 if (! defined($maxstu)); + $minstu = 0 if (! defined($minstu)); + my $title; if ($maxstu == $minstu) { - $graphlink = &Apache::loncommon::DrawGraph - ('Attempt '.$i.', '.$maxstu.' students', - 'Foil Number', - 'Percent Correct', - 100, - $PlotData[$i]->{'good'}, - $PlotData[$i]->{'bad'}); + $title = 'Attempt '.$i.', '.$maxstu.' students'; } else { - $graphlink = &Apache::loncommon::DrawGraph - ('Attempt '.$i.', '.$minstu.'-'.$maxstu. - ' students', - 'Foil Number', - 'Percent Correct', - 100, - $PlotData[$i]->{'good'}, - $PlotData[$i]->{'bad'}); + $title = 'Attempt '.$i.', '.$minstu.'-'.$maxstu.' students'; } + my @Datasets; + foreach my $option ('_correct',@{$ORdata->{'Options'}}) { + next if (! exists($PlotData[$i]->{$option})); + push(@Datasets,$PlotData[$i]->{$option}); + } + my $graphlink = &Apache::loncommon::DrawGraph($title, + $xlabel, + $ylabel, + 100, + $plotcolors, + @Datasets); push(@Plots,$graphlink); } # # Should this be something the user can set? Too many dialogs! - my $plots_per_row = 2; while (my $plotlink = shift(@Plots)) { - $table .= ''; - for (my $i=1;$i<$plots_per_row;$i++) { - if ($plotlink = shift(@Plots)) { - $table .= ''; - } else { - $table .= ''; - } - } - $table .= "\n"; + $table .= '\n"; } $table .= "
'.$plotlink.''.$plotlink.'
'.$plotlink.''.$extrakey."
\n"; return ($table); @@ -265,35 +410,25 @@ sub analyze_option_data_by_tries { foreach my $row (@$PerformanceData) { next if (! defined($row)); my ($grading,$submission,$time,$tries) = @$row; + next if ($grading eq 'MISSING_ANSWER'); my @Foilgrades = split('&',$grading); my @Foilsubs = split('&',$submission); for (my $numtries = 1; $numtries <= $maxtries; $numtries++) { if ($tries == $numtries) { - foreach my $foilgrade (@Foilgrades) { - my ($foilid,$correct) = split('=',$foilgrade); + for (my $i=0;$i<=$#Foilgrades;$i++) { + my ($foilid,$correct) = split('=',$Foilgrades[$i]); + my (undef,$submission) = split('=',$Foilsubs[$i]); + $submission = &Apache::lonnet::unescape($submission); if ($correct) { - $Trydata{$foilid}->[$numtries]->{'correct'}++; + $Trydata{$foilid}->[$numtries]->{'_correct'}++; } else { - $Trydata{$foilid}->[$numtries]->{'incorrect'}++; + $Trydata{$foilid}->[$numtries]->{$submission}++; } + $Trydata{$foilid}->[$numtries]->{'_total'}++; } } } } - foreach my $foilid (keys(%Trydata)) { - foreach my $tryhash (@{$Trydata{$foilid}}) { - next if ((! exists($tryhash->{'correct'}) && - ! exists($tryhash->{'incorrect'})) || - ($tryhash->{'correct'} < 1 && - $tryhash->{'incorrect'} < 1)); - $tryhash->{'total'} = $tryhash->{'correct'} + - $tryhash->{'incorrect'}; - $tryhash->{'percent_corr'} = 100 * - ($tryhash->{'correct'} / - ($tryhash->{'correct'} + $tryhash->{'incorrect'}) - ); - } - } return %Trydata; } @@ -307,9 +442,14 @@ sub analyze_option_data_by_tries { sub time_analysis { my ($PerformanceData,$ORdata) = @_; my $num_plots = $ENV{'form.NumPlots'}; - my ($table,@Foils) = &build_foil_index($ORdata); + my ($table,$Foils,$Concepts) = &build_foil_index($ORdata); my $num_data = scalar(@$PerformanceData)-1; my $percent = sprintf('%2f',100/$num_plots); + my $extratable = ''; + if ($ENV{'form.AnalyzeAs'} eq 'Foils') { + $extratable = &build_option_index($ORdata); + } + $table .= "\n"; for (my $i=0;$i<$num_plots;$i++) { my $starttime = &Apache::lonhtmlcommon::get_date_from_form ('startdate_'.$i); @@ -334,28 +474,32 @@ sub time_analysis { last if ($PerformanceData->[$j]->[2] > $endtime); } $end_index = $j; - $plottitle = 'Tries plot '.$i; + $plottitle = $ENV{'form.plottitle_'.$i}; } - ($plothtml,$starttime,$endtime,$data) = &analyze_option_data_by_time($PerformanceData, $begin_index,$end_index, - $plottitle, - @Foils); + $plottitle,$Foils, + $Concepts,$ORdata); my $startdateform = &Apache::lonhtmlcommon::date_setter ('Statistics','startdate_'.$i,$starttime); my $enddateform = &Apache::lonhtmlcommon::date_setter ('Statistics','enddate_'.$i,$endtime); - $plothtml.= "
\n". - "Start Time: "." ".$startdateform."
\n". - "End Time  : "." ".$enddateform."
\n"; - $table.=$plothtml; + $table.="\n"; } + $table .="
".$plothtml.''. + "Start Time:  ".$startdateform."
". + "End Time  : "." ".$enddateform."
". + 'Plot Title  :'.(" "x3). + '
'.$extratable. + "
\n"; return $table; } sub analyze_option_data_by_time { - my ($PerformanceData,$begin_index,$end_index,$description,@Foils) = @_; + my ($PerformanceData,$begin_index, + $end_index,$description,$Foils,$Concepts,$ORdata) = @_; my %TimeData; # # Get the start and end times for this segment of the plot @@ -367,51 +511,73 @@ sub analyze_option_data_by_time { my $row = $PerformanceData->[$i]; next if (! defined($row)); my ($grading,$submission,$time,$tries) = @$row; + next if ($grading eq 'MISSING_ANSWER'); my @Foilgrades = split('&',$grading); my @Foilsubs = split('&',$submission); - foreach my $foilgrade (@Foilgrades) { - my ($foilid,$correct) = split('=',$foilgrade); + for (my $j=0;$j<=$#Foilgrades;$j++) { + my ($foilid,$correct) = split('=',$Foilgrades[$j]); + my (undef,$submission) = split('=',$Foilsubs[$j]); if ($correct) { - $TimeData{$foilid}->{'correct'}++; + $TimeData{$foilid}->{'_correct'}++; } else { - $TimeData{$foilid}->{'incorrect'}++; + $submission = &HTML::Entities::decode($submission); + $submission =~ s/\%20/ /g; + $TimeData{$foilid}->{$submission}++; } + $TimeData{$foilid}->{'_total'}++; } } # # Compute the total and percent correct - my @Plotdata1; - my @Plotdata2; - foreach my $foilid (@Foils) { - if (! exists($TimeData{$foilid}->{'correct'})) { - $TimeData{$foilid}->{'correct'} = 0; + my @Plotdata; + my ($xlabel,$ylabel); + if ($ENV{'form.AnalyzeAs'} eq 'Foils') { + $xlabel = 'Foil Number'; + $ylabel = 'Option Chosen'; + foreach my $foil (@$Foils) { + my $total = $TimeData{$foil}->{'_total'}; + my $optionidx = 0; + foreach my $option ('_correct',@{$ORdata->{'Options'}}) { + if ($total > 0) { + push(@{$Plotdata[$optionidx]}, + 100 * $TimeData{$foil}->{$option} / $total); + } else { + push(@{$Plotdata[$optionidx]},0); + } + } continue { + $optionidx++; + } } - if (! exists($TimeData{$foilid}->{'incorrect'})) { - $TimeData{$foilid}->{'incorrect'} = 0; + } else { + $xlabel = 'Concept Number'; + $ylabel = 'Percent Correct'; + foreach my $concept (@$Concepts) { + my $correct; + my $total; + foreach my $foil (@{$concept->{'foils'}}) { + $correct+=$TimeData{$foil}->{'_correct'}; + $total +=$TimeData{$foil}->{'_total'}; + } + if ($total > 0) { + push(@{$Plotdata[0]},100 * $correct / $total); + } else { + push(@{$Plotdata[0]},0); + } } - $TimeData{$foilid}->{'total'} = $TimeData{$foilid}->{'correct'} + - $TimeData{$foilid}->{'incorrect'}; - $TimeData{$foilid}->{'percent_corr'} = 100 * - $TimeData{$foilid}->{'correct'} / - $TimeData{$foilid}->{'total'}; - push (@Plotdata1, $TimeData{$foilid}->{'percent_corr'}); - push (@Plotdata2,100-$TimeData{$foilid}->{'percent_corr'}); } # # Create the plot my $graphlink = &Apache::loncommon::DrawGraph ($description,#'Time Interval Analysis', - 'Foil Number', - 'Percent Correct / Incorrect', + $xlabel, + $ylabel, 100, - \@Plotdata1,\@Plotdata2); + $plotcolors, + @Plotdata); # return ($graphlink,$starttime,$endtime,\%TimeData); } - - - ######################################################### ######################################################### ## @@ -422,8 +588,8 @@ sub analyze_option_data_by_time { sub CreateInterface { ## ## Environment variable initialization - if (! exists$ENV{'form.AnalyzeBy'}) { - $ENV{'form.AnalyzeBy'} = 'Tries'; + if (! exists$ENV{'form.AnalyzeOver'}) { + $ENV{'form.AnalyzeOver'} = 'Tries'; } ## ## Build the menu @@ -456,39 +622,59 @@ sub CreateInterface { }; &Apache::lonstatistics::MapSelect('Maps','multiple,all',5, $only_seq_with_assessments); -# $Str .= ''; - # + ## + ## $Str .= ''; - $Str .= ''.&mt('Analyze By '); - $Str .='
'; - # - $Str .= '
'.&mt('Number of Plots:'); - $Str .= ''; + $Str .= '
'; + } + { + $Str .= ''.&mt('Analyze as '); + $Str .='
'; + } + { + $Str .= '
'.&mt('Number of Plots:'); + $Str .= ''; + } } - $Str .= ''; $Str .= ''; - # + ## + ## $Str .= ''."\n"; $Str .= ''."\n"; return ($Str); @@ -501,7 +687,6 @@ sub OptionResponseProblemSelector { next if ($seq->{'num_assess'}<1); my $seq_str = ''; foreach my $res (@{$seq->{'contents'}}) { -# &Apache::lonnet::logthis('checking '.$res->{'title'}); next if ($res->{'type'} ne 'assessment'); foreach my $part (@{$res->{'parts'}}) { my $partdata = $res->{'partdata'}->{$part}; @@ -564,20 +749,17 @@ sub get_resource_from_symb { return undef; } +## +## get problem data and put it into a useful data structure. +## note: we must force each foil and option to not begin or end with +## spaces as they are stored without such data. +## sub get_problem_data { my ($url) = @_; -# my $Answ=&Apache::lonnet::ssi($URI,('grade_target' => 'analyze', -# 'grade_username' => $sname, -# 'grade_domain' => $sdom, -# 'grade_courseid' => $cid, -# 'grade_symb' => $symb)); my $Answ=&Apache::lonnet::ssi($url,('grade_target' => 'analyze')); (my $garbage,$Answ)=split(/_HASH_REF__/,$Answ,2); my %Answer; %Answer=&Apache::lonnet::str2hash($Answ); -# &Apache::lonnet::logthis('keys of %Answer = '.join(', ',(keys(%Answer)))); -# &Apache::lonnet::logthis('$Answer{parts} = '. -# join(', ',@{$Answer{'parts'}})); my %Partdata; foreach my $part (@{$Answer{'parts'}}) { while (my($key,$value) = each(%Answer)) { @@ -591,45 +773,22 @@ sub get_problem_data { } elsif ($key =~ /^concept\.(.*)$/) { my $concept = $1; foreach my $foil (@$value) { - $Partdata{$part}->{$foil}->{'Concept'}=$concept; + $Partdata{$part}->{'Foils'}->{$foil}->{'Concept'}= + $concept; } } - # &Apache::lonnet::logthis($part.' '.$key.' (array) = '. - # join(', ',@$value)); } else { - $value =~ s/^\s*//g; - $value =~ s/\s*$//g; if ($key=~ /^foil\.text\.(.*)$/) { my $foil = $1; - $Partdata{$part}->{'Foiltext'}->{$foil}=$value; + $Partdata{$part}->{'Foils'}->{$foil}->{'name'}=$foil; + $Partdata{$part}->{'Foils'}->{$foil}->{'text'}=$value; } elsif ($key =~ /^foil\.value\.(.*)$/) { my $foil = $1; - $Partdata{$part}->{'FoilValues'}->{$foil}=$value; + $Partdata{$part}->{'Foils'}->{$foil}->{'value'}=$value; } -# &Apache::lonnet::logthis($part.' '.$key.' = '.$value); } } } - -# my $parts=''; -# foreach my $elm (@{$Answer{"parts"}}) { -# $parts.="$elm,"; -# } -# chop($parts); -# my $conc=''; -# foreach my $elm (@{$Answer{"$parts.concepts"}}) { -# $conc.="$elm@"; -# } -# chop($conc); -# -# @Concepts=split(/\@/,$conc); -# foreach my $concept (@{$Answer{"$parts.concepts"}}) { -# foreach my $foil (@{$Answer{"$parts.concept.$concept"}}) { -# $foil_to_concept{$foil} = $concept; -# #$ConceptData{$foil} = $Answer{"$parts.foil.value.$foil"}; -# } -# } -# return $symb; return %Partdata; }