--- loncom/interface/statistics/lonsubmissiontimeanalysis.pm 2005/04/19 13:59:35 1.24 +++ loncom/interface/statistics/lonsubmissiontimeanalysis.pm 2020/11/18 15:31:40 1.38 @@ -1,6 +1,6 @@ # The LearningOnline Network with CAPA # -# $Id: lonsubmissiontimeanalysis.pm,v 1.24 2005/04/19 13:59:35 matthew Exp $ +# $Id: lonsubmissiontimeanalysis.pm,v 1.38 2020/11/18 15:31:40 raeburn Exp $ # # Copyright Michigan State University Board of Trustees # @@ -31,6 +31,7 @@ use strict; use Apache::lonnet; use Apache::loncommon(); use Apache::lonhtmlcommon(); +use Apache::lonquickgrades(); use Apache::loncoursedata(); use Apache::lonstatistics; use Apache::lonstathelpers; @@ -66,12 +67,17 @@ sub BuildSubmissionTimePage { # &Apache::lonstatistics::PrepareClasslist(); # + $r->print(&Apache::lonhtmlcommon::breadcrumbs('Submission Time Plots')); + &Apache::lonquickgrades::startGradeScreen($r,'statistics'); $r->print(&CreateInterface()); # my @Students = @Apache::lonstatistics::Students; # if (@Students < 1) { - $r->print('

There are no students in the sections selected

'); + $r->print('
' + .&mt('There are no students in the sections selected.') + .'
' + ); } # my @CacheButtonHTML = @@ -81,7 +87,8 @@ sub BuildSubmissionTimePage { if (! exists($env{'form.problemchoice'}) || exists($env{'form.SelectAnother'})) { my $submit_button = ''; +# &mt('Graph Problem Submission Times').'" />'; + &mt('Generate Graph').'" />'; $r->print($submit_button.' 'x5); $r->print('

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

'); $r->print(&Apache::lonstathelpers::problem_selector('.', @@ -98,20 +105,9 @@ sub BuildSubmissionTimePage { $r->rflush(); # # Determine which problem we are to analyze - my $current_problem = &Apache::lonstathelpers::get_target_from_id - ($env{'form.problemchoice'}); - # - my ($navmap,$prev,$curr,$next) = - &Apache::lonstathelpers::get_prev_curr_next($current_problem, - '.', - 'part'); - if (exists($env{'form.PrevProblemAnalysis'}) && defined($prev)) { - $current_problem = $prev; - } elsif (exists($env{'form.NextProblemAnalysis'}) && defined($next)) { - $current_problem = $next; - } else { - $current_problem = $curr; - } + my ($navmap,$current_problem) = &get_current_problem(); # need to retrieve $navmap + # to support $resource->* calls + # for src and compTitle (below) # # Store the current problem choice and send it out in the form $env{'form.problemchoice'} = @@ -124,24 +120,46 @@ sub BuildSubmissionTimePage { # my $resource = $current_problem->{'resource'}; if (! defined($resource)) { - $r->print('resource is undefined'); + $r->print('
' + .&mt('Resource is undefined.') + .'
' + ); } else { $r->print('

'.$resource->compTitle.'

'); $r->print('

'.$resource->src.'

'); - $r->print('

'. + $r->print('

'. &Apache::lonstatistics::section_and_enrollment_description(). - '

'); + '

'); $r->rflush(); $r->print(&Apache::lonstathelpers::render_resource($resource)); $r->print('
'); $r->rflush(); - $r->print(&analyze_times($r,$resource->symb,\@Students, - $current_problem->{'part'})); + if (@Students) { + $r->print(&analyze_times($r,$resource->symb,\@Students, + $current_problem->{'part'})); + } } $r->print('
'); } } +sub get_current_problem { + my $current_problem = &Apache::lonstathelpers::get_target_from_id + ($env{'form.problemchoice'}); + my ($navmap,$prev,$curr,$next) = + &Apache::lonstathelpers::get_prev_curr_next($current_problem, + '.', + 'part'); + if (exists($env{'form.PrevProblemAnalysis'}) && defined($prev)) { + $current_problem = $prev; + } elsif (exists($env{'form.NextProblemAnalysis'}) && defined($next)) { + $current_problem = $next; + } else { + $current_problem = $curr; + } + return ($navmap,$current_problem); +} + ######################################################### ######################################################### ## @@ -174,15 +192,20 @@ sub analyze_times { # Get the data my $SubData = &Apache::loncoursedata::get_response_time_data ([&Apache::lonstatistics::get_selected_sections()], + [&Apache::lonstatistics::get_selected_groups()], $Apache::lonstatistics::enrollment_status, $symb,$part); if (! defined($SubData) || ! ref($SubData)) { - $html.= '

There is no submission data for this problem at all

'; + $html.= '
' + .&mt('There is no submission data for this problem at all.') + .'
'; return $html; } my $NumSub = scalar(@{$SubData}); if (! @{$SubData}) { - $html.= '

There is no submission data for this problem

