Description:
fix display bugs in corrent score
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r782:97cd56a5f1d8 - - 3 files changed: 5 inserted, 4 deleted
@@ -444,48 +444,49 | |||||
|
444 |
|
444 | ||
|
445 | @st = <<-SQL |
|
445 | @st = <<-SQL |
|
446 | SELECT l.created_at as submitted_at ,-1 as id,u.login,u.full_name,l.ip_address,"" as problem_id,"" as points,l.user_id |
|
446 | SELECT l.created_at as submitted_at ,-1 as id,u.login,u.full_name,l.ip_address,"" as problem_id,"" as points,l.user_id |
|
447 | FROM logins l INNER JOIN users u on l.user_id = u.id |
|
447 | FROM logins l INNER JOIN users u on l.user_id = u.id |
|
448 | WHERE l.created_at >= ? AND l.created_at <= ? AND #{condition} |
|
448 | WHERE l.created_at >= ? AND l.created_at <= ? AND #{condition} |
|
449 | UNION |
|
449 | UNION |
|
450 | SELECT s.submitted_at,s.id,u.login,u.full_name,s.ip_address,s.problem_id,s.points,s.user_id |
|
450 | SELECT s.submitted_at,s.id,u.login,u.full_name,s.ip_address,s.problem_id,s.points,s.user_id |
|
451 | FROM submissions s INNER JOIN users u ON s.user_id = u.id |
|
451 | FROM submissions s INNER JOIN users u ON s.user_id = u.id |
|
452 | WHERE s.submitted_at >= ? AND s.submitted_at <= ? AND #{condition} |
|
452 | WHERE s.submitted_at >= ? AND s.submitted_at <= ? AND #{condition} |
|
453 | ORDER BY submitted_at |
|
453 | ORDER BY submitted_at |
|
454 | SQL |
|
454 | SQL |
|
455 |
|
455 | ||
|
456 | p = [@st,@since_time,@until_time] + @sid + [@since_time,@until_time] + @sid |
|
456 | p = [@st,@since_time,@until_time] + @sid + [@since_time,@until_time] + @sid |
|
457 | @logs = Submission.joins(:problem).find_by_sql(p) |
|
457 | @logs = Submission.joins(:problem).find_by_sql(p) |
|
458 |
|
458 | ||
|
459 |
|
459 | ||
|
460 |
|
460 | ||
|
461 |
|
461 | ||
|
462 |
|
462 | ||
|
463 | end |
|
463 | end |
|
464 |
|
464 | ||
|
465 | protected |
|
465 | protected |
|
466 |
|
466 | ||
|
467 | def calculate_max_score(problems, users,since_id,until_id, get_last_score = false) |
|
467 | def calculate_max_score(problems, users,since_id,until_id, get_last_score = false) |
|
|
468 | + #score[i] = user #i's user stat where i is the index (not id) | ||
|
468 | scorearray = Array.new |
|
469 | scorearray = Array.new |
|
469 | users.each do |u| |
|
470 | users.each do |u| |
|
470 | ustat = Array.new |
|
471 | ustat = Array.new |
|
471 | ustat[0] = u |
|
472 | ustat[0] = u |
|
472 | problems.each do |p| |
|
473 | problems.each do |p| |
|
473 | unless get_last_score |
|
474 | unless get_last_score |
|
474 | #get max score |
|
475 | #get max score |
|
475 | max_points = 0 |
|
476 | max_points = 0 |
|
476 | Submission.find_in_range_by_user_and_problem(u.id,p.id,since_id,until_id).each do |sub| |
|
477 | Submission.find_in_range_by_user_and_problem(u.id,p.id,since_id,until_id).each do |sub| |
|
477 | max_points = sub.points if sub and sub.points and (sub.points > max_points) |
|
478 | max_points = sub.points if sub and sub.points and (sub.points > max_points) |
|
478 | end |
|
479 | end |
|
479 | ustat << [(max_points.to_f*100/p.full_score).round, (max_points>=p.full_score)] |
|
480 | ustat << [(max_points.to_f*100/p.full_score).round, (max_points>=p.full_score)] |
|
480 | else |
|
481 | else |
|
481 | #get latest score |
|
482 | #get latest score |
|
482 | sub = Submission.find_last_by_user_and_problem(u.id,p.id) |
|
483 | sub = Submission.find_last_by_user_and_problem(u.id,p.id) |
|
483 | if (sub!=nil) and (sub.points!=nil) and p and p.full_score |
|
484 | if (sub!=nil) and (sub.points!=nil) and p and p.full_score |
|
484 | ustat << [(sub.points.to_f*100/p.full_score).round, (sub.points>=p.full_score)] |
|
485 | ustat << [(sub.points.to_f*100/p.full_score).round, (sub.points>=p.full_score)] |
|
485 | else |
|
486 | else |
|
486 | ustat << [0,false] |
|
487 | ustat << [0,false] |
|
487 | end |
|
488 | end |
|
488 | end |
|
489 | end |
|
489 | end |
|
490 | end |
|
490 | scorearray << ustat |
|
491 | scorearray << ustat |
|
491 | end |
|
492 | end |
@@ -1,41 +1,41 | |||||
|
1 | %table.table.sortable.table-striped.table-bordered.table-condensed |
|
1 | %table.table.sortable.table-striped.table-bordered.table-condensed |
|
2 | %thead |
|
2 | %thead |
|
3 | %tr |
|
3 | %tr |
|
4 | %th Login |
|
4 | %th Login |
|
5 | %th Name |
|
5 | %th Name |
|
6 | / %th Activated? |
|
6 | / %th Activated? |
|
7 | / %th Logged_in |
|
7 | / %th Logged_in |
|
8 | / %th Contest(s) |
|
8 | / %th Contest(s) |
|
9 | %th Remark |
|
9 | %th Remark |
|
10 | - @problems.each do |p| |
|
10 | - @problems.each do |p| |
|
11 | %th.text-right= p.name.gsub('_',' ') |
|
11 | %th.text-right= p.name.gsub('_',' ') |
|
12 | %th.text-right Total |
|
12 | %th.text-right Total |
|
13 | %th.text-right Passed |
|
13 | %th.text-right Passed |
|
14 | %tbody |
|
14 | %tbody |
|
15 |
- - sum = Array.new(@ |
|
15 | + - sum = Array.new(@problems.count,0) |
|
16 |
- - nonzero = Array.new(@ |
|
16 | + - nonzero = Array.new(@problems.count,0) |
|
17 |
- - full = Array.new(@ |
|
17 | + - full = Array.new(@problems.count,0) |
|
18 | - @scorearray.each do |sc| |
|
18 | - @scorearray.each do |sc| |
|
19 | %tr |
|
19 | %tr |
|
20 | - total,num_passed = 0,0 |
|
20 | - total,num_passed = 0,0 |
|
21 | - sc.each_index do |i| |
|
21 | - sc.each_index do |i| |
|
22 | - if i == 0 |
|
22 | - if i == 0 |
|
23 | %td= link_to sc[i].login, stat_user_path(sc[i]) |
|
23 | %td= link_to sc[i].login, stat_user_path(sc[i]) |
|
24 | %td= sc[i].full_name |
|
24 | %td= sc[i].full_name |
|
25 | / %td= sc[i].activated |
|
25 | / %td= sc[i].activated |
|
26 | / %td= sc[i].try(:contest_stat).try(:started_at) ? 'yes' : 'no' |
|
26 | / %td= sc[i].try(:contest_stat).try(:started_at) ? 'yes' : 'no' |
|
27 | / %td= sc[i].contests.collect {|c| c.name}.join(', ') |
|
27 | / %td= sc[i].contests.collect {|c| c.name}.join(', ') |
|
28 | %td= sc[i].remark |
|
28 | %td= sc[i].remark |
|
29 | - else |
|
29 | - else |
|
30 | %td.text-right= sc[i][0] |
|
30 | %td.text-right= sc[i][0] |
|
31 | - total += sc[i][0] |
|
31 | - total += sc[i][0] |
|
32 | - num_passed += 1 if sc[i][1] |
|
32 | - num_passed += 1 if sc[i][1] |
|
33 | - sum[i] += sc[i][0] |
|
33 | - sum[i] += sc[i][0] |
|
34 | - nonzero[i] += 1 if sc[i][0] > 0 |
|
34 | - nonzero[i] += 1 if sc[i][0] > 0 |
|
35 | - full[i] += 1 if sc[i][1] |
|
35 | - full[i] += 1 if sc[i][1] |
|
36 | %td.text-right= total |
|
36 | %td.text-right= total |
|
37 | %td.text-right= num_passed |
|
37 | %td.text-right= num_passed |
|
38 | %tfoot |
|
38 | %tfoot |
|
39 | %tr |
|
39 | %tr |
|
40 | %td Summation |
|
40 | %td Summation |
|
41 | %td |
|
41 | %td |
@@ -1,11 +1,11 | |||||
|
1 | .container-fluid |
|
1 | .container-fluid |
|
2 | %h1 Current Score |
|
2 | %h1 Current Score |
|
3 | = form_tag current_score_report_path, method: 'get' do |
|
3 | = form_tag current_score_report_path, method: 'get' do |
|
4 | Show only users from this group |
|
4 | Show only users from this group |
|
5 |
- = select_tag "group_id", options_from_collection_for_select( Group.all, 'id','name',params[:group_ |
|
5 | + = select_tag "group_id", options_from_collection_for_select( Group.all, 'id','name',params[:group_id]), id: 'group_name',class: 'select2', style: 'width: 20em'; |
|
6 | = submit_tag 'Apply',class: 'btn btn-default' |
|
6 | = submit_tag 'Apply',class: 'btn btn-default' |
|
7 |
|
7 | ||
|
8 | %br |
|
8 | %br |
|
9 |
|
9 | ||
|
10 |
|
10 | ||
|
11 | = render "score_table" |
|
11 | = render "score_table" |
You need to be logged in to leave comments.
Login now