wxpython: Timer object not working, or just me?

[Steve Holden]

John Salerno wrote:
> I'd be curious to know if this works any differently on other 
> computers/platforms or while other things are happening in the 
> background. I can't tell if it's the Timer object that isn't keep 
> accurate time (although a test with time.time() seems to show that it 
> is), or if I'm just messing up my algorithm to fill the progress bar. If 
> I put in 10 seconds, the progress bar will be fully filled at the end, 
> but if you put 30, it doesn't. As the number gets higher, the less the 
> bar gets filled when it's done running. (52 seconds only fills half the 
> bar at the end!)
> 
> Hopefully someone can look at my logic (mostly in the OnTimer method) 
> and see right away how I'm screwing this up.
> 
> Thanks.
> 
> -------------------------
> 
> import wx
> 
> 
> class MyTimer(wx.Frame):
> 
>      def __init__(self):
>          wx.Frame.__init__(self, None, wx.ID_ANY,
>                            style=wx.DEFAULT_FRAME_STYLE ^ wx.RESIZE_BORDER)
>          panel = wx.Panel(self)
> 
>          mainSizer = wx.BoxSizer(wx.VERTICAL)
>          inputSizer = wx.BoxSizer(wx.HORIZONTAL)
> 
>          self.minutesPassed = 1
>          self.progress = wx.Gauge(panel, wx.ID_ANY, 100, size=(300, 20))
>          self.status = wx.StaticText(panel, wx.ID_ANY, 'Enter a time.')
>          prompt = wx.StaticText(panel, wx.ID_ANY, 'Time to wait:')
>          self.input = wx.TextCtrl(panel, wx.ID_ANY, '30', size=(20, 20))
>          self.start = wx.Button(panel, wx.ID_ANY, 'Start')
>          self.reset = wx.Button(panel, wx.ID_ANY, 'Reset')
>          self.reset.Disable()
>          self.timer = wx.Timer(self)
> 
>          mainSizer.Add(self.progress, flag=wx.ALIGN_CENTER | wx.ALL ^ 
> wx.BOTTOM,
>                        border=10)
>          mainSizer.Add(self.status, flag=wx.ALIGN_CENTER | wx.ALL, 
> border=10)
>          mainSizer.Add(inputSizer, flag=wx.ALIGN_CENTER | wx.BOTTOM, 
> border=10)
>          inputSizer.Add(prompt, flag=wx.ALIGN_CENTER)
>          inputSizer.Add(self.input, flag=wx.ALIGN_CENTER | wx.LEFT | 
> wx.RIGHT,
>                         border=5)
>          inputSizer.Add(self.start, flag=wx.ALIGN_CENTER)
>          inputSizer.Add(self.reset, flag=wx.ALIGN_CENTER)
> 
>          self.Bind(wx.EVT_TEXT_ENTER, self.OnStart, self.input)
>          self.Bind(wx.EVT_BUTTON, self.OnStart, self.start)
>          self.Bind(wx.EVT_TIMER, self.OnTimer, self.timer)
>          self.Bind(wx.EVT_BUTTON, self.OnReset, self.reset)
> 
>          panel.SetSizer(mainSizer)
>          mainSizer.Fit(self)
> 
>      def OnStart(self, event):
>          self.time = int(self.input.GetValue())
>          self.countdown = self.time - 1
>          self.start.Disable()
>          self.reset.Enable()
>          self.input.Disable()
>          self.status.SetLabel('%s minutes remaining.' % self.time)
>          self.timer.Start(1000)
> 
>      def OnReset(self, event):
>          self.input.Clear()
>          self.status.SetLabel('Enter a new time.')
>          self.progress.SetValue(0)
>          self.input.Enable()
>          self.minutesPassed = 1
> 
>      def OnTimer(self, event):
>          if self.countdown != 0:
>              #print self.minutesPassed, self.countdown
>              self.progress.SetValue(self.minutesPassed * (100 / self.time))
>              self.status.SetLabel('%s minutes remaining.' % self.countdown)
>              self.minutesPassed += 1
>              self.countdown -= 1
>          else:
>              self.progress.SetValue(self.minutesPassed * (100 / self.time))
>              self.status.SetLabel('%s minutes have elapsed.' % self.time)
> 
> 
> class MyApp(wx.App):
> 
>      def OnInit(self):
>          frame = MyTimer()
>          self.SetTopWindow(frame)
>          frame.Show()
>          return True
> 
> 
> if __name__ == '__main__':
>      app = MyApp(False)
>      app.MainLoop()

Integer arithmetic. Try changing the "100" to "100.0" and you'll see it 
works flawlessly.

regards
  Steve
-- 
Steve Holden       +44 150 684 7255  +1 800 494 3119
Holden Web LLC/Ltd          http://www.holdenweb.com
Skype: holdenweb       http://holdenweb.blogspot.com
Recent Ramblings     http://del.icio.us/steve.holden