[BUG] Standard function pow cannot be executed by an agent

[Bilokin]

Hello,

my steps are failing when the agent tries to invoke the standard pow function with the third argument

Code to reproduce the error

import smolagents
inter = smolagents.LocalPythonInterpreter(additional_authorized_imports=[], tools={})
inter('pow(2,2,2)', {})

Error logs (if any)

InterpreterError: Code execution failed at line 'pow(2,2,2)' due to: TypeError:pow expected 2 arguments, got 3

Expected behavior
A clear and concise description of what you expected to happen.

Packages version:
version 0.1.7 and main

Perhaps there are other standard functions that are affected by this error, a unit test would be nice

[albertvillanova]

The LocalPythonInterpreter uses pow as an alias of math.pow, which only accepts 2 arguments.

Any reason for this choice? @aymeric-roucher

[sysradium]

@Bilokin

smolagents/src/smolagents/local_python_executor.py

Line 88 in dbd1e3c

"pow": math.pow,

But if you want it can be hacked around 🙃:

import smolagents

inter = smolagents.LocalPythonInterpreter(additional_authorized_imports=[], tools={"bpow": pow})
inter("bpow(2,2,2)", {})

The name has to be different as BASE takes precedence here: https://github.com/sysradium/smolagents/blob/main/src/smolagents/local_python_executor.py#L1332

@albertvillanova original math.pow accepts 2 arguments. So to me everything looks good, not a bug 🤔 It is in line with math.sqrt and the rest ...

[Bilokin]

Thanks for the fast reply!
The code I provided is just to reproduce the error, and the real-life error happens when the agent is trying to compute an expression with power operation with 3 arguments in an execution step.

I checked the other functions here and I think only pow is affected, so the fix should be just using the built-in pow instead of math.pow on line 88 as pointed out by @sysradium.

Also, I am curious, why are there math functions in BASE_PYTHON_TOOLS if math needs to be imported like any other package?

[sysradium]

Probably we need to hear from the big man himself, since those "renames" appear to be there from the very inception :) sysradium@f3dcf1f#diff-fee74ba77e514d876543376ff66d78c26eae4ce359b9f3073e3be10fba6e4950R61