modify app

app.py

@@ -1,57 +1,104 @@
 import os
 import gradio as gr
 import requests
+import inspect
 import pandas as pd
-from tools import youtube_tool
-from smolagents import DuckDuckGoSearchTool, CodeAgent, WikipediaSearchTool, InferenceClientModel, GoogleSearchTool, \
-    FinalAnswerTool, ToolCallingAgent
+from smolagents import CodeAgent, InferenceClientModel, ToolCallingAgent
+from smolagents import GoogleSearchTool, VisitWebpageTool, WikipediaSearchTool
+from tools import FinalAnswerTool, get_current_time, calculate_basic_math, save_note, list_saved_notes
+
 
 # (Keep Constants as is)
 # --- Constants ---
 DEFAULT_API_URL = "https://agents-course-unit4-scoring.hf.space"
-serpapi_key = os.environ.get("SERPAPI_KEY")
-
-tool = [
-    GoogleSearchTool(provider="serpapi"),
-    WikipediaSearchTool(),
-    youtube_tool,
-    FinalAnswerTool()
-]
 
-my_instructions = (
-    "You are a general AI assistant. I will ask you a question. Report your thoughts, and finish your answer with the following template: FINAL ANSWER: [YOUR FINAL ANSWER]. YOUR FINAL ANSWER should be a number OR as few words as possible OR a comma separated list of numbers and/or strings. If you are asked for a number, don't use comma to write your number neither use units such as $ or percent sign unless specified otherwise. If you are asked for a string, don't use articles, neither abbreviations (e.g. for cities), and write the digits in plain text unless specified otherwise. If you are asked for a comma separated list, apply the above rules depending of whether the element to be put in the list is a number or a string."
-)
 # --- Basic Agent Definition ---
 # ----- THIS IS WERE YOU CAN BUILD WHAT YOU WANT ------
+
+final_answer = FinalAnswerTool()
+
+manager_model = InferenceClientModel(
+    model_id='deepseek-ai/DeepSeek-R1',  # it is possible that this model may be overloaded
+    # max_tokens=8096,
+    provider="together",
+    custom_role_conversions=None,
+)
+
+agent_model = InferenceClientModel(
+    model_id='Qwen/Qwen2.5-Coder-32B-Instruct',  # it is possible that this model may be overloaded
+    provider="together",
+    max_tokens=8096,
+    custom_role_conversions=None,
+)
+
+web_agent = ToolCallingAgent(
+    model=agent_model,
+    tools=[
+        GoogleSearchTool("serper"), VisitWebpageTool()
+    ],
+    name="web_agent",
+    description="Browses the web to find information",
+    max_steps=10,
+)
+
+managed_web_agent = CodeAgent(
+    model=agent_model,
+    managed_agents=[web_agent],
+    tools=[],
+    name='search_agent',
+    description="Runs web searches for you. Give it your query as an argument."
+)
+
+wiki_agent = ToolCallingAgent(
+    model=agent_model,
+    tools=[
+        WikipediaSearchTool(), VisitWebpageTool()
+    ],
+    name="web_agent",
+    description="Browses the web to find information",
+    max_steps=10,
+)
+
+managed_wiki_agent = CodeAgent(
+    model=agent_model,
+    managed_agents=[wiki_agent],
+    tools=[],
+    name='wiki_agent',
+    description="Runs wikipedia searches for you. Give it your query as an argument."
+)
+
+
 class BasicAgent:
     def __init__(self):
