Updated the "votes_per_day" method in graph controller to work better and
author<jlsharps@mit.edu> <>
Thu, 9 Aug 2007 02:03:49 +0000 (22:03 -0400)
committer<jlsharps@mit.edu> <>
Thu, 9 Aug 2007 02:03:49 +0000 (22:03 -0400)
read easier. Also, with Mako's insistence and help, wrote the "votes_per_interval" method (and helpers) in graph controller. The worst part of that
method is when I assign names to the labels_hash, ideas welcome, but it
works for now. Even though the graph uses fixed intervals and is named
"votes_per_interval" it actually shows votes over time (per interval),
so I may rename it in the near future (in fact I would've renamed it for
this commit, but I didn't think till I was writing this statement.)

app/controllers/graph_controller.rb
app/controllers/quickvote_controller.rb
app/controllers/user_controller.rb [deleted file]
app/models/quick_vote.rb
app/views/quickvote/results.rhtml

index ebc5da7c8f066535509028632a22cc408425940c..4f33ac3a3f7b02d212a7362ca9b26f10d0e22f3a 100644 (file)
@@ -1,12 +1,12 @@
+require 'date'
 class GraphController < ApplicationController
-
   # produce a graph of votes per day during an election
   def votes_per_day
-    
     @election = Election.find(params[:id])
     data, labels = get_votes_per_day_data(@election)
-
-    line = Gruff::Line.new
+    
+    line = Gruff::Line.new("700x400")
+    line.theme = { :background_colors => ['#73BF26', '#ffffff'] }
     line.title = "Voters Per Day"
     line.font = File.expand_path('/usr/X11R6/lib/X11/fonts/TTF/Vera.ttf',
                                  RAILS_ROOT)
@@ -22,40 +22,120 @@ class GraphController < ApplicationController
     send_data(line.to_blob, :disposition => 'inline', :type => 'image/png')
   end
  
+  def votes_per_interval
+    @election = Election.find(params[:id])
+    data, labels = get_votes_per_interval_data(@election)
+    
+    line = Gruff::Line.new("700x400")
+    line.theme = { :background_colors => ['#73BF26', '#ffffff'] }
+    line.title = "Voters Over Time"
+    line.font = File.expand_path('/usr/X11R6/lib/X11/fonts/TTF/Vera.ttf',
+                               RAILS_ROOT)
+    
+    line.data("#{@election.name}", data )
+    line.labels = labels
+    
+    line.x_axis_label = "Intervals"
+    line.y_axis_label = "Number of Votes"
+    line.minimum_value = 0.0
+                               
+    line.draw
+    send_data(line.to_blob, :disposition => 'inline', :type => 'image/png')  
+  end
+ private 
   # generate the data and labels for each graph
   def get_votes_per_day_data(election)
-
-    voter_times = Array.new
+    voter_days = Array.new
     unique_days = Array.new
-    voters_per_day = Array.new
-    days_hash = Hash.new
+    total_per_day = Array.new
+    election_days = Hash.new
     
+    #turn election startdate into date object, and create the range of election
+    startdate = Date.parse(election.startdate.to_s)
+    election_range = startdate..Date.today
+    
+    # create a hash with all the dates of the election in String format
+    # referenced by their order in the election
+    election_range.each_with_index do |day, index|
+      election_days[index] = day.to_s
+    end
+    
+    # Now I need to create an array with all the times votes were made
     election.votes.each do |vote|
-      unless voter_times.any? {|utime| utime == vote.time}
-        voter_times << vote.time
-      end
+        voter_days << Date.parse(vote.time.to_s)
     end
+    voter_days.sort!
     
-    voter_times.sort!
-  
-    voter_times.each_with_index do |time, index| 
-      count = 1
-      # TODO: add comment
-      unless unique_days.any? { |d1| d1.eql?(time.mon.to_s + "/" + time.day.to_s) }
-        unique_days << (time.mon.to_s + "/" + time.day.to_s)
-        count += (voter_times[(index+1)..-1].find_all \
-          {|t| t.mon == time.mon && t.day == time.day}).size
-        voters_per_day << count
-      end      
+    # Now I need to count how many times each each date appears in voter_days,
+    # and put that number into a votes_per_day array, the 'data' for the graph    
+    #Create an array of unique days from voter_days
+    voter_days.each do |day|
+      unless unique_days.any? {|date| date.eql?(day)}
+        unique_days << day
+      end
     end
+    unique_days.sort!
     
-    unique_days.each_with_index do |fmtdate, index|
-      days_hash[index] = fmtdate
+    #find all dates where those days = date at current index, put size of returned
+    #array into total_per_day
+    unique_days.each_with_index do |date, index|
+      total_per_day << (voter_days.select {|day| day.eql?(date)}).size
     end    
 
     # return the data and the labels
-    return voters_per_day, days_hash
+    return total_per_day, election_days
    
   end
+  
+  def get_votes_per_interval_data(election)
+    labels_hash = Hash.new
+    buckets = Hash.new
+    total_per_interval = Array.new
+    
+    starttime = election.startdate
+    timedelta = Time.now - starttime
+    numcols = 10
+    interval_length = timedelta/numcols
+    
+    # Make a hash, buckets, indexed by time intervals and containing empty arrays
+    # The time object must come first in addition! 
+    # i would start at 0, i+1 goes from 0 up till numcols
+    numcols.times {|i| buckets[starttime + ((i+1)*interval_length)] = []}
+     
+    # Put votes into bucket according to the time interval to which they belong,
+    # referenced by their key
+    # Will build a graph over time, as each successive interval wil lhave more
+    # vote objects  
+    election.votes.each do |vote|
+      buckets.keys.sort.each do |inter|
+        if vote.time < inter
+          buckets[inter] << vote
+        end
+      end
+    end
+  
+    total_per_interval = buckets.keys.sort.collect {|key| buckets[key].size}
+    
+    # Create the hash for the labels. Each graph has ten columns, and three
+    # will be labeled
+    if timedelta < 2.hours #under two hours use minutes for labels
+      labels_hash[0] = starttime.min.to_s
+      labels_hash[(numcols/2)-1] = (starttime + (timedelta/2)).min.to_s
+      labels_hash[numcols-1] = Time.now.min.to_s
+    elsif timedelta < 2.days #more than 2 hours means use hours for labels
+      labels_hash[0] = starttime.hour.to_s
+      labels_hash[(numcols/2)-1] = (starttime + (timedelta/2)).hour.to_s
+      labels_hash[numcols-1] = Time.now.hour.to_s
+    else #more than 2 days means use dates for labels
+      labels_hash[0] = (Date.parse(starttime.to_s)).to_s
+      labels_hash[(numcols/2)-1] = (Date.parse(starttime + (timedelta/2))).to_s
+      labels_hash[numcols-1] = (Date.today).to_s
+    end
+    
+    # Make sure to return an array for data and hash for labels
+    return total_per_interval, labels_hash   
+  end
 
 end
index c7e52d6b12fc5d9040629bd183069cc9b157b8ff..d842f63433cfec8f389b45904a6507f193975ec0 100644 (file)
@@ -12,7 +12,7 @@ class QuickvoteController < ApplicationController
   def create
     if params[:quickvote] 
       @quickvote = QuickVote.new(params[:quickvote])
-
+     
       # store the candidate grabbed through ajax and stored in flash
       @quickvote.candidatelist = flash[:candlist]
 
diff --git a/app/controllers/user_controller.rb b/app/controllers/user_controller.rb
deleted file mode 100644 (file)
index 11cf095..0000000
+++ /dev/null
@@ -1,8 +0,0 @@
-class UserController < ApplicationController
-  layout 'main'
-
-  def home
-    redirect_to :controller => 'site', :action => 'index'
-  end
-
-end
index 8d4e6e82141f8b8c67dab67d0103c170a97f0324..eb4e63c421f32a4381a5ae1645730c2f9f912772 100644 (file)
@@ -16,6 +16,7 @@ class QuickVote < Election
   
   def initialize(params={})
     super
+    self.startdate = Time.now
     self.enddate =  Time.now + 30.days
     self.active = 1
     self.anonymous = 1
index e06b5d6b32b91dc855cc2ee46cb46c135457d61b..030d2467d925d6d49282e988cdd1bbdf1e79987f 100644 (file)
@@ -162,4 +162,8 @@ by several other names.</p>
 <% end %>
 </table>
 
-<%= image_tag( graph_url( :action => 'votes_per_day', :id => @election ) ) %>
+<%= image_tag( graph_url( :action => 'votes_per_day', :id => @election ) ) %><br />
+<%= image_tag( graph_url(:action => 'votes_per_interval', :id => @election)) %>
+
+
+

Benjamin Mako Hill || Want to submit a patch?