'; + $html.= '
' + .&mt('There is no submission data for this problem.') + .'
'; return $html; } # Process the data @@ -202,7 +225,7 @@ sub analyze_times { my @Ydata=(0); # number of submissions my @AnsData=(0); # number of correct submissions my @Xlabel=($WeekDay[$wday]); # Labels of itmes - my @BinEnd; # The end time of each bin + my @BinEnd=($endtime); # The end time of each bin my $cumulative_answers = 0; # The sum of @AnsData my %students; # which students have attempted the problem? # @@ -214,7 +237,7 @@ sub analyze_times { $Ydata[$bincount] = 0; $AnsData[$bincount] = 0; $endtime += $binsize; - push(@BinEnd,$endtime); + $BinEnd[$bincount] = $endtime; if ($bincount % (86400/$binsize) == 0) { $wday++; $wday %= 7; @@ -236,7 +259,7 @@ sub analyze_times { $Ydata[$bincount]=0; $AnsData[$bincount]=0; $endtime += $binsize; - push(@BinEnd,$endtime); + $BinEnd[$bincount]=$endtime; if ($bincount % (86400/$binsize) == 0) { $wday ++; $wday %= 7; @@ -259,25 +282,19 @@ sub analyze_times { } # # Build the data table - $htmltable = '

'. - ''. - ''. - ''. + $htmltable = '

'.&mt('Student submission data').'

'. + &Apache::loncommon::start_data_table(). + &Apache::loncommon::start_data_table_header_row(). + &Apache::loncommon::start_data_table_row(). '

'. - ''. ''. ''. - ''. ''. - ''. ''. - ''. ''. - ''. ''. - ''. - ''. + &Apache::loncommon::end_data_table_row(). + &Apache::loncommon::end_data_table_header_row(). ''; my @CumulativeCorrect=(0); my @corr_as_percent_of_selected; @@ -290,28 +307,23 @@ sub analyze_times { sprintf('%3.1f',100*$CumulativeCorrect[$i]/scalar(@$students)); if ($Ydata[$i] != 0) { next if (! defined($BinEnd[$i]) || $BinEnd[$i] == 0); - $htmltable .= - ''. - ''. - ''. - ''. + ''. - ''. + ''. ''. - ''. ''. - ''. ''. - ''. ''. - ''. ''. - ''.$/; + &Apache::loncommon::end_data_table_row().$/; } } - $htmltable .= '
'.&mt('Begin').''.(' 'x3).''.&mt('End').''.&mt('Submissions (plotted)').''.(' 'x3).''.&mt('Correct Submissions (not plotted)').''.(' 'x3).''.&mt('Cumulative Correct of those attempting the problem (not plotted)').''.(' 'x3).''.&mt('Cumulative Percent Correct of those attempting the problem (not plotted)').''.(' 'x3).''.&mt('Cumulative Percent Correct of selected students (plotted)').'
'. + $htmltable .= + &Apache::loncommon::start_data_table_row(). + ''. &Apache::lonlocal::locallocaltime($BinEnd[$i]-$binsize). - ' '. + ''. &Apache::lonlocal::locallocaltime($BinEnd[$i]).''.$Ydata[$i].(' 'x3).' '.$AnsData[$i].(' 'x3).' '.$CumulativeCorrect[$i].' '.$corr_as_percent_of_answering[$i].' '.$corr_as_percent_of_selected[$i].'

'; + $htmltable .= ''.&Apache::loncommon::end_data_table().'

'; # # Build the plot my $title = '';#'Number of Submissions and Number Correct'; @@ -395,30 +407,32 @@ sub CreateInterface { ## ## Build the menu my $Str = ''; - $Str .= &Apache::lonhtmlcommon::breadcrumbs(undef,'Submission Time Plots'); $Str .= '

'; - $Str .= ''."\n"; - $Str .= ''; - $Str .= ''; - $Str .= ''; - $Str .= ''."\n"; + $Str .= &Apache::loncommon::start_data_table(); + $Str .= &Apache::loncommon::start_data_table_header_row(); + $Str .= ''; + $Str .= ''; + $Str .= ''; + $Str .= &Apache::loncommon::end_data_table_header_row(); ## ## - $Str .= ''; # + $Str .= ''; + # $Str .= ''; # - $Str .= ''."\n"; - $Str .= '
'.&mt('Sections').''.&mt('Enrollment Status').'
'.&mt('Sections').''.&mt('Groups').''.&mt('Access Status').'
'."\n"; + $Str .= &Apache::loncommon::start_data_table_row(); + $Str .= ''."\n"; $Str .= &Apache::lonstatistics::SectionSelect('Section','multiple',4); $Str .= ''."\n"; + $Str .= &Apache::lonstatistics::GroupSelect('Group','multiple',4); + $Str .= ''; $Str .= &Apache::lonhtmlcommon::StatusOptions(undef,undef,4); $Str .= '
'."\n"; + $Str .= &Apache::loncommon::end_data_table_row(); + $Str .= &Apache::loncommon::end_data_table(); # - $Str .= ''.&mt('Status: [_1]', - ''). - ''.'

'; + $Str .= '

'; ## return $Str; }