cafe-grader-web Commit - r457:c81651eecbbd

cafe-grader-web

Commit `r457:c81651eecbbd` public

parent | child

Description:

add more problem stat

Commit status:

[Not Reviewed]

References:

algo

Diff options:

| | | |

Comments:

0 Commit comments 0 Inline Comments

Unresolved TODOs:

There are no unresolved TODOs

Add another comment

r457:c81651eecbbd - Thu, 25 Sep 2014 18:30:21 - 7 files changed: 144 inserted, 43 deleted

app/views/application/_bar_graph.html.haml created 644 ¶ +44

@@ -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')}

app/controllers/problems_controller.rb ¶ +20 -2

                   @problem.available = !(@problem.available)
                   @problem.save
                 end
                 def turn_all_off
                   Problem.find(:all,
                                :conditions => "available = 1").each do |problem|
                     problem.available = false
                     problem.save
                   end
                   redirect_to :action => 'list'
                 end
                 def turn_all_on
                   Problem.find(:all,
                                :conditions => "available = 0").each do |problem|
                     problem.available = true
                     problem.save
                   end
                   redirect_to :action => 'list'
                 end
                 def stat
                   @problem = Problem.find(params[:id])
-            -     if !@problem.available
+            +     unless @problem.available or session[:admin]
                     redirect_to :controller => 'main', :action => 'list'
-            -     else
+            +       return
+            +     end
                   @submissions = Submission.includes(:user).where(problem_id: params[:id]).order(:user_id,:id)
+            +
+            +     #stat summary
+            +     range =65
+            +     @histogram = { data: Array.new(range,0), summary: {} }
+            +     @histogram[:data] = Array.new(range,0)
+            +     user = Hash.new(0)
+            +     @submissions.find_each do |sub|
+            +       d = (DateTime.now.in_time_zone - sub.submitted_at) / 24 / 60 / 60
+            +       @histogram[:data][d.to_i] += 1 if d < range
+            +       user[sub.user_id] = [user[sub.user_id], (sub.points >= @problem.full_score) ? 1 : 0].max
                   end
+            +     @histogram[:summary][:max] = [@histogram[:data].max,1].max
+            +
+            +     @summary = { attempt: user.count, solve: 0 }
+            +     user.each_value { |v| @summary[:solve] += 1 if v == 1 }
                 end
                 def manage
                   @problems = Problem.find(:all, :order => 'date_added DESC')
                 end
                 def do_manage
                   if params.has_key? 'change_date_added'
                     change_date_added
                   elsif params.has_key? 'add_to_contest'
                     add_to_contest
                   elsif params.has_key? 'enable_problem'
                     set_available(true)
                   elsif params.has_key? 'disable_problem'
                     set_available(false)
                   end
                   redirect_to :action => 'manage'
                 end
                 def import
                   @allow_test_pair_import = allow_test_pair_import?
                 end
                 def do_import
                       p.contests << contest
                     end
                   end
                 end
                 def set_available(avail)
                   problems = get_problems_from_params
                   problems.each do |p|
                     p.available = avail
                     p.save
                   end
                 end
                 def get_problems_from_params
                   problems = []
                   params.keys.each do |k|
                     if k.index('prob-')==0
                       name, id, order = k.split('-')
                       problems << Problem.find(id)
                     end
                   end
                   problems
                 end
+            +   def get_problems_stat
                 end
+            +
+            + end

app/controllers/report_controller.rb ¶ +25 -28

                       else
                         @submissions[s.user_id][:sub][s.problem_id][:sub_ids] << s.id
                       end
                       @submissions[s.user_id][:count] += 1
                     end
                   end
                 end
                 def problem_hof
                   # gen problem list
                   @user = User.find(session[:user_id])
                   @problems = @user.available_problems
                   # get selected problems or the default
                   if params[:id]
                     begin
                       @problem = Problem.available.find(params[:id])
                     rescue
                       redirect_to action: :problem_hof
                       flash[:notice] = 'Error: submissions for that problem are not viewable.'
                       return
                     end
                   end
-            -     if @problem
+            +     return unless @problem
-            -       #aggregrate by language
-            -       @by_lang = {}
+            +     @by_lang = {} #aggregrate by language
+            +
+            +     range =65
+            +     @histogram = { data: Array.new(range,0), summary: {} }
+            +     @histogram[:data] = Array.new(range,0)
+            +     @summary = {count: 0, solve: 0, attempt: 0}
+            +     user = Hash.new(0)
                   Submission.where(problem_id: @problem.id).find_each do |sub|
