--- loncom/interface/statistics/lonsubmissiontimeanalysis.pm 2005/03/08 17:39:35 1.20 +++ loncom/interface/statistics/lonsubmissiontimeanalysis.pm 2020/11/18 15:38:56 1.35.2.4 @@ -1,6 +1,6 @@ # The LearningOnline Network with CAPA # -# $Id: lonsubmissiontimeanalysis.pm,v 1.20 2005/03/08 17:39:35 matthew Exp $ +# $Id: lonsubmissiontimeanalysis.pm,v 1.35.2.4 2020/11/18 15:38:56 raeburn Exp $ # # Copyright Michigan State University Board of Trustees # @@ -28,7 +28,7 @@ package Apache::lonsubmissiontimeanalysis; use strict; -use Apache::lonnet(); +use Apache::lonnet; use Apache::loncommon(); use Apache::lonhtmlcommon(); use Apache::loncoursedata(); @@ -37,7 +37,6 @@ use Apache::lonstathelpers; use Apache::lonlocal; use HTML::Entities(); use Time::Local(); -use Spreadsheet::WriteExcel(); my $plotcolors = ['#33ff00', '#ff33cc', '#990000', '#aaaa66', '#663399', '#ff9933', @@ -67,25 +66,31 @@ sub BuildSubmissionTimePage { # &Apache::lonstatistics::PrepareClasslist(); # + $r->print(&Apache::lonhtmlcommon::breadcrumbs('Submission Time Plots')); $r->print(&CreateInterface()); # my @Students = @Apache::lonstatistics::Students; # if (@Students < 1) { - $r->print('<h2>There are no students in the sections selected</h2>'); + $r->print('<div class="LC_warning">' + .&mt('There are no students in the sections selected.') + .'</div>' + ); } # my @CacheButtonHTML = &Apache::lonstathelpers::manage_caches($r,'Statistics','stats_status'); $r->rflush(); # - if (! exists($ENV{'form.problemchoice'}) || - exists($ENV{'form.SelectAnother'})) { - $r->print('<input type="submit" name="" value="'. - &mt('Graph Problem Submission Times').'" />'); - $r->print(' 'x5); + if (! exists($env{'form.problemchoice'}) || + exists($env{'form.SelectAnother'})) { + my $submit_button = '<input type="submit" name="" value="'. +# &mt('Graph Problem Submission Times').'" />'; + &mt('Generate Graph').'" />'; + $r->print($submit_button.' 'x5); $r->print('<h3>'.&mt('Please select a problem to analyze').'</h3>'); - $r->print(&Apache::lonstathelpers::problem_selector('.')); + $r->print(&Apache::lonstathelpers::problem_selector('.', + $submit_button)); } else { foreach my $button (@SubmitButtons) { $r->print('<input type="submit" name="'.$button->{'name'}.'" '. @@ -98,47 +103,61 @@ 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'} = + $env{'form.problemchoice'} = &Apache::lonstathelpers::make_target_id($current_problem); $r->print('<input type="hidden" name="problemchoice" value="'. - $ENV{'form.problemchoice'}.'" />'); + $env{'form.problemchoice'}.'" />'); # $r->print('<hr />'); $r->rflush(); # my $resource = $current_problem->{'resource'}; if (! defined($resource)) { - $r->print('resource is undefined'); + $r->print('<div class="LC_warning">' + .&mt('Resource is undefined.') + .'</div>' + ); } else { $r->print('<h1>'.$resource->compTitle.'</h1>'); $r->print('<h3>'.$resource->src.'</h3>'); + $r->print('<p>'. + &Apache::lonstatistics::section_and_enrollment_description(). + '</p>'); $r->rflush(); $r->print(&Apache::lonstathelpers::render_resource($resource)); $r->print('<br />'); $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('<hr />'); } } +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); +} + ######################################################### ######################################################### ## @@ -170,16 +189,21 @@ sub analyze_times { my $html; # holds results of analysis # Get the data my $SubData = &Apache::loncoursedata::get_response_time_data - (\@Apache::lonstatistics::SelectedSections, + ([&Apache::lonstatistics::get_selected_sections()], + [&Apache::lonstatistics::get_selected_groups()], $Apache::lonstatistics::enrollment_status, $symb,$part); if (! defined($SubData) || ! ref($SubData)) { - $html.= '<h2>There is no submission data for this problem at all</h2>'; + $html.= '<div class="LC_warning">' + .&mt('There is no submission data for this problem at all.') + .'</div>'; return $html; } my $NumSub = scalar(@{$SubData}); if (! @{$SubData}) { - $html.= '<h2>There is no submission data for this problem</h2>'; + $html.= '<div class="LC_warning">' + .&mt('There is no submission data for this problem.') + .'</div>'; return $html; } # Process the data @@ -199,7 +223,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? # @@ -211,7 +235,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; @@ -233,7 +257,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; @@ -256,25 +280,19 @@ sub analyze_times { } # # Build the data table - $htmltable = '<p>'. - '<table rules="groups" frame="border" '. - 'summary="Student submission data">'. - '<thead>'. - '<tr>'. + $htmltable = '<br><h3>'.&mt('Student submission data').'</h3><p>'. + &Apache::loncommon::start_data_table(). + &Apache::loncommon::start_data_table_header_row(). + &Apache::loncommon::start_data_table_row(). '<th valign="bottom">'.&mt('Begin').'</th>'. - '<th>'.(' 'x3).'</th>'. '<th valign="bottom">'.&mt('End').'</th>'. '<th valign="bottom">'.&mt('Submissions (plotted)').'</th>'. - '<th>'.(' 'x3).'</th>'. '<th valign="bottom">'.&mt('Correct Submissions (not plotted)').'</th>'. - '<th>'.(' 'x3).'</th>'. '<th valign="bottom">'.&mt('Cumulative Correct of those attempting the problem (not plotted)').'</th>'. - '<th>'.(' 'x3).'</th>'. '<th valign="bottom">'.&mt('Cumulative Percent Correct of those attempting the problem (not plotted)').'</th>'. - '<th>'.(' 'x3).'</th>'. '<th valign="bottom">'.&mt('Cumulative Percent Correct of selected students (plotted)').'</th>'. - '</tr>'. - '</thead>'. + &Apache::loncommon::end_data_table_row(). + &Apache::loncommon::end_data_table_header_row(). '<tbody>'; my @CumulativeCorrect=(0); my @corr_as_percent_of_selected; @@ -287,28 +305,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 .= - '<tr>'. - '<td align="right"><nobr>'. + $htmltable .= + &Apache::loncommon::start_data_table_row(). + '<td align="right"><span class="LC_nobreak">'. &Apache::lonlocal::locallocaltime($BinEnd[$i]-$binsize). - '</nobr></td>'. - '<td> </td>'. - '<td align="right"><nobr>'. + '</span></td>'. + '<td align="right"><span class="LC_nobreak">'. &Apache::lonlocal::locallocaltime($BinEnd[$i]).'</td>'. - '</nobr></td>'. + '</span></td>'. '<td align="right">'.$Ydata[$i].(' 'x3).'</td>'. - '<td> </td>'. '<td align="right">'.$AnsData[$i].(' 'x3).'</td>'. - '<td> </td>'. '<td align="right">'.$CumulativeCorrect[$i].'</td>'. - '<td> </td>'. '<td align="right">'.$corr_as_percent_of_answering[$i].'</td>'. - '<td> </td>'. '<td align="right">'.$corr_as_percent_of_selected[$i].'</td>'. - '</tr>'.$/; + &Apache::loncommon::end_data_table_row().$/; } } - $htmltable .= '</tbody></table></p>'; + $htmltable .= '</tbody>'.&Apache::loncommon::end_data_table().'</p>'; # # Build the plot my $title = '';#'Number of Submissions and Number Correct'; @@ -386,36 +399,38 @@ sub Process_Row { sub CreateInterface { ## ## Environment variable initialization - if (! exists$ENV{'form.AnalyzeOver'}) { - $ENV{'form.AnalyzeOver'} = 'Tries'; + if (! exists$env{'form.AnalyzeOver'}) { + $env{'form.AnalyzeOver'} = 'Tries'; } ## ## Build the menu my $Str = ''; - $Str .= &Apache::lonhtmlcommon::breadcrumbs(undef,'Submission Time Plots'); $Str .= '<p>'; - $Str .= '<table cellspacing="5">'."\n"; - $Str .= '<tr>'; - $Str .= '<th align="center">'.&mt('Sections').'</th>'; - $Str .= '<th align="center">'.&mt('Enrollment Status').'</th>'; - $Str .= '</tr>'."\n"; + $Str .= &Apache::loncommon::start_data_table(); + $Str .= &Apache::loncommon::start_data_table_header_row(); + $Str .= '<th>'.&mt('Sections').'</th>'; + $Str .= '<th>'.&mt('Groups').'</th>'; + $Str .= '<th>'.&mt('Access Status').'</th>'; + $Str .= &Apache::loncommon::end_data_table_header_row(); ## ## - $Str .= '<tr><td align="center">'."\n"; + $Str .= &Apache::loncommon::start_data_table_row(); + $Str .= '<td align="center">'."\n"; $Str .= &Apache::lonstatistics::SectionSelect('Section','multiple',4); $Str .= '</td>'; # + $Str .= '<td align="center">'."\n"; + $Str .= &Apache::lonstatistics::GroupSelect('Group','multiple',4); + $Str .= '</td>'; + # $Str .= '<td align="center">'; $Str .= &Apache::lonhtmlcommon::StatusOptions(undef,undef,4); $Str .= '</td>'; # - $Str .= '</tr>'."\n"; - $Str .= '</table>'."\n"; + $Str .= &Apache::loncommon::end_data_table_row(); + $Str .= &Apache::loncommon::end_data_table(); # - $Str .= '<nobr>'.&mt('Status: [_1]', - '<input type="text" '. - 'name="stats_status" size="60" value="" />'). - '</nobr>'.'</p>'; + $Str .= '</p>'; ## return $Str; }