Update app.py

app.py

@@ -1,24 +1,14 @@
-from smolagents import CodeAgent,DuckDuckGoSearchTool, HfApiModel,load_tool,tool
 import datetime
 import requests
 import pytz
 import yaml
-from tools.final_answer import FinalAnswerTool
-from tools.weather import WeatherForecast
+import json
 
+from tools.weather import WeatherForecast
+from tools.final_answer import FinalAnswerTool
+from smolagents import CodeAgent, HfApiModel, load_tool, tool
 from Gradio_UI import GradioUI
 
-# Below is an example of a tool that does nothing. Amaze us with your creativity !
-@tool
-def my_custom_tool(arg1:str, arg2:int)-> str: #it's import to specify the return type
-    #Keep this format for the description / args / args description but feel free to modify the tool
-    """A tool that does nothing yet 
-    Args:
-        arg1: the first argument
-        arg2: the second argument
-    """
-    return "What magic will you build ?"
-
 @tool
 def get_current_time_in_timezone(timezone: str) -> str:
     """A tool that fetches the current local time in a specified timezone.
@@ -41,13 +31,13 @@ def get_weather_forecast(city_name: str) -> str:
         city_name: A string representing a valid city (e.g., 'Bangalore').
     """
 
-    latitude, longitude = get_coordinates_no_api_key(city_name)
+    latitude, longitude = get_coordinates(city_name)
     
     base_url = "https://api.open-meteo.com/v1/forecast"  # No API key needed for this version
     params = {
         "latitude": latitude,
         "longitude": longitude,
-        "hourly": "temperature_2m,precipitation,wind_speed",
+        "hourly": "temperature_2m",
         "daily": "temperature_2m_max,temperature_2m_min,precipitation_sum",
         "forecast_days": 7,
         "timezone": "auto"
@@ -70,12 +60,14 @@ def get_weather_forecast(city_name: str) -> str:
         print(f"Error decoding JSON response: {e}")
         return None
 
-def get_coordinates_no_api_key(city_name: str) -> [float, float]:
+def get_coordinates(city_name: str) -> [float, float]:
     """Gets coordinates using OpenStreetMap's Nominatim (no API key, but with limitations)."""
 
     # This approach is less reliable and might have rate limits.
     # It's suitable for basic use cases but not for production.
-
+    headers = {
+        'User-Agent': 'MyGeocodingApp/1.0 (youremail@example.com)'  # Replace with your actual email
+    }
     geocoding_url = "https://nominatim.openstreetmap.org/search" # No API Key needed
     params = {
         "q": city_name,
@@ -103,32 +95,17 @@ def get_coordinates_no_api_key(city_name: str) -> [float, float]:
         print(f"Error parsing geocoding response: {e}")
         return None, None
 
-# def get_weather(city: str) -> str:
-#     """A tool that fetches the current weather of a specified city.
-#     Args:
-#         city: A string representing a valid city (e.g., 'Bangalore').
-#     """
-#     try:
-#         response =  requests.get('https://api.open-meteo.com/v1/forecast?latitude=52.52&longitude=13.41&hourly=temperature_2m&models=bom_access_global')
-#         weatherJSON = response.json()
-#         cityName = weatherJSON.get('location').get('name')
-#         cityTemp = weatherJSON.get('current').get('temp_c')
-#         cityCondition = weatherJSON.get('current').get('condition').get('text')
-#         return f"The current Temperature in {cityName} is {cityTemp} and conditions are {cityCondition}"
-#     except Exception as e:
-#         return f"Error fetching weather conditions for {city}"
-
 final_answer = FinalAnswerTool()
 weather_forecast = WeatherForecast()
 
-# If the agent does not answer, the model is overloaded, please use another model or the following Hugging Face Endpoint that also contains qwen2.5 coder:
 # model_id='https://pflgm2locj2t89co.us-east-1.aws.endpoints.huggingface.cloud' 
+model_id='Qwen/Qwen2.5-Coder-32B-Instruct',# it is possible that this model may be overloaded
 
 model = HfApiModel(
-max_tokens=2096,
-temperature=0.5,
-model_id='Qwen/Qwen2.5-Coder-32B-Instruct',# it is possible that this model may be overloaded
-custom_role_conversions=None,
+    max_tokens=2096,
+    temperature=0.5,
+    model_id=model_id,
+    custom_role_conversions=None,
 )