+            +       #histogram
+            +       d = (DateTime.now.in_time_zone - sub.submitted_at) / 24 / 60 / 60
+            +       @histogram[:data][d.to_i] += 1 if d < range
+            +
+            +       @summary[:count] += 1
+            +       user[sub.user_id] = [user[sub.user_id], (sub.points >= @problem.full_score) ? 1 : 0].max
+            +
                     lang = Language.find_by_id(sub.language_id)
                     next unless lang
                     next unless sub.points >= @problem.full_score
                     #initialize
                     unless @by_lang.has_key?(lang.pretty_name)
                       @by_lang[lang.pretty_name] = {
                         runtime: { avail: false, value: 2**30-1 },
                         memory: { avail: false, value: 2**30-1 },
                         length: { avail: false, value: 2**30-1 },
                         first: { avail: false, value: DateTime.new(3000,1,1) }
                       }
                     end
                     if sub.max_runtime and sub.max_runtime < @by_lang[lang.pretty_name][:runtime][:value]
-            -           @by_lang[lang.pretty_name][:runtime] = {
+            +         @by_lang[lang.pretty_name][:runtime] = { avail: true, user_id: sub.user_id, value: sub.max_runtime, sub_id: sub.id }
-            -             avail: true,
-            -             user_id: sub.user_id,
-            -             value: sub.max_runtime,
-            -             sub_id: sub.id
-            -           }
                     end
                     if sub.peak_memory and sub.peak_memory < @by_lang[lang.pretty_name][:memory][:value]
-            -           @by_lang[lang.pretty_name][:memory] = {
+            +         @by_lang[lang.pretty_name][:memory] = { avail: true, user_id: sub.user_id, value: sub.peak_memory, sub_id: sub.id }
-            -             avail: true,
-            -             user_id: sub.user_id,
-            -             value: sub.peak_memory,
-            -             sub_id: sub.id
-            -           }
                     end
                     if sub.submitted_at and sub.submitted_at < @by_lang[lang.pretty_name][:first][:value] and
                         !sub.user.admin?
-            -           @by_lang[lang.pretty_name][:first] = {
+            +         @by_lang[lang.pretty_name][:first] = { avail: true, user_id: sub.user_id, value: sub.submitted_at, sub_id: sub.id }
-            -             avail: true,
-            -             user_id: sub.user_id,
-            -             value: sub.submitted_at,
-            -             sub_id: sub.id
-            -           }
                     end
                     if @by_lang[lang.pretty_name][:length][:value] > sub.effective_code_length
-            -           @by_lang[lang.pretty_name][:length] = {
+            +         @by_lang[lang.pretty_name][:length] = { avail: true, user_id: sub.user_id, value: sub.effective_code_length, sub_id: sub.id }
-            -             avail: true,
-            -             user_id: sub.user_id,
-            -             value: sub.effective_code_length,
-            -             sub_id: sub.id
-            -           }
                     end
                   end
                   #process user_id
                   @by_lang.each do |lang,prop|
                     prop.each do |k,v|
                       v[:user] = User.exists?(v[:user_id]) ? User.find(v[:user_id]).full_name : "(NULL)"
                     end
                   end
                   #sum into best
                   if @by_lang and @by_lang.first
                     @best = @by_lang.first[1].clone
                     @by_lang.each do |lang,prop|
                       if @best[:runtime][:value] >= prop[:runtime][:value]
                         @best[:runtime] = prop[:runtime]
                         @best[:runtime][:lang] = lang
                       end
                       if @best[:memory][:value] >= prop[:memory][:value]
                         @best[:memory] = prop[:memory]
                         @best[:memory][:lang] = lang
                       end
                       if @best[:length][:value] >= prop[:length][:value]
                         @best[:length] = prop[:length]
                         @best[:length][:lang] = lang
                       end
                       if @best[:first][:value] >= prop[:first][:value]
                         @best[:first] = prop[:first]
                         @best[:first][:lang] = lang
                       end
                     end
                   end
-            -     end
+            +     @histogram[:summary][:max] = [@histogram[:data].max,1].max
+            +     @summary[:attempt] = user.count
+            +     user.each_value { |v| @summary[:solve] += 1 if v == 1 }
                 end
+            +
               end

app/models/problem.rb ¶ +7

                   if not problem.save
                     return problem, 'Error importing'
                   end
                   import_to_db = params.has_key? :import_to_db
                   importer = TestdataImporter.new(problem)
                   if not importer.import_from_file(import_params[:file],
                                                    import_params[:time_limit],
                                                    import_params[:memory_limit],
                                                    import_params[:checker_name],
                                                    import_to_db)
                     problem.errors.add_to_base('Import error.')
                   end
                   return problem, importer.log_msg
                 end
                 def self.download_file_basedir
                   return "#{Rails.root}/data/tasks"
                 end
