Description:
added test pair assignment, requests new input, downloads input
Commit status:
[Not Reviewed]
References:
Comments:
0 Commit comments 0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
Add another comment

r213:805be1d05f2d - - 13 files changed: 179 inserted, 6 deleted

@@ -0,0 +1,5
1 + class TestPairAssignment < ActiveRecord::Base
2 + belongs_to :user
3 + belongs_to :test_pair
4 + belongs_to :problem
5 + end
@@ -0,0 +1,9
1 + class AddNumberToTestPair < ActiveRecord::Migration
2 + def self.up
3 + add_column 'test_pairs', 'number', :integer
4 + end
5 +
6 + def self.down
7 + remove_column 'test_pairs', 'number'
8 + end
9 + end
@@ -0,0 +1,16
1 + class CreateTestPairAssignments < ActiveRecord::Migration
2 + def self.up
3 + create_table :test_pair_assignments do |t|
4 + t.integer "user_id"
5 + t.integer "problem_id"
6 + t.integer "test_pair_id"
7 + t.integer "test_pair_number"
8 + t.integer "request_number"
9 + t.timestamps
10 + end
11 + end
12 +
13 + def self.down
14 + drop_table :test_pair_assignments
15 + end
16 + end
@@ -0,0 +1,9
1 + class AddSubmittedToTestPairAssignment < ActiveRecord::Migration
2 + def self.up
3 + add_column 'test_pair_assignments', 'submitted', :boolean
4 + end
5 +
6 + def self.down
7 + remove_column 'test_pair_assignments', 'submitted'
8 + end
9 + end
@@ -0,0 +1,7
1 + # Read about fixtures at http://ar.rubyonrails.org/classes/Fixtures.html
2 +
3 + # one:
4 + # column: value
5 + #
6 + # two:
7 + # column: value
@@ -0,0 +1,8
1 + require 'test_helper'
2 +
3 + class TestPairAssignmentTest < ActiveSupport::TestCase
4 + # Replace this with your real tests.
5 + test "the truth" do
6 + assert true
7 + end
8 + end
@@ -131,96 +131,127
131 131 end
132 132 prepare_grading_result(@submission)
133 133 end
134 134
135 135 def load_output
136 136 if !Configuration.show_grading_result or params[:num]==nil
137 137 redirect_to :action => 'list' and return
138 138 end
139 139 @user = User.find(session[:user_id])
140 140 @submission = Submission.find(params[:id])
141 141 if @submission.user!=@user
142 142 flash[:notice] = 'You are not allowed to view result of other users.'
143 143 redirect_to :action => 'list' and return
144 144 end
145 145 case_num = params[:num].to_i
146 146 out_filename = output_filename(@user.login,
147 147 @submission.problem.name,
148 148 @submission.id,
149 149 case_num)
150 150 if !FileTest.exists?(out_filename)
151 151 flash[:notice] = 'Output not found.'
152 152 redirect_to :action => 'list' and return
153 153 end
154 154
155 155 response.headers['Content-Type'] = "application/force-download"
156 156 response.headers['Content-Disposition'] = "attachment; filename=\"output-#{case_num}.txt\""
157 157 response.headers["X-Sendfile"] = out_filename
158 158 response.headers['Content-length'] = File.size(out_filename)
159 159 render :nothing => true
160 160 end
161 161
162 162 def error
163 163 @user = User.find(session[:user_id])
164 164 end
165 165
166 166 # announcement refreshing and hiding methods
167 167
168 168 def announcements
169 169 if params.has_key? 'recent'
170 170 prepare_announcements(params[:recent])
171 171 else
172 172 prepare_announcements
173 173 end
174 174 render(:partial => 'announcement',
175 175 :collection => @announcements,
176 176 :locals => {:announcement_effect => true})
177 177 end
178 178
179 + # actions for Code Jom
180 + def new_input
181 + problem = Problem.find(params[:id])
182 + user = User.find(session[:user_id])
183 + if user.can_request_new_test_pair_for? problem
184 + assignment = user.get_new_test_pair_assignment_for problem
185 + assignment.save
186 +
187 + send_data(assignment.test_pair.input,
188 + { :filename => "#{problem.name}-#{assignment.request_number}.in",
189 + :type => 'text/plain' })
190 + else
191 + flash[:notice] = 'You cannot request new input now.'
192 + redirect_to :action => 'list'
193 + end
194 + end
195 +
196 + def download
197 + problem = Problem.find(params[:id])
198 + user = User.find(session[:user_id])
199 + recent_assignment = user.get_recent_test_pair_assignment_for problem
200 + if recent_assignment != nil
201 + send_data(recent_assignment.test_pair.input,
202 + { :filename => "#{problem.name}-#{recent_assignment.request_number}.in",
203 + :type => 'text/plain' })
204 + else
205 + flash[:notice] = 'You have not request for any input data for this problem.'
206 + redirect_to :action => 'list'
207 + end
208 + end
209 +
179 210 protected
180 211
181 212 def prepare_announcements(recent=nil)
182 213 if Configuration.show_tasks_to?(@user)
183 214 @announcements = Announcement.find_published(true)
184 215 else
185 216 @announcements = Announcement.find_published
186 217 end
187 218 if recent!=nil
188 219 recent_id = recent.to_i
189 220 @announcements = @announcements.find_all { |a| a.id > recent_id }
190 221 end
191 222 end
192 223
193 224 def prepare_list_information
194 225 @problems = Problem.find_available_problems
195 226 @prob_submissions = Array.new
196 227 @user = User.find(session[:user_id])
197 228 @problems.each do |p|
198 229 sub = Submission.find_last_by_user_and_problem(@user.id,p.id)
199 230 if sub!=nil
200 231 @prob_submissions << { :count => sub.number, :submission => sub }
201 232 else
202 233 @prob_submissions << { :count => 0, :submission => nil }
203 234 end
204 235 end
205 236 prepare_announcements
206 237 end
207 238
208 239 def check_viewability
209 240 @user = User.find(session[:user_id])
210 241 if (!Configuration.show_tasks_to?(@user)) and
211 242 ((action_name=='submission') or (action_name=='submit'))
212 243 redirect_to :action => 'list' and return
213 244 end
214 245 end
215 246
216 247 def prepare_grading_result(submission)
217 248 if Configuration.task_grading_info.has_key? submission.problem.name
218 249 grading_info = Configuration.task_grading_info[submission.problem.name]
219 250 else
220 251 # guess task info from problem.full_score
221 252 cases = submission.problem.full_score / 10
222 253 grading_info = {
223 254 'testruns' => cases,
224 255 'testcases' => cases
225 256 }
226 257 end
@@ -1,60 +1,75
1 1 class Problem < ActiveRecord::Base
2 2
3 3 belongs_to :description
4 4 has_many :test_pairs, :dependent => :delete_all
5 5
6 6 validates_presence_of :name
7 7 validates_format_of :name, :with => /^\w+$/
8 8 validates_presence_of :full_name
9 9
10 10 DEFAULT_TIME_LIMIT = 1
11 11 DEFAULT_MEMORY_LIMIT = 32
12 12
13 + def test_pair_count
14 + @test_pair_count ||= test_pairs.size
15 + end
16 +
17 + def uses_random_test_pair?
18 + test_pair_count != 0
19 + end
20 +
21 + def random_test_pair(forbidden_numbers=nil)
22 + begin
23 + test_num = 1 + rand(test_pair_count)
24 + end while forbidden_numbers!=nil and forbidden_numbers.include? test_num
25 + test_pairs.find_by_number test_num
26 + end
27 +
13 28 def self.find_available_problems
14 29 find(:all, :conditions => {:available => true}, :order => "date_added DESC")
15 30 end
16 31
17 32 def self.create_from_import_form_params(params, old_problem=nil)
18 33 problem = old_problem || Problem.new
19 34 import_params = Problem.extract_params_and_check(params, problem)
20 35
21 36 if not problem.valid?
22 37 return problem, 'Error importing'
23 38 end
24 39
25 40 problem.full_score = 100
26 41 problem.date_added = Time.new
27 42 problem.test_allowed = true
28 43 problem.output_only = false
29 44 problem.available = false
30 45
31 46 if not problem.save
32 47 return problem, 'Error importing'
33 48 end
34 49
35 50 import_to_db = params.has_key? :import_to_db
36 51
37 52 importer = TestdataImporter.new(problem)
38 53
39 54 if not importer.import_from_file(import_params[:file],
40 55 import_params[:time_limit],
41 56 import_params[:memory_limit],
42 57 import_to_db)
43 58 problem.errors.add_to_base('Import error.')
44 59 end
45 60
46 61 return problem, importer.log_msg
47 62 end
48 63
49 64 protected
50 65
51 66 def self.to_i_or_default(st, default)
52 67 if st!=''
53 68 st.to_i
54 69 else
55 70 default
56 71 end
57 72 end
58 73
59 74 def self.extract_params_and_check(params, problem)
60 75 time_limit = Problem.to_i_or_default(params[:time_limit],
@@ -1,3 +1,9
1 + # TestPair stores an input-solution pair for a problem. This is used
2 + # in a certain "test-pair"-type problem for the CodeJom competition
3 + # which follows the Google Code Jam format, i.e., a participant only
4 + # submits a solution to a single random input that the participant
5 + # requested. This input-solution pair is a TestPair.
6 +
1 7 class TestPair < ActiveRecord::Base
2 - belongs_to :problem
8 + belongs_to :problem
3 9 end
@@ -1,117 +1,161
1 1 require 'digest/sha1'
2 2
3 3 class User < ActiveRecord::Base
4 4
5 5 has_and_belongs_to_many :roles
6 6
7 7 has_many :test_requests, :order => "submitted_at DESC"
8 8
9 9 has_many :messages,
10 10 :class_name => "Message",
11 11 :foreign_key => "sender_id",
12 12 :order => 'created_at DESC'
13 13
14 14 has_many :replied_messages,
15 15 :class_name => "Message",
16 16 :foreign_key => "receiver_id",
17 17 :order => 'created_at DESC'
18 18
19 + has_many :test_pair_assignments, :dependent => :delete_all
20 +
19 21 belongs_to :site
20 22 belongs_to :country
21 23
22 24 named_scope :activated_users, :conditions => {:activated => true}
23 25
24 26 validates_presence_of :login
25 27 validates_uniqueness_of :login
26 28 validates_format_of :login, :with => /^[\_A-Za-z0-9]+$/
27 29 validates_length_of :login, :within => 3..30
28 30
29 31 validates_presence_of :full_name
30 32 validates_length_of :full_name, :minimum => 1
31 33
32 34 validates_presence_of :password, :if => :password_required?
33 35 validates_length_of :password, :within => 4..20, :if => :password_required?
34 36 validates_confirmation_of :password, :if => :password_required?
35 37
36 38 validates_format_of :email,
37 39 :with => /\A([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})\Z/i,
38 40 :if => :email_validation?
39 41 validate :uniqueness_of_email_from_activated_users,
40 42 :if => :email_validation?
41 43 validate :enough_time_interval_between_same_email_registrations,
42 44 :if => :email_validation?
43 45
44 46 # these are for ytopc
45 47 # disable for now
46 48 #validates_presence_of :province
47 49
48 50 attr_accessor :password
49 51
50 52 before_save :encrypt_new_password
51 53 before_save :assign_default_site
52 54
53 55 def self.authenticate(login, password)
54 56 user = find_by_login(login)
55 57 return user if user && user.authenticated?(password)
56 58 end
57 59
58 60 def authenticated?(password)
59 61 if self.activated
60 62 hashed_password == User.encrypt(password,self.salt)
61 63 else
62 64 false
63 65 end
64 66 end
65 67
66 68 def admin?
67 69 self.roles.detect {|r| r.name == 'admin' }
68 70 end
69 71
72 + # These are methods related to test pairs
73 +
74 + def get_test_pair_assignments_for(problem)
75 + test_pair_assignments.find_all { |a| a.problem_id == problem.id }
76 + end
77 +
78 + def get_recent_test_pair_assignment_for(problem)
79 + assignments = get_test_pair_assignments_for problem
80 + if assignments.length == 0
81 + return nil
82 + else
83 + recent = assignments[0]
84 + assignments.each do |a|
85 + recent = a if a.request_number > recent.request_number
86 + end
87 + return recent
88 + end
89 + end
90 +
91 + def can_request_new_test_pair_for?(problem)
92 + recent = get_recent_test_pair_assignment_for problem
93 + return (recent == nil or recent.submitted)
94 + end
95 +
96 + def get_new_test_pair_assignment_for(problem)
97 + previous_assignment_numbers =
98 + get_test_pair_assignments_for(problem).collect {|a| a.test_pair_number }
99 + test_pair = problem.random_test_pair(previous_assignment_numbers)
100 + if test_pair
101 + assignment = TestPairAssignment.new(:user => self,
102 + :problem => problem,
103 + :test_pair => test_pair,
104 + :test_pair_number => test_pair.number,
105 + :request_number =>
106 + previous_assignment_numbers.length + 1,
107 + :submitted => false)
108 + return assignment
109 + else
110 + return nil
111 + end
112 + end
113 +
70 114 def email_for_editing
71 115 if self.email==nil
72 116 "(unknown)"
73 117 elsif self.email==''
74 118 "(blank)"
75 119 else
76 120 self.email
77 121 end
78 122 end
79 123
80 124 def email_for_editing=(e)
81 125 self.email=e
82 126 end
83 127
84 128 def alias_for_editing
85 129 if self.alias==nil
86 130 "(unknown)"
87 131 elsif self.alias==''
88 132 "(blank)"
89 133 else
90 134 self.alias
91 135 end
92 136 end
93 137
94 138 def alias_for_editing=(e)
95 139 self.alias=e
96 140 end
97 141
98 142 def activation_key
99 143 if self.hashed_password==nil
100 144 encrypt_new_password
101 145 end
102 146 Digest::SHA1.hexdigest(self.hashed_password)[0..7]
103 147 end
104 148
105 149 def verify_activation_key(key)
106 150 key == activation_key
107 151 end
108 152
109 153 def self.random_password(length=5)
110 154 chars = 'abcdefghjkmnopqrstuvwxyz'
111 155 password = ''
112 156 length.times { password << chars[rand(chars.length - 1)] }
113 157 password
114 158 end
115 159
116 160 def self.find_non_admin_with_prefix(prefix='')
117 161 users = User.find(:all)
@@ -1,18 +1,26
1 1 <tr class="info-<%= (problem_counter%2==0) ? "even" : "odd" %>">
2 2 <td>
3 3 <%= "#{problem_counter+1}" %>
4 4 </td>
5 5 <td>
6 6 <%= "#{problem.full_name} (#{problem.name})" %>
7 7 <%= link_to "[#{t 'main.problem_desc'}]", problem.url, :popup => true if (problem.url!=nil) and (problem.url!='') %>
8 8 </td>
9 9 <td align="center">
10 10 <%= @prob_submissions[problem_counter][:count] %>
11 11 </td>
12 12 <td>
13 - <%= render :partial => 'submission_short',
14 - :locals => {
15 - :submission => @prob_submissions[problem_counter][:submission],
16 - :problem_name => problem.name }%>
13 + <span id="problem-form-<%= problem.id %>">
14 + <% form_tag "new_input/#{problem.id}", :method => :post do -%>
15 + <input type="submit" value="New input"/>
16 + <% end -%>
17 + <% form_tag "download/#{problem.id}", :method => :post do -%>
18 + <input type="submit" value="Download input"/>
19 + <% end -%>
20 + <% form_tag "submit_solution/#{problem.id}", :method => :post do -%>
21 + <input type="file">
22 + <input type="submit" value="Submit solution"/>
23 + <% end -%>
24 + </span>
17 25 </td>
18 26 </tr>
@@ -1,60 +1,60
1 1 # This file is auto-generated from the current state of the database. Instead of editing this file,
2 2 # please use the migrations feature of Active Record to incrementally modify your database, and
3 3 # then regenerate this schema definition.
4 4 #
5 5 # Note that this schema.rb definition is the authoritative source for your database schema. If you need
6 6 # to create the application database on another system, you should be using db:schema:load, not running
7 7 # all the migrations from scratch. The latter is a flawed and unsustainable approach (the more migrations
8 8 # you'll amass, the slower it'll run and the greater likelihood for issues).
9 9 #
10 10 # It's strongly recommended to check this file into your version control system.
11 11
12 - ActiveRecord::Schema.define(:version => 20100113094740) do
12 + ActiveRecord::Schema.define(:version => 20100118174404) do
13 13
14 14 create_table "announcements", :force => true do |t|
15 15 t.string "author"
16 16 t.text "body"
17 17 t.boolean "published"
18 18 t.datetime "created_at"
19 19 t.datetime "updated_at"
20 20 t.boolean "frontpage", :default => false
21 21 t.boolean "contest_only", :default => false
22 22 t.string "title"
23 23 end
24 24
25 25 create_table "configurations", :force => true do |t|
26 26 t.string "key"
27 27 t.string "value_type"
28 28 t.string "value"
29 29 t.datetime "created_at"
30 30 t.datetime "updated_at"
31 31 end
32 32
33 33 create_table "countries", :force => true do |t|
34 34 t.string "name"
35 35 t.datetime "created_at"
36 36 t.datetime "updated_at"
37 37 end
38 38
39 39 create_table "descriptions", :force => true do |t|
40 40 t.text "body"
41 41 t.boolean "markdowned"
42 42 t.datetime "created_at"
43 43 t.datetime "updated_at"
44 44 end
45 45
46 46 create_table "grader_processes", :force => true do |t|
47 47 t.string "host", :limit => 20
48 48 t.integer "pid"
49 49 t.string "mode"
50 50 t.boolean "active"
51 51 t.datetime "created_at"
52 52 t.datetime "updated_at"
53 53 t.integer "task_id"
54 54 t.string "task_type"
55 55 t.boolean "terminated"
56 56 end
57 57
58 58 add_index "grader_processes", ["host", "pid"], :name => "index_grader_processes_on_ip_and_pid"
59 59
60 60 create_table "languages", :force => true do |t|
@@ -110,95 +110,107
110 110
111 111 add_index "roles_users", ["user_id"], :name => "index_roles_users_on_user_id"
112 112
113 113 create_table "sessions", :force => true do |t|
114 114 t.string "session_id"
115 115 t.text "data"
116 116 t.datetime "updated_at"
117 117 end
118 118
119 119 add_index "sessions", ["session_id"], :name => "index_sessions_on_session_id"
120 120 add_index "sessions", ["updated_at"], :name => "index_sessions_on_updated_at"
121 121
122 122 create_table "sites", :force => true do |t|
123 123 t.string "name"
124 124 t.boolean "started"
125 125 t.datetime "start_time"
126 126 t.datetime "created_at"
127 127 t.datetime "updated_at"
128 128 t.integer "country_id"
129 129 t.string "password"
130 130 end
131 131
132 132 create_table "submissions", :force => true do |t|
133 133 t.integer "user_id"
134 134 t.integer "problem_id"
135 135 t.integer "language_id"
136 136 t.text "source"
137 137 t.binary "binary"
138 138 t.datetime "submitted_at"
139 139 t.datetime "compiled_at"
140 140 t.text "compiler_message"
141 141 t.datetime "graded_at"
142 142 t.integer "points"
143 143 t.text "grader_comment"
144 144 t.integer "number"
145 145 t.string "source_filename"
146 146 end
147 147
148 148 add_index "submissions", ["user_id", "problem_id", "number"], :name => "index_submissions_on_user_id_and_problem_id_and_number", :unique => true
149 149 add_index "submissions", ["user_id", "problem_id"], :name => "index_submissions_on_user_id_and_problem_id"
150 150
151 151 create_table "tasks", :force => true do |t|
152 152 t.integer "submission_id"
153 153 t.datetime "created_at"
154 154 t.integer "status"
155 155 t.datetime "updated_at"
156 156 end
157 157
158 + create_table "test_pair_assignments", :force => true do |t|
159 + t.integer "user_id"
160 + t.integer "problem_id"
161 + t.integer "test_pair_id"
162 + t.integer "test_pair_number"
163 + t.integer "request_number"
164 + t.datetime "created_at"
165 + t.datetime "updated_at"
166 + t.boolean "submitted"
167 + end
168 +
158 169 create_table "test_pairs", :force => true do |t|
159 170 t.integer "problem_id"
160 171 t.text "input"
161 172 t.text "solution"
162 173 t.datetime "created_at"
163 174 t.datetime "updated_at"
175 + t.integer "number"
164 176 end
165 177
166 178 create_table "test_requests", :force => true do |t|
167 179 t.integer "user_id"
168 180 t.integer "problem_id"
169 181 t.integer "submission_id"
170 182 t.string "input_file_name"
171 183 t.string "output_file_name"
172 184 t.string "running_stat"
173 185 t.integer "status"
174 186 t.datetime "updated_at"
175 187 t.datetime "submitted_at"
176 188 t.datetime "compiled_at"
177 189 t.text "compiler_message"
178 190 t.datetime "graded_at"
179 191 t.string "grader_comment"
180 192 t.datetime "created_at"
181 193 t.float "running_time"
182 194 t.string "exit_status"
183 195 t.integer "memory_usage"
184 196 end
185 197
186 198 add_index "test_requests", ["user_id", "problem_id"], :name => "index_test_requests_on_user_id_and_problem_id"
187 199
188 200 create_table "users", :force => true do |t|
189 201 t.string "login", :limit => 50
190 202 t.string "full_name"
191 203 t.string "hashed_password"
192 204 t.string "salt", :limit => 5
193 205 t.string "alias"
194 206 t.string "email"
195 207 t.integer "site_id"
196 208 t.integer "country_id"
197 209 t.boolean "activated", :default => false
198 210 t.datetime "created_at"
199 211 t.datetime "updated_at"
200 212 end
201 213
202 214 add_index "users", ["login"], :name => "index_users_on_login", :unique => true
203 215
204 216 end
@@ -54,81 +54,84
54 54 Dir.mkdir extract_dir
55 55 rescue Errno::EEXIST
56 56 end
57 57
58 58 if ext=='.tar.gz' or ext=='.tgz'
59 59 cmd = "tar -zxvf #{testdata_filename} -C #{extract_dir}"
60 60 elsif ext=='.tar'
61 61 cmd = "tar -xvf #{testdata_filename} -C #{extract_dir}"
62 62 elsif ext=='.zip'
63 63 cmd = "unzip -o #{testdata_filename} -d #{extract_dir}"
64 64 else
65 65 return nil
66 66 end
67 67
68 68 system(cmd)
69 69
70 70 files = Dir["#{extract_dir}/**/*1*.in"]
71 71 return nil if files.length==0
72 72
73 73 return File.dirname(files[0])
74 74 end
75 75
76 76 def save_testdata_file(tempfile)
77 77 ext = TestdataImporter.long_ext(tempfile.original_filename)
78 78 testdata_filename = File.join(Dir.tmpdir,"#{@problem.name}#{ext}")
79 79
80 80 return nil if tempfile==""
81 81
82 82 if tempfile.instance_of?(Tempfile)
83 83 tempfile.close
84 84 FileUtils.move(tempfile.path,testdata_filename)
85 85 else
86 86 File.open(testdata_filename, "wb") do |f|
87 87 f.write(tempfile.read)
88 88 end
89 89 end
90 90
91 91 return testdata_filename
92 92 end
93 93
94 94 def import_test_pairs(dirname)
95 95 test_num = 1
96 96 while FileTest.exists? "#{dirname}/#{test_num}.in"
97 97 in_filename = "#{dirname}/#{test_num}.in"
98 98 sol_filename = "#{dirname}/#{test_num}.sol"
99 99
100 100 break if not FileTest.exists? sol_filename
101 101
102 + puts "#{dirname}"
103 +
102 104 test_pair = TestPair.new(:input => open(in_filename).read,
103 105 :solution => open(sol_filename).read,
106 + :number => test_num,
104 107 :problem => @problem)
105 108 break if not test_pair.save
106 109
107 110 test_num += 1
108 111 end
109 112 return test_num > 1
110 113 end
111 114
112 115 def import_problem_description(dirname)
113 116 html_files = Dir["#{dirname}/*.html"]
114 117 markdown_files = Dir["#{dirname}/*.md"] + Dir["#{dirname}/*.markdown"]
115 118 if (html_files.length != 0) or (markdown_files.length != 0)
116 119 description = @problem.description || Description.new
117 120
118 121 if html_files.length != 0
119 122 filename = html_files[0]
120 123 description.markdowned = false
121 124 else
122 125 filename = markdown_files[0]
123 126 description.markdowned = true
124 127 end
125 128
126 129 description.body = open(filename).read
127 130 description.save
128 131 @problem.description = description
129 132 @problem.save
130 133 return "\nProblem description imported from #{filename}."
131 134 end
132 135 end
133 136
134 137 end
You need to be logged in to leave comments. Login now