Description:
add more problem stat
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r457:c81651eecbbd - - 7 files changed: 144 inserted, 43 deleted
@@ -0,0 +1,44 | |||
|
1 | + - param = {} unless param | |
|
2 | + - graph_height = param[:graph_height] || 100 | |
|
3 | + - bar_width = param[:bar_width] || 14 | |
|
4 | + - graph_width = (bar_width * histogram[:data].count) + 20 | |
|
5 | + :css | |
|
6 | + .hist_bar { | |
|
7 | + width: #{bar_width-1}px; | |
|
8 | + position: absolute; | |
|
9 | + background-color: lightblue; | |
|
10 | + } | |
|
11 | + .hist_fill { | |
|
12 | + width: #{bar_width-1}px; | |
|
13 | + position: absolute; | |
|
14 | + background-color: #eee; | |
|
15 | + } | |
|
16 | + .hist_text { | |
|
17 | + position: absolute; | |
|
18 | + font-size:5px; | |
|
19 | + } | |
|
20 | + | |
|
21 | + %div{style: "position: relative; width: #{graph_width}px; height: 150px; background-color:#fff;" } | |
|
22 | + //draw background | |
|
23 | + - histogram[:data].each_index do |i| | |
|
24 | + - height = histogram[:data][i] * graph_height / histogram[:summary][:max] | |
|
25 | + - top = graph_height - height | |
|
26 | + - left = graph_width - (i+1)*bar_width | |
|
27 | + %div.hist_fill{style: "top: 0px; height: #{graph_height - height}px; left: #{left}px;" } | |
|
28 | + // draw horizontal line | |
|
29 | + - line = 3 | |
|
30 | + - line.times do |i| | |
|
31 | + - top = graph_height - graph_height * (i+0.5)/ line | |
|
32 | + %div{style: "position:absolute;width: #{graph_width-21}px;height: 1px;left: 20px;top:#{top}px;background-color: #333;"} | |
|
33 | + %div.hist_text{style: "position:absolute;left: 0px;top:#{top-6}px"} | |
|
34 | + =((i+0.5) * histogram[:summary][:max] / line).to_i | |
|
35 | + // draw the actual bar and text | |
|
36 | + - @histogram[:data].each_index do |i| | |
|
37 | + - height = histogram[:data][i] * graph_height / histogram[:summary][:max] | |
|
38 | + - top = graph_height - height | |
|
39 | + - left = graph_width - (i+1)*bar_width | |
|
40 | + %div.hist_bar{style: "top: #{top}px; height: #{height}px; left: #{left}px; dae: #{histogram[:data][i]}" } | |
|
41 | + - if i % 7 == 1 | |
|
42 | + %div.hist_text{style: "top:#{graph_height + 5}px;left: #{left}px;"} #{(Time.zone.today - i.day).strftime('%-d')} | |
|
43 | + - if (Time.now.in_time_zone - i.day).day == 15 | |
|
44 | + %div.hist_text{style: "top:#{graph_height + 15}px;left: #{left}px;"} #{(Time.zone.today - i.day).strftime('%b')} |
@@ -141,29 +141,44 | |||
|
141 | 141 | |
|
142 | 142 | def turn_all_on |
|
143 | 143 | Problem.find(:all, |
|
144 | 144 | :conditions => "available = 0").each do |problem| |
|
145 | 145 | problem.available = true |
|
146 | 146 | problem.save |
|
147 | 147 | end |
|
148 | 148 | redirect_to :action => 'list' |
|
149 | 149 | end |
|
150 | 150 | |
|
151 | 151 | def stat |
|
152 | 152 | @problem = Problem.find(params[:id]) |
|
153 |
- |
|
|
153 | + unless @problem.available or session[:admin] | |
|
154 | 154 | redirect_to :controller => 'main', :action => 'list' |
|
155 | - else | |
|
155 | + return | |
|
156 | + end | |
|
156 | 157 |
|
|
158 | + | |
|
159 | + #stat summary | |
|
160 | + range =65 | |
|
161 | + @histogram = { data: Array.new(range,0), summary: {} } | |
|
162 | + @histogram[:data] = Array.new(range,0) | |
|
163 | + user = Hash.new(0) | |
|
164 | + @submissions.find_each do |sub| | |
|
165 | + d = (DateTime.now.in_time_zone - sub.submitted_at) / 24 / 60 / 60 | |
|
166 | + @histogram[:data][d.to_i] += 1 if d < range | |
|
167 | + user[sub.user_id] = [user[sub.user_id], (sub.points >= @problem.full_score) ? 1 : 0].max | |
|
157 | 168 | end |
|
169 | + @histogram[:summary][:max] = [@histogram[:data].max,1].max | |
|
170 | + | |
|
171 | + @summary = { attempt: user.count, solve: 0 } | |
|
172 | + user.each_value { |v| @summary[:solve] += 1 if v == 1 } | |
|
158 | 173 | end |
|
159 | 174 | |
|
160 | 175 | def manage |
|
161 | 176 | @problems = Problem.find(:all, :order => 'date_added DESC') |
|
162 | 177 | end |
|
163 | 178 | |
|
164 | 179 | def do_manage |
|
165 | 180 | if params.has_key? 'change_date_added' |
|
166 | 181 | change_date_added |
|
167 | 182 | elsif params.has_key? 'add_to_contest' |
|
168 | 183 | add_to_contest |
|
169 | 184 | elsif params.has_key? 'enable_problem' |
@@ -248,13 +263,16 | |||
|
248 | 263 | |
|
249 | 264 | def get_problems_from_params |
|
250 | 265 | problems = [] |
|
251 | 266 | params.keys.each do |k| |
|
252 | 267 | if k.index('prob-')==0 |
|
253 | 268 | name, id, order = k.split('-') |
|
254 | 269 | problems << Problem.find(id) |
|
255 | 270 | end |
|
256 | 271 | end |
|
257 | 272 | problems |
|
258 | 273 | end |
|
259 | 274 | |
|
275 | + def get_problems_stat | |
|
260 | 276 | end |
|
277 | + | |
|
278 | + end |
@@ -96,77 +96,70 | |||
|
96 | 96 | |
|
97 | 97 | # get selected problems or the default |
|
98 | 98 | if params[:id] |
|
99 | 99 | begin |
|
100 | 100 | @problem = Problem.available.find(params[:id]) |
|
101 | 101 | rescue |
|
102 | 102 | redirect_to action: :problem_hof |
|
103 | 103 | flash[:notice] = 'Error: submissions for that problem are not viewable.' |
|
104 | 104 | return |
|
105 | 105 | end |
|
106 | 106 | end |
|
107 | 107 | |
|
108 |
- |
|
|
109 | - #aggregrate by language | |
|
110 | - @by_lang = {} | |
|
108 | + return unless @problem | |
|
109 | + | |
|
110 | + @by_lang = {} #aggregrate by language | |
|
111 | + | |
|
112 | + range =65 | |
|
113 | + @histogram = { data: Array.new(range,0), summary: {} } | |
|
114 | + @histogram[:data] = Array.new(range,0) | |
|
115 | + @summary = {count: 0, solve: 0, attempt: 0} | |
|
116 | + user = Hash.new(0) | |
|
111 | 117 |
|
|
118 | + #histogram | |
|
119 | + d = (DateTime.now.in_time_zone - sub.submitted_at) / 24 / 60 / 60 | |
|
120 | + @histogram[:data][d.to_i] += 1 if d < range | |
|
121 | + | |
|
122 | + @summary[:count] += 1 | |
|
123 | + user[sub.user_id] = [user[sub.user_id], (sub.points >= @problem.full_score) ? 1 : 0].max | |
|
124 | + | |
|
112 | 125 |
|
|
113 | 126 |
|
|
114 | 127 |
|
|
115 | 128 | |
|
116 | 129 |
|
|
117 | 130 |
|
|
118 | 131 |
|
|
119 | 132 |
|
|
120 | 133 |
|
|
121 | 134 |
|
|
122 | 135 |
|
|
123 | 136 |
|
|
124 | 137 |
|
|
125 | 138 | |
|
126 | 139 |
|
|
127 | - @by_lang[lang.pretty_name][:runtime] = { | |
|
128 | - avail: true, | |
|
129 | - user_id: sub.user_id, | |
|
130 | - value: sub.max_runtime, | |
|
131 | - sub_id: sub.id | |
|
132 | - } | |
|
140 | + @by_lang[lang.pretty_name][:runtime] = { avail: true, user_id: sub.user_id, value: sub.max_runtime, sub_id: sub.id } | |
|
133 | 141 |
|
|
134 | 142 | |
|
135 | 143 |
|
|
136 | - @by_lang[lang.pretty_name][:memory] = { | |
|
137 | - avail: true, | |
|
138 | - user_id: sub.user_id, | |
|
139 | - value: sub.peak_memory, | |
|
140 | - sub_id: sub.id | |
|
141 | - } | |
|
144 | + @by_lang[lang.pretty_name][:memory] = { avail: true, user_id: sub.user_id, value: sub.peak_memory, sub_id: sub.id } | |
|
142 | 145 |
|
|
143 | 146 | |
|
144 | 147 |
|
|
145 | 148 |
|
|
146 | - @by_lang[lang.pretty_name][:first] = { | |
|
147 | - avail: true, | |
|
148 | - user_id: sub.user_id, | |
|
149 | - value: sub.submitted_at, | |
|
150 | - sub_id: sub.id | |
|
151 | - } | |
|
149 | + @by_lang[lang.pretty_name][:first] = { avail: true, user_id: sub.user_id, value: sub.submitted_at, sub_id: sub.id } | |
|
152 | 150 |
|
|
153 | 151 | |
|
154 | 152 |
|
|
155 | - @by_lang[lang.pretty_name][:length] = { | |
|
156 | - avail: true, | |
|
157 | - user_id: sub.user_id, | |
|
158 | - value: sub.effective_code_length, | |
|
159 | - sub_id: sub.id | |
|
160 | - } | |
|
153 | + @by_lang[lang.pretty_name][:length] = { avail: true, user_id: sub.user_id, value: sub.effective_code_length, sub_id: sub.id } | |
|
161 | 154 |
|
|
162 | 155 |
|
|
163 | 156 | |
|
164 | 157 |
|
|
165 | 158 |
|
|
166 | 159 |
|
|
167 | 160 |
|
|
168 | 161 |
|
|
169 | 162 |
|
|
170 | 163 | |
|
171 | 164 |
|
|
172 | 165 |
|
@@ -181,15 +174,19 | |||
|
181 | 174 |
|
|
182 | 175 |
|
|
183 | 176 |
|
|
184 | 177 |
|
|
185 | 178 |
|
|
186 | 179 |
|
|
187 | 180 |
|
|
188 | 181 |
|
|
189 | 182 |
|
|
190 | 183 |
|
|
191 | 184 |
|
|
192 | 185 |
|
|
193 | - end | |
|
186 | + | |
|
187 | + @histogram[:summary][:max] = [@histogram[:data].max,1].max | |
|
188 | + @summary[:attempt] = user.count | |
|
189 | + user.each_value { |v| @summary[:solve] += 1 if v == 1 } | |
|
194 | 190 | end |
|
191 | + | |
|
195 | 192 | end |
@@ -46,24 +46,31 | |||
|
46 | 46 | import_params[:checker_name], |
|
47 | 47 | import_to_db) |
|
48 | 48 | problem.errors.add_to_base('Import error.') |
|
49 | 49 | end |
|
50 | 50 | |
|
51 | 51 | return problem, importer.log_msg |
|
52 | 52 | end |
|
53 | 53 | |
|
54 | 54 | def self.download_file_basedir |
|
55 | 55 | return "#{Rails.root}/data/tasks" |
|
56 | 56 | end |
|
57 | 57 | |
|
58 | + def get_submission_stat | |
|
59 | + result = Hash.new | |
|
60 | + #total number of submission | |
|
61 | + result[:total_sub] = Submission.where(problem_id: self.id).count | |
|
62 | + result[:attempted_user] = Submission.where(problem_id: self.id).group_by(:user_id) | |
|
63 | + end | |
|
64 | + | |
|
58 | 65 | protected |
|
59 | 66 | |
|
60 | 67 | def self.to_i_or_default(st, default) |
|
61 | 68 | if st!='' |
|
62 | 69 | result = st.to_i |
|
63 | 70 | end |
|
64 | 71 | result ||= default |
|
65 | 72 | end |
|
66 | 73 | |
|
67 | 74 | def self.to_f_or_default(st, default) |
|
68 | 75 | if st!='' |
|
69 | 76 | result = st.to_f |
@@ -1,20 +1,37 | |||
|
1 | 1 | :css |
|
2 | 2 | .fix-width { |
|
3 | 3 | font-family: "Consolas, Monaco, Droid Sans Mono,Mono, Monospace,Courier" |
|
4 | 4 | } |
|
5 | 5 | |
|
6 | 6 | %h1 Problem stat: #{@problem.name} |
|
7 | 7 | %h2 Overview |
|
8 | 8 | |
|
9 | + | |
|
10 | + %table.info | |
|
11 | + %thead | |
|
12 | + %tr.info-head | |
|
13 | + %th Stat | |
|
14 | + %th Value | |
|
15 | + %tbody | |
|
16 | + %tr{class: cycle('info-even','info-odd')} | |
|
17 | + %td Submissions | |
|
18 | + %td= @submissions.count | |
|
19 | + %tr{class: cycle('info-even','info-odd')} | |
|
20 | + %td Solved/Attempted User | |
|
21 | + %td #{@summary[:solve]}/#{@summary[:attempt]} (#{(@summary[:solve]*100.0/@summary[:attempt]).round(1)}%) | |
|
22 | + | |
|
23 | + %h2 Submissions Count | |
|
24 | + = render partial: 'application/bar_graph', locals: { histogram: @histogram } | |
|
25 | + | |
|
9 | 26 | %h2 Submissions |
|
10 | 27 | - if @submissions and @submissions.count > 0 |
|
11 | 28 | %table.info#main_table |
|
12 | 29 | %thead |
|
13 | 30 | %tr.info-head |
|
14 | 31 | %th ID |
|
15 | 32 | %th Login |
|
16 | 33 | %th Name |
|
17 | 34 | %th Submitted_at |
|
18 | 35 | %th Points |
|
19 | 36 | %th comment |
|
20 | 37 | %tbody |
@@ -1,57 +1,76 | |||
|
1 | 1 | :css |
|
2 | 2 | .hof_user { color: orangered; font-style: italic; } |
|
3 | 3 | .hof_language { color: green; font-style: italic; } |
|
4 | 4 | .hof_value { color: deeppink;font-style: italic; } |
|
5 | + .info_param { font-weight: bold;text-align: right; } | |
|
5 | 6 | |
|
6 |
- %h |
|
|
7 | + %h1 (#{Problem.find(params[:id]).name}) #{Problem.find(params[:id]).full_name} | |
|
7 | 8 | |
|
9 | + %h2 Problem Stat | |
|
10 | + %table.info | |
|
11 | + %thead | |
|
12 | + %tr.info-head | |
|
13 | + %th Stat | |
|
14 | + %th Value | |
|
15 | + %tbody | |
|
16 | + %tr{class: cycle('info-even','info-odd')} | |
|
17 | + %td.info_param Submissions | |
|
18 | + %td= @summary[:count] | |
|
19 | + %tr{class: cycle('info-even','info-odd')} | |
|
20 | + %td.info_param Solved/Attempted User | |
|
21 | + %td #{@summary[:solve]}/#{@summary[:attempt]} (#{(@summary[:solve]*100.0/@summary[:attempt]).round(1)}%) | |
|
8 | 22 | - if @best |
|
9 | - %b Best Runtime: | |
|
23 | + %tr{class: cycle('info-even','info-odd')} | |
|
24 | + %td.info_param Best Runtime | |
|
25 | + %td | |
|
10 | 26 | by #{link_to @best[:runtime][:user], controller:'users', action:'profile', id:@best[:memory][:user_id]} |
|
11 | 27 | using <span class="hof_language">#{@best[:runtime][:lang]}</span> |
|
12 | 28 | with <span class="hof_value">#{@best[:runtime][:value] * 1000} milliseconds</span> |
|
13 | 29 | at submission |
|
14 | 30 | = link_to("#" + @best[:runtime][:sub_id].to_s, controller: 'graders', action: 'submission', id:@best[:runtime][:sub_id]) |
|
15 | - %br/ | |
|
16 | 31 | |
|
17 | - %b Best Memory Usage: | |
|
32 | + %tr{class: cycle('info-even','info-odd')} | |
|
33 | + %td.info_param Best Memory Usage | |
|
34 | + %td | |
|
18 | 35 | by #{link_to @best[:memory][:user], controller:'users', action:'profile', id:@best[:memory][:user_id]} |
|
19 | 36 | using <span class="hof_language">#{@best[:memory][:lang]}</span> |
|
20 | 37 | with <span class="hof_value">#{number_with_delimiter(@best[:memory][:value])} kbytes </span> |
|
21 | 38 | at submission |
|
22 | 39 | = link_to("#" + @best[:memory][:sub_id].to_s, controller: 'graders' , action: 'submission', id:@best[:memory][:sub_id]) |
|
23 | - %br/ | |
|
24 | 40 | |
|
25 | - %b Shortest Code: | |
|
41 | + %tr{class: cycle('info-even','info-odd')} | |
|
42 | + %td.info_param Shortest Code | |
|
43 | + %td | |
|
26 | 44 | by #{link_to @best[:length][:user], controller:'users', action:'profile', id:@best[:length][:user_id]} |
|
27 | 45 | using <span class="hof_language">#{@best[:length][:lang]}</span> |
|
28 | 46 | with <span class="hof_value">#{@best[:length][:value]} bytes</span> |
|
29 | 47 | at submission |
|
30 | 48 | = link_to("#" + @best[:length][:sub_id].to_s, controller: 'graders' , action: 'submission', id: @best[:length][:sub_id]) |
|
31 | - %br/ | |
|
32 | 49 | |
|
33 | - %b First solver: | |
|
50 | + %tr{class: cycle('info-even','info-odd')} | |
|
51 | + %td.info_param First solver | |
|
52 | + %td | |
|
34 | 53 | #{link_to @best[:first][:user], controller:'users', action:'profile', id:@best[:first][:user_id]} is the first solver |
|
35 | 54 | using <span class="hof_language">#{@best[:first][:lang]}</span> |
|
36 | 55 | on <span class="hof_value">#{@best[:first][:value]}</span> |
|
37 | 56 | at submission |
|
38 | 57 | = link_to("#" + @best[:first][:sub_id].to_s, controller: 'graders' , action: 'submission', id: @best[:first][:sub_id]) |
|
39 | - %br/ | |
|
40 | 58 | |
|
41 | 59 | |
|
42 | 60 |
|
|
43 | 61 |
|
|
44 | 62 |
|
|
45 | 63 | |
|
64 | + - if @best | |
|
46 | 65 | %h2 By language |
|
47 | 66 | |
|
48 | 67 | %table.info |
|
49 | 68 | %thead |
|
50 | 69 | %tr.info-head |
|
51 | 70 | %th Language |
|
52 | 71 | %th Best runtime (ms) |
|
53 | 72 | %th Best memory (kbytes) |
|
54 | 73 | %th Shortest Code (bytes) |
|
55 | 74 | %th First solver |
|
56 | 75 | %tbody |
|
57 | 76 | - @by_lang.each do |lang,value| |
@@ -66,14 +85,12 | |||
|
66 | 85 | = "(#{number_with_delimiter(value[:memory][:value])} @" |
|
67 | 86 | = "#{link_to("#" + value[:memory][:sub_id].to_s, controller: 'graders' , action: 'submission', id: value[:memory][:sub_id])} )".html_safe |
|
68 | 87 | %td |
|
69 | 88 | = link_to value[:length][:user], controller: 'users', action: 'profile', id: value[:length][:user_id] |
|
70 | 89 | = "(#{value[:length][:value]} @" |
|
71 | 90 | = "#{link_to("#" + value[:length][:sub_id].to_s, controller: 'graders' , action: 'submission', id: value[:length][:sub_id])} )".html_safe |
|
72 | 91 | %td |
|
73 | 92 | - if value[:first][:user] != '(NULL)' #TODO: i know... this is wrong... |
|
74 | 93 | = link_to value[:first][:user], controller: 'users', action: 'profile', id: value[:first][:user_id] |
|
75 | 94 | = "(#{value[:first][:value]} @" |
|
76 | 95 | = "#{link_to("#" + value[:first][:sub_id].to_s, controller: 'graders' , action: 'submission', id: value[:first][:sub_id])} )".html_safe |
|
77 | 96 | |
|
78 | - - else | |
|
79 | - %h3 No submissions |
@@ -1,22 +1,23 | |||
|
1 | 1 | |
|
2 | 2 | /- if params[:id] |
|
3 | 3 | / %h1 Tasks Hall of Fame |
|
4 | 4 | / = link_to('[back to All-Time Hall of Fame]', action: 'problem_hof', id: nil ) |
|
5 | 5 | /- else |
|
6 | 6 | / %h1 All-Time Hall of Fame |
|
7 | 7 | |
|
8 | 8 | |
|
9 |
- %h1 |
|
|
9 | + %h1 Hall of Fame | |
|
10 | 10 | .task-menu |
|
11 | 11 | Tasks |
|
12 | 12 | %br/ |
|
13 | 13 | - @problems.each do |prob| |
|
14 | 14 | = link_to( "[#{prob.name}]", {id: prob.id}) |
|
15 | 15 | |
|
16 | 16 | - unless params[:id] |
|
17 | 17 | /=render partial: 'all_time_hof' |
|
18 | 18 | Please select a problem. |
|
19 | 19 | - else |
|
20 | 20 | =render partial: 'task_hof' |
|
21 | + %h2 Submission History | |
|
22 | + =render partial: 'application/bar_graph', locals: { histogram: @histogram } | |
|
21 | 23 | |
|
22 | - |
You need to be logged in to leave comments.
Login now