+            +   def get_submission_stat
+            +     result = Hash.new
+            +     #total number of submission
+            +     result[:total_sub] = Submission.where(problem_id: self.id).count
+            +     result[:attempted_user] = Submission.where(problem_id: self.id).group_by(:user_id)
+            +   end
+            +
                 protected
                 def self.to_i_or_default(st, default)
                   if st!=''
                     result = st.to_i
                   end
                   result ||= default
                 end
                 def self.to_f_or_default(st, default)
                   if st!=''
                     result = st.to_f
                   end
                   result ||= default
                 end
                 def self.extract_params_and_check(params, problem)
                   time_limit = Problem.to_f_or_default(params[:time_limit],
                                                        DEFAULT_TIME_LIMIT)
                   memory_limit = Problem.to_i_or_default(params[:memory_limit],
                                                          DEFAULT_MEMORY_LIMIT)
                   if time_limit<=0 or time_limit >60
                     problem.errors.add_to_base('Time limit out of range.')

app/views/problems/stat.html.haml ¶ +17

               :css
                 .fix-width {
                   font-family: "Consolas, Monaco, Droid Sans Mono,Mono, Monospace,Courier"
                 }
               %h1 Problem stat: #{@problem.name}
               %h2 Overview
+            +
+            + %table.info
+            +   %thead
+            +     %tr.info-head
+            +       %th Stat
+            +       %th Value
+            +   %tbody
+            +     %tr{class: cycle('info-even','info-odd')}
+            +       %td Submissions
+            +       %td= @submissions.count
+            +     %tr{class: cycle('info-even','info-odd')}
+            +       %td Solved/Attempted User
+            +       %td #{@summary[:solve]}/#{@summary[:attempt]} (#{(@summary[:solve]*100.0/@summary[:attempt]).round(1)}%)
+            +
+            + %h2 Submissions Count
+            + = render partial: 'application/bar_graph', locals: { histogram: @histogram }
+            +
               %h2 Submissions
               - if @submissions and @submissions.count > 0
                 %table.info#main_table
                   %thead
                     %tr.info-head
                       %th ID
                       %th Login
                       %th Name
                       %th Submitted_at
                       %th Points
                       %th comment
                   %tbody
                     - row_odd,curr = true,''
                     - @submissions.each do |sub|
                       - next unless sub.user
                       - row_odd,curr = !row_odd, sub.user if curr != sub.user
                       %tr{class: row_odd ? "info-odd" : "info-even"}
                         %td= link_to sub.id, controller: 'graders', action: 'submission', id: sub.id
                         %td= link_to sub.user.login, controller: :users, action: :profile, id: sub.user.id
                         %td= sub.user.full_name
                         %td= time_ago_in_words(sub.submitted_at) + " ago"
                         %td= sub.points
                         %td.fix-width= sub.grader_comment
               - else

app/views/report/_task_hof.html.haml ¶ +28 -11

               :css
                 .hof_user { color: orangered; font-style: italic; }
                 .hof_language { color: green; font-style: italic; }
                 .hof_value { color: deeppink;font-style: italic; }
