--- loncom/interface/statistics/lonproblemanalysis.pm 2003/09/29 21:13:23 1.24 +++ loncom/interface/statistics/lonproblemanalysis.pm 2003/10/14 22:02:49 1.38 @@ -1,6 +1,7 @@ # The LearningOnline Network with CAPA # -# $Id: lonproblemanalysis.pm,v 1.24 2003/09/29 21:13:23 matthew Exp $ + +# $Id: lonproblemanalysis.pm,v 1.38 2003/10/14 22:02:49 matthew Exp $ # # Copyright Michigan State University Board of Trustees # @@ -24,82 +25,534 @@ # # http://www.lon-capa.org/ # -# (Navigate problems for statistical reports -# YEAR=2002 -# 5/12,7/26,9/7,11/22 Behrouz Minaei -# -### package Apache::lonproblemanalysis; use strict; use Apache::lonnet(); +use Apache::loncommon(); use Apache::lonhtmlcommon(); use Apache::loncoursedata(); use Apache::lonstatistics; use Apache::lonlocal; +use HTML::Entities(); sub BuildProblemAnalysisPage { my ($r,$c)=@_; $r->print('

'.&mt('Option Response Problem Analysis').'

'); - if (exists($ENV{'form.problemchoice'})) { - # This is me getting around my own cleverness: - &Apache::lonstatistics::MapSelect('Maps','multiple,all',5, - undef); + $r->print(&CreateInterface()); + # + my @Students = @Apache::lonstatistics::Students; + # + if (exists($ENV{'form.updatecaches'}) || + (exists($ENV{'form.firstanalysis'}) && + $ENV{'form.firstanalysis'} ne 'no')) { + &Apache::lonstatistics::Gather_Full_Student_Data($r); + } + if (! exists($ENV{'form.firstanalysis'})) { + $r->print(''); + } else { + $r->print(''); + } + if (exists($ENV{'form.problemchoice'}) && + ! exists($ENV{'form.SelectAnother'})) { + $r->print(''); + $r->print(' 'x5); + $r->print(''); + $r->print(' 'x5); + $r->print(''); + $r->print(' 'x5); + $r->print(''); + $r->print(''); + $r->print(' 'x5); # - my ($symb,$id) = &get_problem_symb( + $r->print('
'); + # + my ($symb,$part,$resid) = &get_problem_symb( &Apache::lonnet::unescape($ENV{'form.problemchoice'}) ); - $r->print('
'); + # my $resource = &get_resource_from_symb($symb); if (defined($resource)) { + my %Data = &get_problem_data($resource->{'src'}); + my $ORdata = $Data{$part.'.'.$resid}; + ## + ## Render the problem + my $base; + ($base,undef) = ($resource->{'src'} =~ m|(.*/)[^/]*$|); + $base = "http://".$ENV{'SERVER_NAME'}.$base; + my $rendered_problem = + &Apache::lonnet::ssi_body($resource->{'src'}); + $rendered_problem =~ s/<\s*form\s*/)|<\/nop>|g; $r->print('
'. - # Oh this is dumb! Need to rewrite relative links - # otherwise images (for example) will not show. - &Apache::lonnet::ssi_body($resource->{'src'}). + ''. + $rendered_problem. '
'); + ## + ## Analyze the problem + my $PerformanceData = + &Apache::loncoursedata::get_optionresponse_data + (\@Students,$symb,$resid); + if (defined($PerformanceData) && + ref($PerformanceData) eq 'ARRAY') { + if ($ENV{'form.AnalyzeOver'} eq 'Tries') { + my $analysis_html = &tries_analysis($PerformanceData, + $ORdata); + $r->print($analysis_html); + } elsif ($ENV{'form.AnalyzeOver'} eq 'Time') { + my $analysis_html = &time_analysis($PerformanceData, + $ORdata); + $r->print($analysis_html); + } else { + $r->print('

'. + &mt('The analysis you have selected is '. + 'not supported at this time'). + '

'); + } + } else { + $r->print('

