Description:
add struggle report
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r463:ef96a7a29f25 - - 3 files changed: 43 inserted, 1 deleted
@@ -0,0 +1,17 | |||||
|
|
1 | + %table.info | ||
|
|
2 | + %thead | ||
|
|
3 | + %tr.info-head | ||
|
|
4 | + %th Problem | ||
|
|
5 | + %th User | ||
|
|
6 | + %th tries | ||
|
|
7 | + %tbody | ||
|
|
8 | + - @struggle.each do |s| | ||
|
|
9 | + %tr | ||
|
|
10 | + %td | ||
|
|
11 | + = link_to "(#{s[:problem].name})", controller: :problems, action: :stat, id: s[:problem] | ||
|
|
12 | + = s[:problem].full_name | ||
|
|
13 | + %td | ||
|
|
14 | + = link_to "(#{s[:user].login})", controller: :users, action: :profile, id: s[:user] | ||
|
|
15 | + = s[:user].full_name | ||
|
|
16 | + %td | ||
|
|
17 | + = s[:tries] |
@@ -1,191 +1,216 | |||||
|
1 | class ReportController < ApplicationController |
|
1 | class ReportController < ApplicationController |
|
2 |
|
2 | ||
|
3 | before_filter :admin_authorization, only: [:login_stat,:submission_stat] |
|
3 | before_filter :admin_authorization, only: [:login_stat,:submission_stat] |
|
4 | before_filter(only: [:problem_hof]) { |c| |
|
4 | before_filter(only: [:problem_hof]) { |c| |
|
5 | return false unless authenticate |
|
5 | return false unless authenticate |
|
6 |
|
6 | ||
|
7 | if GraderConfiguration["right.user_view_submission"] |
|
7 | if GraderConfiguration["right.user_view_submission"] |
|
8 | return true; |
|
8 | return true; |
|
9 | end |
|
9 | end |
|
10 |
|
10 | ||
|
11 | admin_authorization |
|
11 | admin_authorization |
|
12 | } |
|
12 | } |
|
13 |
|
13 | ||
|
14 | def login_stat |
|
14 | def login_stat |
|
15 | @logins = Array.new |
|
15 | @logins = Array.new |
|
16 |
|
16 | ||
|
17 | date_and_time = '%Y-%m-%d %H:%M' |
|
17 | date_and_time = '%Y-%m-%d %H:%M' |
|
18 | begin |
|
18 | begin |
|
19 | md = params[:since_datetime].match(/(\d+)-(\d+)-(\d+) (\d+):(\d+)/) |
|
19 | md = params[:since_datetime].match(/(\d+)-(\d+)-(\d+) (\d+):(\d+)/) |
|
20 | @since_time = Time.zone.local(md[1].to_i,md[2].to_i,md[3].to_i,md[4].to_i,md[5].to_i) |
|
20 | @since_time = Time.zone.local(md[1].to_i,md[2].to_i,md[3].to_i,md[4].to_i,md[5].to_i) |
|
21 | rescue |
|
21 | rescue |
|
22 | @since_time = DateTime.new(1000,1,1) |
|
22 | @since_time = DateTime.new(1000,1,1) |
|
23 | end |
|
23 | end |
|
24 | begin |
|
24 | begin |
|
25 | md = params[:until_datetime].match(/(\d+)-(\d+)-(\d+) (\d+):(\d+)/) |
|
25 | md = params[:until_datetime].match(/(\d+)-(\d+)-(\d+) (\d+):(\d+)/) |
|
26 | @until_time = Time.zone.local(md[1].to_i,md[2].to_i,md[3].to_i,md[4].to_i,md[5].to_i) |
|
26 | @until_time = Time.zone.local(md[1].to_i,md[2].to_i,md[3].to_i,md[4].to_i,md[5].to_i) |
|
27 | rescue |
|
27 | rescue |
|
28 | @until_time = DateTime.new(3000,1,1) |
|
28 | @until_time = DateTime.new(3000,1,1) |
|
29 | end |
|
29 | end |
|
30 |
|
30 | ||
|
31 | User.all.each do |user| |
|
31 | User.all.each do |user| |
|
32 | @logins << { id: user.id, |
|
32 | @logins << { id: user.id, |
|
33 | login: user.login, |
|
33 | login: user.login, |
|
34 | full_name: user.full_name, |
|
34 | full_name: user.full_name, |
|
35 | count: Login.where("user_id = ? AND created_at >= ? AND created_at <= ?", |
|
35 | count: Login.where("user_id = ? AND created_at >= ? AND created_at <= ?", |
|
36 | user.id,@since_time,@until_time) |
|
36 | user.id,@since_time,@until_time) |
|
37 | .count(:id), |
|
37 | .count(:id), |
|
38 | min: Login.where("user_id = ? AND created_at >= ? AND created_at <= ?", |
|
38 | min: Login.where("user_id = ? AND created_at >= ? AND created_at <= ?", |
|
39 | user.id,@since_time,@until_time) |
|
39 | user.id,@since_time,@until_time) |
|
40 | .minimum(:created_at), |
|
40 | .minimum(:created_at), |
|
41 | max: Login.where("user_id = ? AND created_at >= ? AND created_at <= ?", |
|
41 | max: Login.where("user_id = ? AND created_at >= ? AND created_at <= ?", |
|
42 | user.id,@since_time,@until_time) |
|
42 | user.id,@since_time,@until_time) |
|
43 | .maximum(:created_at), |
|
43 | .maximum(:created_at), |
|
44 | ip: Login.where("user_id = ? AND created_at >= ? AND created_at <= ?", |
|
44 | ip: Login.where("user_id = ? AND created_at >= ? AND created_at <= ?", |
|
45 | user.id,@since_time,@until_time) |
|
45 | user.id,@since_time,@until_time) |
|
46 | .select(:ip_address).uniq |
|
46 | .select(:ip_address).uniq |
|
47 |
|
47 | ||
|
48 | } |
|
48 | } |
|
49 | end |
|
49 | end |
|
50 | end |
|
50 | end |
|
51 |
|
51 | ||
|
52 | def submission_stat |
|
52 | def submission_stat |
|
53 |
|
53 | ||
|
54 | date_and_time = '%Y-%m-%d %H:%M' |
|
54 | date_and_time = '%Y-%m-%d %H:%M' |
|
55 | begin |
|
55 | begin |
|
56 | @since_time = DateTime.strptime(params[:since_datetime],date_and_time) |
|
56 | @since_time = DateTime.strptime(params[:since_datetime],date_and_time) |
|
57 | rescue |
|
57 | rescue |
|
58 | @since_time = DateTime.new(1000,1,1) |
|
58 | @since_time = DateTime.new(1000,1,1) |
|
59 | end |
|
59 | end |
|
60 | begin |
|
60 | begin |
|
61 | @until_time = DateTime.strptime(params[:until_datetime],date_and_time) |
|
61 | @until_time = DateTime.strptime(params[:until_datetime],date_and_time) |
|
62 | rescue |
|
62 | rescue |
|
63 | @until_time = DateTime.new(3000,1,1) |
|
63 | @until_time = DateTime.new(3000,1,1) |
|
64 | end |
|
64 | end |
|
65 |
|
65 | ||
|
66 | @submissions = {} |
|
66 | @submissions = {} |
|
67 |
|
67 | ||
|
68 | User.find_each do |user| |
|
68 | User.find_each do |user| |
|
69 | @submissions[user.id] = { login: user.login, full_name: user.full_name, count: 0, sub: { } } |
|
69 | @submissions[user.id] = { login: user.login, full_name: user.full_name, count: 0, sub: { } } |
|
70 | end |
|
70 | end |
|
71 |
|
71 | ||
|
72 | Submission.where("submitted_at >= ? AND submitted_at <= ?",@since_time,@until_time).find_each do |s| |
|
72 | Submission.where("submitted_at >= ? AND submitted_at <= ?",@since_time,@until_time).find_each do |s| |
|
73 | if @submissions[s.user_id] |
|
73 | if @submissions[s.user_id] |
|
74 | if not @submissions[s.user_id][:sub].has_key?(s.problem_id) |
|
74 | if not @submissions[s.user_id][:sub].has_key?(s.problem_id) |
|
75 | a = nil |
|
75 | a = nil |
|
76 | begin |
|
76 | begin |
|
77 | a = Problem.find(s.problem_id) |
|
77 | a = Problem.find(s.problem_id) |
|
78 | rescue |
|
78 | rescue |
|
79 | a = nil |
|
79 | a = nil |
|
80 | end |
|
80 | end |
|
81 | @submissions[s.user_id][:sub][s.problem_id] = |
|
81 | @submissions[s.user_id][:sub][s.problem_id] = |
|
82 | { prob_name: (a ? a.full_name : '(NULL)'), |
|
82 | { prob_name: (a ? a.full_name : '(NULL)'), |
|
83 | sub_ids: [s.id] } |
|
83 | sub_ids: [s.id] } |
|
84 | else |
|
84 | else |
|
85 | @submissions[s.user_id][:sub][s.problem_id][:sub_ids] << s.id |
|
85 | @submissions[s.user_id][:sub][s.problem_id][:sub_ids] << s.id |
|
86 | end |
|
86 | end |
|
87 | @submissions[s.user_id][:count] += 1 |
|
87 | @submissions[s.user_id][:count] += 1 |
|
88 | end |
|
88 | end |
|
89 | end |
|
89 | end |
|
90 | end |
|
90 | end |
|
91 |
|
91 | ||
|
92 | def problem_hof |
|
92 | def problem_hof |
|
93 | # gen problem list |
|
93 | # gen problem list |
|
94 | @user = User.find(session[:user_id]) |
|
94 | @user = User.find(session[:user_id]) |
|
95 | @problems = @user.available_problems |
|
95 | @problems = @user.available_problems |
|
96 |
|
96 | ||
|
97 | # get selected problems or the default |
|
97 | # get selected problems or the default |
|
98 | if params[:id] |
|
98 | if params[:id] |
|
99 | begin |
|
99 | begin |
|
100 | @problem = Problem.available.find(params[:id]) |
|
100 | @problem = Problem.available.find(params[:id]) |
|
101 | rescue |
|
101 | rescue |
|
102 | redirect_to action: :problem_hof |
|
102 | redirect_to action: :problem_hof |
|
103 | flash[:notice] = 'Error: submissions for that problem are not viewable.' |
|
103 | flash[:notice] = 'Error: submissions for that problem are not viewable.' |
|
104 | return |
|
104 | return |
|
105 | end |
|
105 | end |
|
106 | end |
|
106 | end |
|
107 |
|
107 | ||
|
108 | return unless @problem |
|
108 | return unless @problem |
|
109 |
|
109 | ||
|
110 | @by_lang = {} #aggregrate by language |
|
110 | @by_lang = {} #aggregrate by language |
|
111 |
|
111 | ||
|
112 | range =65 |
|
112 | range =65 |
|
113 | @histogram = { data: Array.new(range,0), summary: {} } |
|
113 | @histogram = { data: Array.new(range,0), summary: {} } |
|
114 | @summary = {count: 0, solve: 0, attempt: 0} |
|
114 | @summary = {count: 0, solve: 0, attempt: 0} |
|
115 | user = Hash.new(0) |
|
115 | user = Hash.new(0) |
|
116 | Submission.where(problem_id: @problem.id).find_each do |sub| |
|
116 | Submission.where(problem_id: @problem.id).find_each do |sub| |
|
117 | #histogram |
|
117 | #histogram |
|
118 | d = (DateTime.now.in_time_zone - sub.submitted_at) / 24 / 60 / 60 |
|
118 | d = (DateTime.now.in_time_zone - sub.submitted_at) / 24 / 60 / 60 |
|
119 | @histogram[:data][d.to_i] += 1 if d < range |
|
119 | @histogram[:data][d.to_i] += 1 if d < range |
|
120 |
|
120 | ||
|
121 | @summary[:count] += 1 |
|
121 | @summary[:count] += 1 |
|
122 | user[sub.user_id] = [user[sub.user_id], (sub.points >= @problem.full_score) ? 1 : 0].max |
|
122 | user[sub.user_id] = [user[sub.user_id], (sub.points >= @problem.full_score) ? 1 : 0].max |
|
123 |
|
123 | ||
|
124 | lang = Language.find_by_id(sub.language_id) |
|
124 | lang = Language.find_by_id(sub.language_id) |
|
125 | next unless lang |
|
125 | next unless lang |
|
126 | next unless sub.points >= @problem.full_score |
|
126 | next unless sub.points >= @problem.full_score |
|
127 |
|
127 | ||
|
128 | #initialize |
|
128 | #initialize |
|
129 | unless @by_lang.has_key?(lang.pretty_name) |
|
129 | unless @by_lang.has_key?(lang.pretty_name) |
|
130 | @by_lang[lang.pretty_name] = { |
|
130 | @by_lang[lang.pretty_name] = { |
|
131 | runtime: { avail: false, value: 2**30-1 }, |
|
131 | runtime: { avail: false, value: 2**30-1 }, |
|
132 | memory: { avail: false, value: 2**30-1 }, |
|
132 | memory: { avail: false, value: 2**30-1 }, |
|
133 | length: { avail: false, value: 2**30-1 }, |
|
133 | length: { avail: false, value: 2**30-1 }, |
|
134 | first: { avail: false, value: DateTime.new(3000,1,1) } |
|
134 | first: { avail: false, value: DateTime.new(3000,1,1) } |
|
135 | } |
|
135 | } |
|
136 | end |
|
136 | end |
|
137 |
|
137 | ||
|
138 | if sub.max_runtime and sub.max_runtime < @by_lang[lang.pretty_name][:runtime][:value] |
|
138 | if sub.max_runtime and sub.max_runtime < @by_lang[lang.pretty_name][:runtime][:value] |
|
139 | @by_lang[lang.pretty_name][:runtime] = { avail: true, user_id: sub.user_id, value: sub.max_runtime, sub_id: sub.id } |
|
139 | @by_lang[lang.pretty_name][:runtime] = { avail: true, user_id: sub.user_id, value: sub.max_runtime, sub_id: sub.id } |
|
140 | end |
|
140 | end |
|
141 |
|
141 | ||
|
142 | if sub.peak_memory and sub.peak_memory < @by_lang[lang.pretty_name][:memory][:value] |
|
142 | if sub.peak_memory and sub.peak_memory < @by_lang[lang.pretty_name][:memory][:value] |
|
143 | @by_lang[lang.pretty_name][:memory] = { avail: true, user_id: sub.user_id, value: sub.peak_memory, sub_id: sub.id } |
|
143 | @by_lang[lang.pretty_name][:memory] = { avail: true, user_id: sub.user_id, value: sub.peak_memory, sub_id: sub.id } |
|
144 | end |
|
144 | end |
|
145 |
|
145 | ||
|
146 | if sub.submitted_at and sub.submitted_at < @by_lang[lang.pretty_name][:first][:value] and |
|
146 | if sub.submitted_at and sub.submitted_at < @by_lang[lang.pretty_name][:first][:value] and |
|
147 | !sub.user.admin? |
|
147 | !sub.user.admin? |
|
148 | @by_lang[lang.pretty_name][:first] = { avail: true, user_id: sub.user_id, value: sub.submitted_at, sub_id: sub.id } |
|
148 | @by_lang[lang.pretty_name][:first] = { avail: true, user_id: sub.user_id, value: sub.submitted_at, sub_id: sub.id } |
|
149 | end |
|
149 | end |
|
150 |
|
150 | ||
|
151 | if @by_lang[lang.pretty_name][:length][:value] > sub.effective_code_length |
|
151 | if @by_lang[lang.pretty_name][:length][:value] > sub.effective_code_length |
|
152 | @by_lang[lang.pretty_name][:length] = { avail: true, user_id: sub.user_id, value: sub.effective_code_length, sub_id: sub.id } |
|
152 | @by_lang[lang.pretty_name][:length] = { avail: true, user_id: sub.user_id, value: sub.effective_code_length, sub_id: sub.id } |
|
153 | end |
|
153 | end |
|
154 | end |
|
154 | end |
|
155 |
|
155 | ||
|
156 | #process user_id |
|
156 | #process user_id |
|
157 | @by_lang.each do |lang,prop| |
|
157 | @by_lang.each do |lang,prop| |
|
158 | prop.each do |k,v| |
|
158 | prop.each do |k,v| |
|
159 | v[:user] = User.exists?(v[:user_id]) ? User.find(v[:user_id]).full_name : "(NULL)" |
|
159 | v[:user] = User.exists?(v[:user_id]) ? User.find(v[:user_id]).full_name : "(NULL)" |
|
160 | end |
|
160 | end |
|
161 | end |
|
161 | end |
|
162 |
|
162 | ||
|
163 | #sum into best |
|
163 | #sum into best |
|
164 | if @by_lang and @by_lang.first |
|
164 | if @by_lang and @by_lang.first |
|
165 | @best = @by_lang.first[1].clone |
|
165 | @best = @by_lang.first[1].clone |
|
166 | @by_lang.each do |lang,prop| |
|
166 | @by_lang.each do |lang,prop| |
|
167 | if @best[:runtime][:value] >= prop[:runtime][:value] |
|
167 | if @best[:runtime][:value] >= prop[:runtime][:value] |
|
168 | @best[:runtime] = prop[:runtime] |
|
168 | @best[:runtime] = prop[:runtime] |
|
169 | @best[:runtime][:lang] = lang |
|
169 | @best[:runtime][:lang] = lang |
|
170 | end |
|
170 | end |
|
171 | if @best[:memory][:value] >= prop[:memory][:value] |
|
171 | if @best[:memory][:value] >= prop[:memory][:value] |
|
172 | @best[:memory] = prop[:memory] |
|
172 | @best[:memory] = prop[:memory] |
|
173 | @best[:memory][:lang] = lang |
|
173 | @best[:memory][:lang] = lang |
|
174 | end |
|
174 | end |
|
175 | if @best[:length][:value] >= prop[:length][:value] |
|
175 | if @best[:length][:value] >= prop[:length][:value] |
|
176 | @best[:length] = prop[:length] |
|
176 | @best[:length] = prop[:length] |
|
177 | @best[:length][:lang] = lang |
|
177 | @best[:length][:lang] = lang |
|
178 | end |
|
178 | end |
|
179 | if @best[:first][:value] >= prop[:first][:value] |
|
179 | if @best[:first][:value] >= prop[:first][:value] |
|
180 | @best[:first] = prop[:first] |
|
180 | @best[:first] = prop[:first] |
|
181 | @best[:first][:lang] = lang |
|
181 | @best[:first][:lang] = lang |
|
182 | end |
|
182 | end |
|
183 | end |
|
183 | end |
|
184 | end |
|
184 | end |
|
185 |
|
185 | ||
|
186 | @histogram[:summary][:max] = [@histogram[:data].max,1].max |
|
186 | @histogram[:summary][:max] = [@histogram[:data].max,1].max |
|
187 | @summary[:attempt] = user.count |
|
187 | @summary[:attempt] = user.count |
|
188 | user.each_value { |v| @summary[:solve] += 1 if v == 1 } |
|
188 | user.each_value { |v| @summary[:solve] += 1 if v == 1 } |
|
189 | end |
|
189 | end |
|
190 |
|
190 | ||
|
|
191 | + def stuck #report struggling user,problem | ||
|
|
192 | + # init | ||
|
|
193 | + user,problem = nil | ||
|
|
194 | + solve = true | ||
|
|
195 | + tries = 0 | ||
|
|
196 | + @struggle = Array.new | ||
|
|
197 | + record = {} | ||
|
|
198 | + Submission.includes(:problem,:user).order(:problem_id,:user_id).find_each do |sub| | ||
|
|
199 | + if user != sub.user_id or problem != sub.problem_id | ||
|
|
200 | + @struggle << { user: record[:user], problem: record[:problem], tries: tries } unless solve | ||
|
|
201 | + record = {user: sub.user, problem: sub.problem} | ||
|
|
202 | + user,problem = sub.user_id, sub.problem_id | ||
|
|
203 | + solve = false | ||
|
|
204 | + tries = 0 | ||
|
|
205 | + end | ||
|
|
206 | + if sub.points >= sub.problem.full_score | ||
|
|
207 | + solve = true | ||
|
|
208 | + else | ||
|
|
209 | + tries += 1 | ||
|
|
210 | + end | ||
|
|
211 | + end | ||
|
|
212 | + @struggle.sort!{|a,b| b[:tries] <=> a[:tries] } | ||
|
|
213 | + @struggle = @struggle[0..50] | ||
|
|
214 | + end | ||
|
|
215 | + | ||
|
191 | end |
|
216 | end |
@@ -1,7 +1,7 | |||||
|
1 |
|
1 | ||
|
2 | .task-menu |
|
2 | .task-menu |
|
3 | Reports |
|
3 | Reports |
|
4 | %br/ |
|
4 | %br/ |
|
5 | = link_to '[Hall of Fame]', :action => 'problem_hof' |
|
5 | = link_to '[Hall of Fame]', :action => 'problem_hof' |
|
6 |
- = link_to '[S |
|
6 | + = link_to '[Struggle]', :action => 'stuck' |
|
7 | = link_to '[Login]', :action => 'login_stat' |
|
7 | = link_to '[Login]', :action => 'login_stat' |
You need to be logged in to leave comments.
Login now