+            +   .info_param { font-weight: bold;text-align: right; }
-            - %h2 Overall of #{Problem.find(params[:id]).full_name}
+            + %h1 (#{Problem.find(params[:id]).name}) #{Problem.find(params[:id]).full_name}
+            + %h2 Problem Stat
+            + %table.info
+            +   %thead
+            +     %tr.info-head
+            +       %th Stat
+            +       %th Value
+            +   %tbody
+            +     %tr{class: cycle('info-even','info-odd')}
+            +       %td.info_param Submissions
+            +       %td= @summary[:count]
+            +     %tr{class: cycle('info-even','info-odd')}
+            +       %td.info_param Solved/Attempted User
+            +       %td #{@summary[:solve]}/#{@summary[:attempt]} (#{(@summary[:solve]*100.0/@summary[:attempt]).round(1)}%)
                   - if @best
-            -   %b Best Runtime:
+            +       %tr{class: cycle('info-even','info-odd')}
+            +         %td.info_param Best Runtime
+            +         %td
                         by #{link_to @best[:runtime][:user], controller:'users', action:'profile', id:@best[:memory][:user_id]}
                         using <span class="hof_language">#{@best[:runtime][:lang]}</span>
                         with <span class="hof_value">#{@best[:runtime][:value] * 1000} milliseconds</span>
                         at submission
                         = link_to("#" + @best[:runtime][:sub_id].to_s, controller: 'graders', action: 'submission', id:@best[:runtime][:sub_id])
-            -   %br/
-            -   %b Best Memory Usage:
+            +       %tr{class: cycle('info-even','info-odd')}
+            +         %td.info_param Best Memory Usage
+            +         %td
                         by #{link_to @best[:memory][:user], controller:'users', action:'profile', id:@best[:memory][:user_id]}
                         using <span class="hof_language">#{@best[:memory][:lang]}</span>
                         with <span class="hof_value">#{number_with_delimiter(@best[:memory][:value])} kbytes </span>
                         at submission
                         = link_to("#" + @best[:memory][:sub_id].to_s, controller: 'graders' , action: 'submission', id:@best[:memory][:sub_id])
-            -   %br/
-            -   %b Shortest Code:
+            +       %tr{class: cycle('info-even','info-odd')}
+            +         %td.info_param Shortest Code
+            +         %td
                         by #{link_to @best[:length][:user], controller:'users', action:'profile', id:@best[:length][:user_id]}
                         using <span class="hof_language">#{@best[:length][:lang]}</span>
                         with  <span class="hof_value">#{@best[:length][:value]} bytes</span>
                         at submission
                         = link_to("#" + @best[:length][:sub_id].to_s, controller: 'graders' , action: 'submission', id: @best[:length][:sub_id])
-            -   %br/
-            -   %b First solver:
+            +       %tr{class: cycle('info-even','info-odd')}
+            +         %td.info_param First solver
+            +         %td
                         #{link_to @best[:first][:user], controller:'users', action:'profile', id:@best[:first][:user_id]} is the first solver
                         using <span class="hof_language">#{@best[:first][:lang]}</span>
                         on <span class="hof_value">#{@best[:first][:value]}</span>
                         at submission
                         = link_to("#" + @best[:first][:sub_id].to_s, controller: 'graders' , action: 'submission', id: @best[:first][:sub_id])
-            -   %br/
               %p
                 This counts only for submission with 100% score <br/>
                 Right now, java is excluded from memory usage competition. (Because it always uses 2GB memory...)
+            + - if @best
                 %h2 By language
                 %table.info
                   %thead
                     %tr.info-head
                       %th Language
                       %th Best runtime (ms)
                       %th Best memory (kbytes)
                       %th Shortest Code (bytes)
                       %th First solver
                   %tbody
                     - @by_lang.each do |lang,value|
                       %tr{class: cycle('info-even','info-odd')}
                         %td= lang
                         %td
                           = link_to value[:runtime][:user], controller: 'users', action: 'profile', id: value[:runtime][:user_id]
                           = "(#{(value[:runtime][:value] * 1000).to_i} @"
                           = "#{link_to("#" + value[:runtime][:sub_id].to_s, controller: 'graders' , action: 'submission', id: value[:runtime][:sub_id])} )".html_safe
                         %td
                           = link_to value[:memory][:user], controller: 'users', action: 'profile', id: value[:memory][:user_id]
                           = "(#{number_with_delimiter(value[:memory][:value])} @"
                           = "#{link_to("#" + value[:memory][:sub_id].to_s, controller: 'graders' , action: 'submission', id: value[:memory][:sub_id])} )".html_safe
                         %td
                           = link_to value[:length][:user], controller: 'users', action: 'profile', id: value[:length][:user_id]
                           = "(#{value[:length][:value]} @"
                           = "#{link_to("#" + value[:length][:sub_id].to_s, controller: 'graders' , action: 'submission', id: value[:length][:sub_id])} )".html_safe
                         %td
                           - if value[:first][:user] != '(NULL)' #TODO: i know... this is wrong...
                             = link_to value[:first][:user], controller: 'users', action: 'profile', id: value[:first][:user_id]
                             = "(#{value[:first][:value]} @"
                             = "#{link_to("#" + value[:first][:sub_id].to_s, controller: 'graders' , action: 'submission', id: value[:first][:sub_id])} )".html_safe
-            - - else
-            -   %h3 No submissions

app/views/report/problem_hof.html.haml ¶ +3 -2

               /- if params[:id]
               /  %h1 Tasks Hall of Fame
               /  = link_to('[back to All-Time Hall of Fame]', action: 'problem_hof', id: nil  )
               /- else
               /  %h1 All-Time Hall of Fame
-            - %h1 Tasks Hall of Fame
+            + %h1 Hall of Fame
               .task-menu
                 Tasks
                 %br/
                 - @problems.each do |prob|
                   = link_to( "[#{prob.name}]", {id: prob.id})
               - unless params[:id]
                 /=render partial: 'all_time_hof'
                 Please select a problem.
               - else
                 =render partial: 'task_hof'
+            +   %h2 Submission History
+            +   =render partial: 'application/bar_graph', locals: { histogram: @histogram }

Write
Preview

You need to be logged in to leave comments. Login now

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository permissions settings

Sign in to your account

Author

r457:c81651eecbbd - Thu, 25 Sep 2014 18:30:21 - 7 files changed: 144 inserted, 43 deleted

Sign in to your account

Commit r457:c81651eecbbd public

Author

r457:c81651eecbbd - Thu, 25 Sep 2014 18:30:21 - 7 files changed: 144 inserted, 43 deleted

Commit `r457:c81651eecbbd` public