'. + &mt('There is no student data for this problem.'). + '

'); + } } else { $r->print('resource is undefined'); } - $r->print('
    '); - $r->print("
  1. render problem\n"); - $r->print("
  2. image tag for plot\n"); - $r->print("
  3. plot key\n"); - $r->print('
'); - $r->print("
\nProblem choice = $symb $id\n
\n"); $r->print('
'); + } else { + $r->print(''); + $r->print(' 'x5); + $r->print('

'.&mt('Please select a problem to analyze').'

'); + $r->print(&OptionResponseProblemSelector()); } - $r->print(''); - $r->print(&CreateInterface()); - if (! exists($ENV{'form.analysisfirstcall'})) { - return; +} + + +######################################################### +######################################################### +## +## Misc interface routines use by analysis code +## +######################################################### +######################################################### +sub build_foil_index { + my ($ORdata) = @_; + my %Foildata = %{$ORdata->{'Foils'}}; + my @Foils = sort(keys(%Foildata)); + my %Concepts; + foreach my $foilid (@Foils) { + 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}})); } - # Okay, they asked for data, so make sure we get the latest data. - &Apache::lonnet::logthis('got here for some reason'); -# &Apache::lonstatistics::Gather_Full_Student_Data($r); - $r->print(&OptionResponseProblemSelector()); + # + # Build up the table of row labels. + my $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 { + $conceptindex++; + } + $table .= "
'.&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,\@Concepts); } -sub get_problem_symb { - my $problemstring = shift(); - my ($symb,$id) = ($problemstring=~ /^(.*):([^:]*)$/); - return ($symb,$id); +######################################################### +######################################################### +## +## Tries Analysis +## +######################################################### +######################################################### +sub tries_analysis { + my ($PerformanceData,$ORdata) = @_; + my $mintries = 1; + my $maxtries = $ENV{'form.NumPlots'}; + my %ResponseData = &analyze_option_data_by_tries($PerformanceData, + $mintries,$maxtries); + my ($table,$Foils,$Concepts) = &build_foil_index($ORdata); + # + # Compute the data neccessary to make the plots + my @PlotData; + 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)); + } + } + } + # + # Build a table for the plots + $table .= "\n"; + my @Plots; + for (my $i=$mintries;$i<=$maxtries;$i++) { + 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 $title; + if ($maxstu == $minstu) { + $title = 'Attempt '.$i.', '.$maxstu.' students'; + } else { + $title = 'Attempt '.$i.', '.$minstu.'-'.$maxstu.' students'; + } + my $graphlink = &Apache::loncommon::DrawGraph($title, + $xlabel, + 'Percent Correct', + 100, + $PlotData[$i]->{'good'}, + $PlotData[$i]->{'bad'}); + 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 .= "
'.$plotlink.''.$plotlink.'
\n"; + return ($table); } +sub analyze_option_data_by_tries { + my ($PerformanceData,$mintries,$maxtries) = @_; + my %Trydata; + $mintries = 1 if (! defined($mintries) || $mintries < 1); + $maxtries = $mintries if (! defined($maxtries) || $maxtries < $mintries); + foreach my $row (@$PerformanceData) { + next if (! defined($row)); + my ($grading,$submission,$time,$tries) = @$row; + 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); + if ($correct) { + $Trydata{$foilid}->[$numtries]->{'correct'}++; + } else { + $Trydata{$foilid}->[$numtries]->{'incorrect'}++; + } + } + } + } + } + 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'}; + } + } + return %Trydata; +} + +######################################################### +######################################################### +## +## Time Analysis +## +######################################################### +######################################################### +sub time_analysis { + my ($PerformanceData,$ORdata) = @_; + my $num_plots = $ENV{'form.NumPlots'}; + 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); + my $endtime = &Apache::lonhtmlcommon::get_date_from_form + ('enddate_'.$i); + my ($begin_index,$end_index,$plottitle,$plothtml,$data); + if (! defined($starttime) || ! defined($endtime)) { + $begin_index = $i*int($num_data/$num_plots); + $end_index = ($i+1)*int($num_data/$num_plots); + my $lownum = sprintf('%2.1f',$i*$percent); + $lownum =~ s/(\.0)$//; + my $highnum = sprintf('%2.1f',($i+1)*$percent); + $highnum =~ s/(\.0)$//; + $plottitle = $lownum.'% to '.$highnum.'% of submissions'; + } else { + my $j; + while (++$j < scalar(@$PerformanceData)) { + last if ($PerformanceData->[$j]->[2] > $starttime); + } + $begin_index = $j; + while (++$j < scalar(@$PerformanceData)) { + last if ($PerformanceData->[$j]->[2] > $endtime); + } + $end_index = $j; + $plottitle = $ENV{'form.plottitle_'.$i}; + } + ($plothtml,$starttime,$endtime,$data) = + &analyze_option_data_by_time($PerformanceData, + $begin_index,$end_index, + $plottitle, + @$Concepts); + my $startdateform = &Apache::lonhtmlcommon::date_setter + ('Statistics','startdate_'.$i,$starttime); + my $enddateform = &Apache::lonhtmlcommon::date_setter + ('Statistics','enddate_'.$i,$endtime); + $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,@Concepts) = @_; + my %TimeData; + # + # Get the start and end times for this segment of the plot + my $starttime = $PerformanceData->[$begin_index]->[2]; + my $endtime = $PerformanceData->[$end_index ]->[2]; + # + # Compute the number getting the foils correct or incorrects + for (my $i=$begin_index;$i<=$end_index;$i++) { + my $row = $PerformanceData->[$i]; + next if (! defined($row)); + my ($grading,$submission,$time,$tries) = @$row; + my @Foilgrades = split('&',$grading); + my @Foilsubs = split('&',$submission); + foreach my $foilgrade (@Foilgrades) { + my ($foilid,$correct) = split('=',$foilgrade); + if ($correct) { + $TimeData{$foilid}->{'correct'}++; + } else { + $TimeData{$foilid}->{'incorrect'}++; + } + } + } + # + # Compute the total and percent correct + my @Plotdata1; + my @Plotdata2; + 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 ($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)); + } + } + } + # + # 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', + $xlabel, + 'Percent Correct / Incorrect', + 100, + \@Plotdata1,\@Plotdata2); + # + return ($graphlink,$starttime,$endtime,\%TimeData); +} + +######################################################### +######################################################### +## +## Interface +## +######################################################### +######################################################### sub CreateInterface { + ## + ## Environment variable initialization + if (! exists$ENV{'form.AnalyzeOver'}) { + $ENV{'form.AnalyzeOver'} = 'Tries'; + } + ## + ## Build the menu my $Str = ''; $Str .= ''."\n"; $Str .= ''; $Str .= ''; $Str .= ''; - $Str .= ''; +# $Str .= ''; + $Str .= ''; $Str .= ''."\n"; - # + ## + ## $Str .= ''; + # + $Str .= ''; + # +# $Str .= ''."\n"; + ## + ## + $Str .= ''; + ## + ## + $Str .= ''."\n"; $Str .= '
'.&mt('Sections').''.&mt('Enrollment Status').''.&mt('Sequences and Folders').''.&mt('Sequences and Folders').' 
'."\n"; $Str .= &Apache::lonstatistics::SectionSelect('Section','multiple',5); - $Str .= ''; + $Str .= ''; $Str .= &Apache::lonhtmlcommon::StatusOptions(undef,undef,5); - $Str .= ''; + $Str .= ''; my $only_seq_with_assessments = sub { my $s=shift; if ($s->{'num_assess'} < 1) { @@ -108,16 +561,63 @@ sub CreateInterface { return 1; } }; - $Str .= &Apache::lonstatistics::MapSelect('Maps','multiple,all',5, + &Apache::lonstatistics::MapSelect('Maps','multiple,all',5, $only_seq_with_assessments); - $Str .= '
'; + { # These braces are here to organize the code, not scope it. + { + $Str .= ''.&mt('Analyze Over '); + $Str .='
'; + } + { + $Str .= ''.&mt('Analyze as '); + $Str .='
'; + } + { + $Str .= '
'.&mt('Number of Plots:'); + $Str .= ''; + } + } + $Str .= '
'."\n"; - $Str .= ''; - $Str .= ' 'x5; - $Str .= ''; - $Str .= ' 'x5; return ($Str); } @@ -128,8 +628,7 @@ 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'); + next if ($res->{'type'} ne 'assessment'); foreach my $part (@{$res->{'parts'}}) { my $partdata = $res->{'partdata'}->{$part}; if (! exists($partdata->{'option'}) || @@ -140,7 +639,7 @@ sub OptionResponseProblemSelector { my $respid = $partdata->{'ResponseIds'}->[$i]; my $resptype = $partdata->{'ResponseTypes'}->[$i]; if ($resptype eq 'option') { - my $value = &Apache::lonnet::escape($res->{'symb'}.':'.$respid); + my $value = &Apache::lonnet::escape($res->{'symb'}.':'.$part.':'.$respid); my $checked = ''; if ($ENV{'form.problemchoice'} eq $value) { $checked = 'checked '; @@ -166,12 +665,23 @@ sub OptionResponseProblemSelector { return $Str; } +######################################################### +######################################################### +## +## Misc functions +## +######################################################### +######################################################### +sub get_problem_symb { + my $problemstring = shift(); + my ($symb,$partid,$resid) = ($problemstring=~ /^(.*):([^:]*):([^:]*)$/); + return ($symb,$partid,$resid); +} + sub get_resource_from_symb { my ($symb) = @_; - &Apache::lonnet::logthis('target symb = :'.$symb.':'); foreach my $seq (&Apache::lonstatistics::Sequences_with_Assess()) { foreach my $res (@{$seq->{'contents'}}) { - &Apache::lonnet::logthis('symb = :'.$res->{'symb'}.':'); if ($res->{'symb'} eq $symb) { return $res; } @@ -180,47 +690,46 @@ sub get_resource_from_symb { return undef; } -=pod - -sub InitAnalysis { - my ($resource,$sname,$sdom)=@_; - my $symb = $resource-> - my $URI = $hash{'src_'.$rid}; - - 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($URI,('grade_target' => 'analyze')); - +sub get_problem_data { + my ($url) = @_; + my $Answ=&Apache::lonnet::ssi($url,('grade_target' => 'analyze')); (my $garbage,$Answ)=split(/_HASH_REF__/,$Answ,2); - %Answer=(); + my %Answer; %Answer=&Apache::lonnet::str2hash($Answ); - - 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"}; + my %Partdata; + foreach my $part (@{$Answer{'parts'}}) { + while (my($key,$value) = each(%Answer)) { + next if ($key !~ /^$part/); + $key =~ s/^$part\.//; + if (ref($value) eq 'ARRAY') { + if ($key eq 'options') { + $Partdata{$part}->{'Options'}=$value; + } elsif ($key eq 'concepts') { + $Partdata{$part}->{'Concepts'}=$value; + } elsif ($key =~ /^concept\.(.*)$/) { + my $concept = $1; + foreach my $foil (@$value) { + $Partdata{$part}->{'Foils'}->{$foil}->{'Concept'}= + $concept; + } + } + } else { + $value =~ s/^\s*//g; + $value =~ s/\s*$//g; + if ($key=~ /^foil\.text\.(.*)$/) { + my $foil = $1; + $Partdata{$part}->{'Foils'}->{$foil}->{'name'}=$foil; + $Partdata{$part}->{'Foils'}->{$foil}->{'text'}=$value; + } elsif ($key =~ /^foil\.value\.(.*)$/) { + my $foil = $1; + $Partdata{$part}->{'Foils'}->{$foil}->{'value'}=$value; + } + } } } - return $symb; + return %Partdata; } -=cut - 1; __END__