Skip to content

Commit b2f1232

Browse files
Remove smolagents installation from remote executors (#1274)
1 parent c9513b9 commit b2f1232

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/smolagents/remote_executors.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ def send_tools(self, tools: dict[str, Tool]):
5959
pkg
6060
for tool in tools.values()
6161
for pkg in tool.to_dict()["requirements"]
62-
if pkg not in self.installed_packages
62+
if pkg not in self.installed_packages + ["smolagents"]
6363
}
6464
if packages_to_install:
6565
self.installed_packages.extend(packages_to_install)
@@ -90,9 +90,9 @@ def __call__(self, code_action: str) -> tuple[Any, str, bool]:
9090
return output[0], output[1], is_final_answer
9191

9292
def install_packages(self, additional_imports: list[str]):
93-
additional_imports = additional_imports + ["smolagents"]
94-
_, execution_logs = self.run_code_raise_errors(f"!pip install {' '.join(additional_imports)}")
95-
self.logger.log(execution_logs)
93+
if additional_imports:
94+
_, execution_logs = self.run_code_raise_errors(f"!pip install {' '.join(additional_imports)}")
95+
self.logger.log(execution_logs)
9696
return additional_imports
9797

9898

0 commit comments

Comments
 (0)