--- loncom/interface/statistics/lonproblemanalysis.pm 2003/10/14 14:23:14 1.35 +++ loncom/interface/statistics/lonproblemanalysis.pm 2003/10/14 22:02:49 1.38 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # -# $Id: lonproblemanalysis.pm,v 1.35 2003/10/14 14:23:14 matthew Exp $ +# $Id: lonproblemanalysis.pm,v 1.38 2003/10/14 22:02:49 matthew Exp $ # # Copyright Michigan State University Board of Trustees # @@ -35,6 +35,7 @@ use Apache::lonhtmlcommon(); use Apache::loncoursedata(); use Apache::lonstatistics; use Apache::lonlocal; +use HTML::Entities(); sub BuildProblemAnalysisPage { my ($r,$c)=@_; @@ -100,11 +101,11 @@ sub BuildProblemAnalysisPage { (\@Students,$symb,$resid); if (defined($PerformanceData) && ref($PerformanceData) eq 'ARRAY') { - if ($ENV{'form.AnalyzeBy'} eq 'Tries') { + if ($ENV{'form.AnalyzeOver'} eq 'Tries') { my $analysis_html = &tries_analysis($PerformanceData, $ORdata); $r->print($analysis_html); - } elsif ($ENV{'form.AnalyzeBy'} eq 'Time') { + } elsif ($ENV{'form.AnalyzeOver'} eq 'Time') { my $analysis_html = &time_analysis($PerformanceData, $ORdata); $r->print($analysis_html); @@ -142,26 +143,85 @@ sub BuildProblemAnalysisPage { ######################################################### sub build_foil_index { my ($ORdata) = @_; - my @Foils = sort(keys(%{$ORdata->{'Foiltext'}})); - my %Row_Label; + my %Foildata = %{$ORdata->{'Foils'}}; + my @Foils = sort(keys(%Foildata)); + my %Concepts; foreach my $foilid (@Foils) { - my $value = $ORdata->{'Foiltext'}->{$foilid}; - $Row_Label{$foilid} = $ORdata->{'Foiltext'}->{$foilid}; + push(@{$Concepts{$Foildata{$foilid}->{'Concept'}}}, + $foilid); + } + undef(@Foils); + # Having gathered the concept information in a hash, we now translate it + # into an array because we need to be consistent about order. + # Also put the foils in order, too. + my $sortfunction = sub { + my %Numbers = (one => 1, + two => 2, + three => 3, + four => 4, + five => 5, + six => 6, + seven => 7, + eight => 8, + nine => 9, + ten => 10,); + my $a1 = $a; + my $b1 = $b; + if (exists($Numbers{$a})) { + $a1 = $Numbers{$a}; + } + if (exists($Numbers{$b})) { + $b1 = $Numbers{$b}; + } + $a1 cmp $b1; + }; + my @Concepts; + foreach my $concept (sort $sortfunction (keys(%Concepts))) { + push(@Concepts,{name => $concept, + foils => [@{$Concepts{$concept}}]}); + push(@Foils,(@{$Concepts{$concept}})); } # # Build up the table of row labels. my $table = ''."\n"; - $table .= ''. - '\n"; - my $index = 1; - foreach my $foilid (@Foils) { - $table .= ''. - '\n"; + $table .= ''. + ''. + ''. + ''. + ''. + ''. + ''. + "\n"; + my $conceptindex = 1; + my $foilindex = 1; + foreach my $concept (@Concepts) { + my @FoilsInConcept = @{$concept->{'foils'}}; + my $firstfoil = shift(@FoilsInConcept); + $table .= ''. + ''. + ''. + ''. + ''. + ''. + ''. + "\n"; + foreach my $foilid (@FoilsInConcept) { + $table .= ''. + ''. + ''. + ''. + ''. + ''. + ''. + "\n"; + } continue { + $foilindex++; + } } continue { - $index++; + $conceptindex++; } $table .= "
'.&mt('Foil Number').''.&mt('Foil Text')."
'.$index.''.$Row_Label{$foilid}."
'.&mt('Concept Number').''.&mt('Concept').''.&mt('Foil Number').''.&mt('Foil Name').''.&mt('Foil Text').''.&mt('Correct Value').'
'.$conceptindex.''.$concept->{'name'}.''.$foilindex++.''.$Foildata{$firstfoil}->{'name'}.''.$Foildata{$firstfoil}->{'text'}.''.$Foildata{$firstfoil}->{'value'}.'
'.$foilindex.''.$Foildata{$foilid}->{'name'}.''.$Foildata{$foilid}->{'text'}.''.$Foildata{$foilid}->{'value'}.'
\n"; - return ($table,@Foils); + return ($table,\@Foils,\@Concepts); } ######################################################### @@ -176,29 +236,56 @@ sub tries_analysis { my $mintries = 1; my $maxtries = $ENV{'form.NumPlots'}; my %ResponseData = &analyze_option_data_by_tries($PerformanceData, - $mintries,$maxtries); - my ($table,@Foils) = &build_foil_index($ORdata); + $mintries,$maxtries); + my ($table,$Foils,$Concepts) = &build_foil_index($ORdata); # # 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 $xlabel; + if ($ENV{'form.AnalyzeAs'} eq 'Foils') { + $xlabel = 'Foil Number'; + foreach my $foilid (@$Foils) { + for (my $i=$mintries;$i<=$maxtries;$i++) { + # + # Gather the per-attempt data + my $percent; + if ($ResponseData{$foilid}->[$i]->{'total'} == 0) { + $percent = 0; + } else { + $percent = $ResponseData{$foilid}->[$i]->{'correct'} / + $ResponseData{$foilid}->[$i]->{'total'}; + } + push (@{$PlotData[$i]->{'total'}}, + $ResponseData{$foilid}->[$i]->{'total'}); + push (@{$PlotData[$i]->{'good'}},100 * $percent); + push (@{$PlotData[$i]->{'bad'}}, 100 *(1-$percent)); + } + } + } else { + # Concept analysis + $xlabel = 'Concept Number'; + foreach my $concept (@$Concepts) { + for (my $i=$mintries;$i<=$maxtries;$i++) { + # + # Gather the per-attempt data + my ($correct,$incorrect,$total); + foreach my $foil (@{$concept->{'foils'}}) { + $correct += $ResponseData{$foil}->[$i]->{'correct'}; + $incorrect += $ResponseData{$foil}->[$i]->{'incorrect'}; + $total += $ResponseData{$foil}->[$i]->{'total'}; + } + push (@{$PlotData[$i]->{'correct'}}, $correct); + push (@{$PlotData[$i]->{'incorrect'}},$incorrect); + push (@{$PlotData[$i]->{'total'}}, $total); + my $percent; + if ($total == 0) { + $percent = 0; + } else { + $percent = $correct/$total; + } + push (@{$PlotData[$i]->{'good'}},100*$percent); + push (@{$PlotData[$i]->{'bad'}},100*(1-$percent)); + } } } # @@ -206,14 +293,14 @@ sub tries_analysis { $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'}; + 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 < $ResponseData{$foilid}->[$i]->{'total'}) { - $maxstu = $ResponseData{$foilid}->[$i]->{'total'}; + if ($maxstu < $count) { + $maxstu = $count; } } $maxstu = 0 if (! $maxstu); @@ -225,7 +312,7 @@ sub tries_analysis { $title = 'Attempt '.$i.', '.$minstu.'-'.$maxstu.' students'; } my $graphlink = &Apache::loncommon::DrawGraph($title, - 'Foil Number', + $xlabel, 'Percent Correct', 100, $PlotData[$i]->{'good'}, @@ -280,11 +367,7 @@ sub analyze_option_data_by_tries { ($tryhash->{'correct'} < 1 && $tryhash->{'incorrect'} < 1)); $tryhash->{'total'} = $tryhash->{'correct'} + - $tryhash->{'incorrect'}; - $tryhash->{'percent_corr'} = 100 * - ($tryhash->{'correct'} / - ($tryhash->{'correct'} + $tryhash->{'incorrect'}) - ); + $tryhash->{'incorrect'}; } } return %Trydata; @@ -300,9 +383,10 @@ 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); + $table .= "
\n"; for (my $i=0;$i<$num_plots;$i++) { my $starttime = &Apache::lonhtmlcommon::get_date_from_form ('startdate_'.$i); @@ -327,28 +411,31 @@ sub time_analysis { last if ($PerformanceData->[$j]->[2] > $endtime); } $end_index = $j; - $plottitle = 'Tries plot '.($i+1); + $plottitle = $ENV{'form.plottitle_'.$i}; } - ($plothtml,$starttime,$endtime,$data) = &analyze_option_data_by_time($PerformanceData, $begin_index,$end_index, $plottitle, - @Foils); + @$Concepts); 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). + '
'. + "
\n"; return $table; } sub analyze_option_data_by_time { - my ($PerformanceData,$begin_index,$end_index,$description,@Foils) = @_; + my ($PerformanceData,$begin_index,$end_index,$description,@Concepts) = @_; my %TimeData; # # Get the start and end times for this segment of the plot @@ -375,26 +462,56 @@ sub analyze_option_data_by_time { # Compute the total and percent correct my @Plotdata1; my @Plotdata2; - foreach my $foilid (@Foils) { - if (! exists($TimeData{$foilid}->{'correct'})) { - $TimeData{$foilid}->{'correct'} = 0; + foreach my $concept (@Concepts) { + my ($correct,$incorrect,$total); + foreach my $foilid (@{$concept->{'foils'}}) { + if (! exists($TimeData{$foilid}->{'correct'})) { + $TimeData{$foilid}->{'correct'} = 0; + } + if (! exists($TimeData{$foilid}->{'incorrect'})) { + $incorrect = 0; + $TimeData{$foilid}->{'incorrect'} = 0; + } + $correct += $TimeData{$foilid}->{'correct'}; + $incorrect += $TimeData{$foilid}->{'incorrect'}; + $total += $TimeData{$foilid}->{'correct'}+ + $TimeData{$foilid}->{'incorrect'}; + $TimeData{$foilid}->{'total'} = $TimeData{$foilid}->{'correct'} + + $TimeData{$foilid}->{'incorrect'}; + my $percent; + if ($TimeData{$foilid}->{'total'} == 0) { + $percent = 0; + } else { + $percent = $TimeData{$foilid}->{'correct'} / + $TimeData{$foilid}->{'total'}; + } + $TimeData{$foilid}->{'percent_corr'} = 100 * $percent; + if ($ENV{'form.AnalyzeAs'} eq 'Foils') { + push (@Plotdata1, $TimeData{$foilid}->{'percent_corr'}); + push (@Plotdata2,100-$TimeData{$foilid}->{'percent_corr'}); + } } - if (! exists($TimeData{$foilid}->{'incorrect'})) { - $TimeData{$foilid}->{'incorrect'} = 0; + if ($ENV{'form.AnalyzeAs'} ne 'Foils') { + if ($total == 0) { + push (@Plotdata1,0); + push (@Plotdata2,100); + } else { + push (@Plotdata1,100 * $correct / $total); + push (@Plotdata2,100 * (1-$correct / $total)); + } } - $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 $xlabel; + if ($ENV{'form.AnalyzeAs'} eq 'Foils') { + $xlabel = 'Foil Number'; + } else { + $xlabel = 'Concept Number'; + } my $graphlink = &Apache::loncommon::DrawGraph ($description,#'Time Interval Analysis', - 'Foil Number', + $xlabel, 'Percent Correct / Incorrect', 100, \@Plotdata1,\@Plotdata2); @@ -402,9 +519,6 @@ sub analyze_option_data_by_time { return ($graphlink,$starttime,$endtime,\%TimeData); } - - - ######################################################### ######################################################### ## @@ -415,8 +529,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 @@ -449,39 +563,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); @@ -494,7 +628,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}; @@ -559,18 +692,10 @@ sub get_resource_from_symb { 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)) { @@ -584,45 +709,24 @@ 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; }