-        self.agent = ToolCallingAgent(
-            model= InferenceClientModel(model_id="openai/gpt-oss-20b", max_tokens=2048, provider="together"),
-            tools=tool,
-            max_steps=7,
-            add_base_tools=False,
-            planning_interval=4,
-            instructions=my_instructions
+        self.agent = CodeAgent(
+            model=manager_model,
+            tools=[final_answer, get_current_time, calculate_basic_math, save_note, list_saved_notes],
+            managed_agents=[managed_web_agent, managed_wiki_agent],
+            additional_authorized_imports=['datetime', 'pandas', 'numpy'],
+            planning_interval=6,
+            max_steps=16,
+            add_base_tools=True,
         )
         print("BasicAgent initialized.")
 
     def __call__(self, question: str) -> str:
         print(f"Agent received question (first 50 chars): {question[:50]}...")
-        # fixed_answer = "This is a default answer."
+
         fixed_answer = self.agent.run(question)
         print(f"Agent returning fixed answer: {fixed_answer}")
         return fixed_answer
 
-def run_and_submit_all( profile: gr.OAuthProfile | None):
+
+def run_and_submit_all(profile: gr.OAuthProfile | None):
     """
     Fetches all questions, runs the BasicAgent on them, submits all answers,
     and displays the results.
     """
     # --- Determine HF Space Runtime URL and Repo URL ---
-    space_id = os.getenv("SPACE_ID") # Get the SPACE_ID for sending link to the code
+    space_id = os.getenv("SPACE_ID")  # Get the SPACE_ID for sending link to the code
 
     if profile:
-        username= f"{profile.username}"
+        username = f"{profile.username}"
         print(f"User logged in: {username}")
     else:
         print("User not logged in.")
@@ -78,16 +125,16 @@ def run_and_submit_all( profile: gr.OAuthProfile | None):
         response.raise_for_status()
         questions_data = response.json()
         if not questions_data:
-             print("Fetched questions list is empty.")
-             return "Fetched questions list is empty or invalid format.", None
+            print("Fetched questions list is empty.")
+            return "Fetched questions list is empty or invalid format.", None
         print(f"Fetched {len(questions_data)} questions.")
     except requests.exceptions.RequestException as e:
         print(f"Error fetching questions: {e}")
         return f"Error fetching questions: {e}", None
     except requests.exceptions.JSONDecodeError as e:
-         print(f"Error decoding JSON response from questions endpoint: {e}")
-         print(f"Response text: {response.text[:500]}")
-         return f"Error decoding server response for questions: {e}", None
+        print(f"Error decoding JSON response from questions endpoint: {e}")
+        print(f"Response text: {response.text[:500]}")
+        return f"Error decoding server response for questions: {e}", None
     except Exception as e:
         print(f"An unexpected error occurred fetching questions: {e}")
         return f"An unexpected error occurred fetching questions: {e}", None
@@ -107,8 +154,8 @@ def run_and_submit_all( profile: gr.OAuthProfile | None):
             answers_payload.append({"task_id": task_id, "submitted_answer": submitted_answer})
             results_log.append({"Task ID": task_id, "Question": question_text, "Submitted Answer": submitted_answer})
         except Exception as e:
-             print(f"Error running agent on task {task_id}: {e}")
-             results_log.append({"Task ID": task_id, "Question": question_text, "Submitted Answer": f"AGENT ERROR: {e}"})
+            print(f"Error running agent on task {task_id}: {e}")
+            results_log.append({"Task ID": task_id, "Question": question_text, "Submitted Answer": f"AGENT ERROR: {e}"})
 
     if not answers_payload:
         print("Agent did not produce any answers to submit.")
@@ -169,11 +216,9 @@ with gr.Blocks() as demo:
     gr.Markdown(
         """
         **Instructions:**
-
         1.  Please clone this space, then modify the code to define your agent's logic, the tools, the necessary packages, etc ...
         2.  Log in to your Hugging Face account using the button below. This uses your HF username for submission.
         3.  Click 'Run Evaluation & Submit All Answers' to fetch questions, run your agent, submit answers, and see the score.
-
         ---
         **Disclaimers:**
         Once clicking on the "submit button, it can take quite some time ( this is the time for the agent to go through all the questions).
@@ -195,10 +240,10 @@ with gr.Blocks() as demo:
     )
 
 if __name__ == "__main__":
-    print("\n" + "-"*30 + " App Starting " + "-"*30)
+    print("\n" + "-" * 30 + " App Starting " + "-" * 30)
     # Check for SPACE_HOST and SPACE_ID at startup for information
     space_host_startup = os.getenv("SPACE_HOST")
-    space_id_startup = os.getenv("SPACE_ID") # Get SPACE_ID at startup
+    space_id_startup = os.getenv("SPACE_ID")  # Get SPACE_ID at startup
 
     if space_host_startup:
         print(f"✅ SPACE_HOST found: {space_host_startup}")
@@ -206,14 +251,14 @@ if __name__ == "__main__":
     else:
         print("ℹ️  SPACE_HOST environment variable not found (running locally?).")
 
-    if space_id_startup: # Print repo URLs if SPACE_ID is found
+    if space_id_startup:  # Print repo URLs if SPACE_ID is found
         print(f"✅ SPACE_ID found: {space_id_startup}")
         print(f"   Repo URL: https://huggingface.co/spaces/{space_id_startup}")
         print(f"   Repo Tree URL: https://huggingface.co/spaces/{space_id_startup}/tree/main")
     else:
         print("ℹ️  SPACE_ID environment variable not found (running locally?). Repo URL cannot be determined.")
 
-    print("-"*(60 + len(" App Starting ")) + "\n")
+    print("-" * (60 + len(" App Starting ")) + "\n")
 
     print("Launching Gradio Interface for Basic Agent Evaluation...")
     demo.launch(debug=True, share=False)

requirements.txt

@@ -1,9 +1,7 @@
 gradio
 requests
 smolagents
-ddgs
-transformers
-wikipedia-api
-pandas
 duckduckgo-search
+wikipedia-api
+markdownify
 youtube_transcript_api

tools.py

@@ -1,6 +1,12 @@
-from smolagents import tool
+from smolagents import tool, Tool
 from youtube_transcript_api import YouTubeTranscriptApi
 from urllib.parse import urlparse, parse_qs
+import os
+import ast
+import operator as op
+from typing import Any, Optional
+from datetime import datetime
+from smolagents import  tool
 
 @tool
 def youtube_tool(video_url: str) -> str:
@@ -33,4 +39,122 @@ def youtube_tool(video_url: str) -> str:
         return full_text
 
     except Exception as e:
-        return f"Error analyzing YouTube video: {str(e)}"
+        return f"Error analyzing YouTube video: {str(e)}"
+
+
+@tool
+def get_current_time() -> str:
+    """Get the current date and time in a readable format."""
+    now = datetime.now()
+    return now.strftime("%Y-%m-%d %H:%M:%S")
+
+
+@tool
+def calculate_basic_math(expression: str) -> str:
+    """
+    Safely evaluate basic mathematical expressions using AST.
+
+    Args:
+        expression: A string containing a mathematical expression like "2+2" or "10*5"
+
+    Returns:
+        The result of the calculation as a string
+    """
+    # Safe operations mapping
+    SAFE_OPS = {
+        ast.Add: op.add,
+        ast.Sub: op.sub,
+        ast.Mult: op.mul,
+        ast.Div: op.truediv,
+        ast.Pow: op.pow,
+        ast.USub: op.neg
+    }
+
+    def _safe_eval(node):
+        """Recursively evaluate AST nodes safely."""
+        if isinstance(node, ast.Num):  # Numbers
+            return node.n
+        elif isinstance(node, ast.Constant):  # Python 3.8+ constant nodes
+            return node.value
+        elif isinstance(node, ast.BinOp):  # Binary operations
+            return SAFE_OPS[type(node.op)](_safe_eval(node.left), _safe_eval(node.right))
+        elif isinstance(node, ast.UnaryOp):  # Unary operations
+            return SAFE_OPS[type(node.op)](_safe_eval(node.operand))
+        else:
+            raise ValueError(f"Unsupported operation: {type(node)}")
+
+    try:
+        # Parse expression into AST and evaluate safely
+        node = ast.parse(expression, mode='eval')
+        result = _safe_eval(node.body)
+        return f"Result: {result}"
+    except Exception as e:
+        return f"Error calculating '{expression}': {str(e)}"
+
+
+@tool
+def save_note(content: str, filename: Optional[str] = None) -> str:
+    """
+    Save a note to a text file.
+
+    Args:
+        content: The content to save in the note
+        filename: Optional filename. If not provided, uses timestamp
+
+    Returns:
+        Success message with filename
+    """
+    try:
+        if filename is None:
+            timestamp = datetime.now().strftime("%Y%m%d_%H%M%S")
+            filename = f"note_{timestamp}.txt"
+
+        # Try to create notes directory, fall back to /tmp if permission denied
+        try:
+            os.makedirs("notes", exist_ok=True)
+            notes_dir = "notes"
+        except PermissionError:
+            notes_dir = "/tmp/notes"
+            os.makedirs(notes_dir, exist_ok=True)
+
+        filepath = os.path.join(notes_dir, filename)
+
+        with open(filepath, 'w', encoding='utf-8') as f:
+            f.write(content)
+
+        return f"✅ Note saved successfully to: {filepath}"
+    except Exception as e:
+        return f"❌ Error saving note: {str(e)}"
+
+
+@tool
+def list_saved_notes() -> str:
+    """List all saved notes in the notes directory."""
+    try:
+        # Check both possible notes directories
+        all_files = []
+        for notes_dir in ["notes", "/tmp/notes"]:
+            if os.path.exists(notes_dir):
+                files = os.listdir(notes_dir)
+                txt_files = [f for f in files if f.endswith('.txt')]
+                all_files.extend([(f, notes_dir) for f in txt_files])
+
+        if not all_files:
+            return "📂 No notes found. Save a note first!"
+
+        file_list = "\n".join([f"- {file} ({location})" for file, location in sorted(all_files)])
+        return f"📂 Saved notes:\n{file_list}"
+    except Exception as e:
+        return f"❌ Error listing notes: {str(e)}"
+
+class FinalAnswerTool(Tool):
+    name = "final_answer"
+    description = "Provides a final answer to the given problem."
+    inputs = {'answer': {'type': 'any', 'description': 'The final answer to the problem'}}
+    output_type = "any"
+
+    def forward(self, answer: Any) -> Any:
+        return answer
+
+    def __init__(self, *args, **kwargs):
+        self.is_initialized = False