search

Steve-V / tgg-BotSteve

Issue tracking and source hosting for the TGG BotSteve module
11 stars 10 forks source link

Remind breaks on very large remind values #96

Closed Steve-V closed 12 years ago

Steve-V commented 12 years ago

(02:28:50 AM) Cprossu: .in 9000000000 years tell Cprossu Should Not See Me (02:28:51 AM) BotSteve: ValueError: (75, 'Value too large for defined data type') (file "/home/steve/src/hg/tgg/hg-botsteve/modules/remind.py", line 104, in remind) (02:28:56 AM) Cprossu: ^ there we go

Steve-V commented 12 years ago

I now have a try-except block in remind to catch the exception, but there is nothing stopping the bad data from going into the storage block.

AstraLuma commented 12 years ago

Should this be considered a bug of the formatting code or of the data validation code? That is, should BotSteve support unreasonable times?

Steve-V commented 12 years ago

Nope!

Fixed by dd9825025a3

Steve-V commented 12 years ago

Nope! Which is why it was fixed in dd98250