search

sopel-irc / sopel

:robot::speech_balloon: An easy-to-use and highly extensible IRC Bot framework. Formerly Willie.
https://sopel.chat
Other
951 stars 403 forks source link

Calculate broke #359

Closed lramati closed 10 years ago

lramati commented 10 years ago

[14:05] <Eyore> .c 5/3 [14:05] <Willie> IndexError: list index out of range (file "/home/willie/jenni/willie/modules/calc.py", line 25, in calculate) [14:05] <Eyore> .c 3 [14:05] <Willie> IndexError: list index out of range (file "/home/willie/jenni/willie/modules/calc.py", line 25, in calculate) [14:05] <Eyore> .calc 3 [14:05] <Willie> IndexError: list index out of range (file "/home/willie/jenni/willie/modules/calc.py", line 25, in calculate)

The problem is that the list generated by spliting the result from google no longer has any parts that start with 'rhs: "'. Beyond that I can't determine anything from my phone

embolalia commented 10 years ago

The problem is that iGoogle no longer exists, and that's what .c is hitting (for whatever reason).

embolalia commented 10 years ago

If we can get this fixed in time, I'd love to release 4.1.0 this weekend.

elad661 commented 10 years ago

Time to build something that will work locally and not relay on a third party service for simple calculations.

embolalia commented 10 years ago

We should figure out what all we want to support in this. I'd be fine with using tools.eval_equation (which is new to me, but apparently used in dice) for .c, and let people do anything it can't do with .wa.

tyrope commented 10 years ago

Fixed by 8b12058