Description:
added test assignment time out
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r222:2b6293a64dbb - - 8 files changed: 42 inserted, 2 deleted
@@ -194,24 +194,29 | |||
|
194 | 194 | end |
|
195 | 195 | end |
|
196 | 196 | |
|
197 | 197 | def submit_solution |
|
198 | 198 | problem = Problem.find(params[:id]) |
|
199 | 199 | user = User.find(session[:user_id]) |
|
200 | 200 | recent_assignment = user.get_recent_test_pair_assignment_for problem |
|
201 | 201 | if recent_assignment == nil |
|
202 | 202 | flash[:notice] = 'You have not requested for any input data for this problem. Please download an input first.' |
|
203 | 203 | redirect_to :action => 'list' and return |
|
204 | 204 | end |
|
205 | 205 | |
|
206 | + if recent_assignment.expired? | |
|
207 | + flash[:notice] = 'The current input is expired. Please download a new input data.' | |
|
208 | + redirect_to :action => 'list' and return | |
|
209 | + end | |
|
210 | + | |
|
206 | 211 | if recent_assignment.submitted |
|
207 | 212 | flash[:notice] = 'You have already submitted an incorrect solution for this input. Please download a new input data.' |
|
208 | 213 | redirect_to :action => 'list' and return |
|
209 | 214 | end |
|
210 | 215 | |
|
211 | 216 | if params[:file] == nil |
|
212 | 217 | flash[:notice] = 'You have not submitted any output.' |
|
213 | 218 | redirect_to :action => 'list' and return |
|
214 | 219 | end |
|
215 | 220 | |
|
216 | 221 | submitted_solution = params[:file].read |
|
217 | 222 | test_pair = recent_assignment.test_pair |
@@ -1,5 +1,11 | |||
|
1 | 1 | class TestPairAssignment < ActiveRecord::Base |
|
2 | + | |
|
2 | 3 | belongs_to :user |
|
3 | 4 | belongs_to :test_pair |
|
4 | 5 | belongs_to :problem |
|
6 | + | |
|
7 | + def expired? | |
|
8 | + return created_at + TEST_ASSIGNMENT_EXPIRATION_DURATION < Time.new.gmtime | |
|
9 | + end | |
|
10 | + | |
|
5 | 11 | end |
@@ -87,25 +87,25 | |||
|
87 | 87 | return nil |
|
88 | 88 | else |
|
89 | 89 | recent = assignments[0] |
|
90 | 90 | assignments.each do |a| |
|
91 | 91 | recent = a if a.request_number > recent.request_number |
|
92 | 92 | end |
|
93 | 93 | return recent |
|
94 | 94 | end |
|
95 | 95 | end |
|
96 | 96 | |
|
97 | 97 | def can_request_new_test_pair_for?(problem) |
|
98 | 98 | recent = get_recent_test_pair_assignment_for problem |
|
99 | - return (recent == nil or recent.submitted) | |
|
99 | + return (recent == nil or recent.submitted or recent.expired?) | |
|
100 | 100 | end |
|
101 | 101 | |
|
102 | 102 | def get_new_test_pair_assignment_for(problem) |
|
103 | 103 | previous_assignment_numbers = |
|
104 | 104 | get_test_pair_assignments_for(problem).collect {|a| a.test_pair_number } |
|
105 | 105 | test_pair = problem.random_test_pair(previous_assignment_numbers) |
|
106 | 106 | if test_pair |
|
107 | 107 | assignment = TestPairAssignment.new(:user => self, |
|
108 | 108 | :problem => problem, |
|
109 | 109 | :test_pair => test_pair, |
|
110 | 110 | :test_pair_number => test_pair.number, |
|
111 | 111 | :request_number => |
@@ -5,18 +5,18 | |||
|
5 | 5 | <head> |
|
6 | 6 | <meta http-equiv="content-type" content="text/html;charset=UTF-8" /> |
|
7 | 7 | <title><%= Configuration['contest.name'] %></title> |
|
8 | 8 | <%= stylesheet_link_tag 'application' %> |
|
9 | 9 | <%= yield :head %> |
|
10 | 10 | </head> |
|
11 | 11 | <body> |
|
12 | 12 | |
|
13 | 13 | <div class="userbar"> |
|
14 | 14 | <%= user_header %> |
|
15 | 15 | </div> |
|
16 | 16 | |
|
17 | - <%= content_tag(:p,flash[:notice],:style => "color:green") if flash[:notice]!=nil %> | |
|
17 | + <% if flash[:notice]!=nil %><div class="notice-bar"><span class="notice"><%= flash[:notice] %></span></div><% end %> | |
|
18 | 18 | |
|
19 | 19 | <%= yield %> |
|
20 | 20 | |
|
21 | 21 | </body> |
|
22 | 22 | </html> |
@@ -1,17 +1,18 | |||
|
1 | 1 | .problem-panel{:id => "problem-panel-#{problem.id}", :style => "display:none"} |
|
2 | 2 | .problem-form{:id => "problem-form-#{problem.id}"} |
|
3 | 3 | - form_tag({ :action => 'download_input', :id => problem.id }, :method => :post) do |
|
4 | 4 | %b Input: |
|
5 | 5 | %input{:type => "submit", :value => "Download input"} |
|
6 | + = "After downloading, you have #{TEST_ASSIGNMENT_EXPIRATION_DURATION/60} minutes to submit." | |
|
6 | 7 | - form_tag({ :action => 'submit_solution', :id => problem.id }, :method => :post, :multipart => true) do |
|
7 | 8 | %b Submit output: |
|
8 | 9 | %input{:type => "file", :name => "file"} |
|
9 | 10 | %input{:type => "submit", :value => "Submit solution"} |
|
10 | 11 | |
|
11 | 12 | .problem-description |
|
12 | 13 | - if problem.description!=nil |
|
13 | 14 | - if problem.description.markdowned |
|
14 | 15 | = markdown(problem.description.body) |
|
15 | 16 | - else |
|
16 | 17 | = problem.description.body |
|
17 | 18 | - else |
@@ -98,12 +98,13 | |||
|
98 | 98 | # VALIDATE_USER_EMAILS = true |
|
99 | 99 | |
|
100 | 100 | # Uncomment so that Apache X-Sendfile is used when delivering files |
|
101 | 101 | # (e.g., in /tasks/view). |
|
102 | 102 | # USE_APACHE_XSENDFILE = true |
|
103 | 103 | |
|
104 | 104 | # Uncomment so that configuration is read only once when the server is loaded |
|
105 | 105 | # Configuration.enable_caching |
|
106 | 106 | |
|
107 | 107 | # OPTIONS FOR CODE JOM |
|
108 | 108 | # -------------------- |
|
109 | 109 | CODEJOM_MAX_ALIVE_LEVEL = 10 |
|
110 | + TEST_ASSIGNMENT_EXPIRATION_DURATION = 5.minute |
@@ -255,12 +255,25 | |||
|
255 | 255 | |
|
256 | 256 | .problem-content { |
|
257 | 257 | float: left; |
|
258 | 258 | margin-left: 10px; |
|
259 | 259 | width: 700px; } |
|
260 | 260 | |
|
261 | 261 | .problem-panel { |
|
262 | 262 | border: 1px black solid; |
|
263 | 263 | padding: 5px; } |
|
264 | 264 | .problem-panel .problem-form { |
|
265 | 265 | border: 1px dotted #99aaee; |
|
266 | 266 | background: #eeeeff; } |
|
267 | + | |
|
268 | + .notice-bar { | |
|
269 | + margin-top: 3px; | |
|
270 | + margin-bottom: 3px; | |
|
271 | + text-align: center; } | |
|
272 | + .notice-bar span.notice { | |
|
273 | + color: white; | |
|
274 | + font-weight: bold; | |
|
275 | + background: #000070; | |
|
276 | + padding: 3px 20px 3px 20px; | |
|
277 | + -moz-border-radius: 2px; | |
|
278 | + -webkit-border-radius: 5px; | |
|
279 | + border-radius: 5px; } |
@@ -301,12 +301,26 | |||
|
301 | 301 | .problem-content |
|
302 | 302 | float: left |
|
303 | 303 | margin-left: 10px |
|
304 | 304 | width: 700px |
|
305 | 305 | |
|
306 | 306 | .problem-panel |
|
307 | 307 | border: 1px black solid |
|
308 | 308 | padding: 5px |
|
309 | 309 | |
|
310 | 310 | .problem-form |
|
311 | 311 | border: 1px dotted #99aaee |
|
312 | 312 | background: #eeeeff |
|
313 | + | |
|
314 | + .notice-bar | |
|
315 | + margin-top: 3px | |
|
316 | + margin-bottom: 3px | |
|
317 | + text-align: center | |
|
318 | + | |
|
319 | + span.notice | |
|
320 | + color: white | |
|
321 | + font-weight: bold | |
|
322 | + background: #000070 | |
|
323 | + padding: 3px 20px 3px 20px | |
|
324 | + -moz-border-radius: 2px | |
|
325 | + -webkit-border-radius: 5px | |
|
326 | + border-radius: 5px |
You need to be logged in to leave comments.
Login now