improved UI and UX for the first tabs

app.py

@@ -1,5 +1,4 @@
 import gradio as gr
-import time
 
 # Local modules
 import fastf1_tools
@@ -11,30 +10,58 @@ from utils.constants import (
     DROPDOWN_SESSION_TYPES,
     MARKDOWN_INTRODUCTION,
     MARKDOWN_OPENF1_EXAMPLES,
-    OPENF1_TOOL_DESCRIPTION
+    OPENF1_TOOL_DESCRIPTION,
+    CONSTRUCTORS_PER_SEASON,
+    DRIVERS_PER_SEASON
 )
 
-iface_driver_championship_standings = gr.Interface(
-    fn=fastf1_tools.driver_championship_standings,
-    inputs=[
-        gr.Number(label="Calendar year", value=CURRENT_YEAR, minimum=1950, maximum=CURRENT_YEAR),
-        gr.Dropdown(label="Driver", choices=DRIVER_NAMES)
-    ],
-    outputs="text",
-    title="World Driver Championship Standings",
-    description="Get the world driver championship standings for a specific driver"
-)
-
-iface_constructor_championship_standings = gr.Interface(
-    fn=fastf1_tools.constructor_championship_standings,
-    inputs=[
-        gr.Number(label="Calendar year", value=CURRENT_YEAR, minimum=1950, maximum=CURRENT_YEAR),
-        gr.Dropdown(label="Constructor", choices=CONSTRUCTOR_NAMES)
-    ],
-    outputs="text",
-    title="World Constructor Championship Standings",
-    description="Get the world constructor championship standings for a specific constructor"
-)
+with gr.Blocks() as iface_driver_championship_standings:
+    gr.Markdown("## World Driver Championship Standings\nGet the world driver championship standings for a specific driver. Note that the older data has gaps and may not be entirely complete.")
+
+    with gr.Row():
+        year_input = gr.Number(label="Calendar year", value=CURRENT_YEAR, minimum=1950, maximum=CURRENT_YEAR)
+        driver_dropdown = gr.Dropdown(label="Driver", choices=DRIVERS_PER_SEASON.get(str(year_input.value), []))
+    output_text = gr.Textbox(label="Result")
+    submit_btn = gr.Button("Submit")
+
+    def update_drivers(year):
+        choices = DRIVERS_PER_SEASON.get(str(year), [])
+        return gr.update(choices=choices, value=(choices[0] if choices else None))
+
+    year_input.blur(
+        update_drivers,
+        inputs=year_input,
+        outputs=driver_dropdown
+    )
+    submit_btn.click(
+        fastf1_tools.driver_championship_standings,
+        inputs=[year_input, driver_dropdown],
+        outputs=output_text
+    )
+
+with gr.Blocks() as iface_constructor_championship_standings:
+    gr.Markdown("## World Constructor Championship Standings\nGet the current/past world constructor championship standings for a specific constructor. Note that the older data has gaps and may not be entirely complete.")
+    
+    with gr.Row():
+        year_input = gr.Number(label="Calendar year", value=CURRENT_YEAR, minimum=1950, maximum=CURRENT_YEAR)
+        constructor_dropdown = gr.Dropdown(label="Constructor", choices=CONSTRUCTORS_PER_SEASON.get(str(year_input.value), []))
+    output_text = gr.Textbox(label="Result")
+    submit_btn = gr.Button("Submit")
+
+    def update_constructors(year):
+        choices = CONSTRUCTORS_PER_SEASON.get(str(year), [])
+        return gr.update(choices=choices, value=(choices[0] if choices else None))
+
+    year_input.blur(
+        update_constructors,
+        inputs=year_input,
+        outputs=constructor_dropdown
+    )
+    submit_btn.click(
+        fastf1_tools.constructor_championship_standings,
+        inputs=[year_input, constructor_dropdown],
+        outputs=output_text
+    )
 
 iface_event_info = gr.Interface(
     fn=fastf1_tools.get_event_info,
@@ -107,36 +134,12 @@ iface_constructor_info = gr.Interface(
     description="Get background information about a specific constructor"
 )
 
-def timeout_test(seconds: int) -> str:
-    """
-    A simple function that sleeps for the given amount of seconds.
-
-    Args:
-        seconds (int): The amount of seconds to sleep
-
-    Returns:
-        str: A message indicating the timeout is completed
-    """
-    time.sleep(seconds)
-    return f"Timeout for {seconds} seconds completed"
-
-iface_timeout_testing = gr.Interface(
-    fn=timeout_test,
-    inputs=[
-        gr.Number(label="Seconds", value=5, minimum=1, maximum=500)
-    ],
-    outputs="text",
-    title="Timeout Testing",
-    description="Test the timeout functionality"
-)
-
 
 # Create your markdown-only tab using Blocks
 with gr.Blocks() as markdown_tab:
     gr.Markdown(MARKDOWN_INTRODUCTION)
 
 
-
 # OpenF1 tools tab
 def openf1_tools_tab():
     with gr.Blocks() as openf1_tools_tab:
@@ -194,8 +197,7 @@ named_interfaces = {
     "Session Results": iface_session_results,
     "Driver Info": iface_driver_info,
     "Constructor Info": iface_constructor_info,
-    "OpenF1 Tools": openf1_tools_tab(),
-    "Timeout Testing": iface_timeout_testing
+    "OpenF1 Tools": openf1_tools_tab()
 }
 
 # Tab names and interfaces

mcp_client.py

@@ -17,7 +17,7 @@ def agent_chat(message: str, history: list):
 
 if __name__ == "__main__":
 
-    list_tools = True # Set to True to only list tools (used for debugging)
+    list_tools = False # Set to True to only list tools (used for debugging)
     local_model = True # If you have Ollama installed, set this to True
     
     try:

todo.txt

@@ -1,5 +0,0 @@
-- For driver championship standings, make the Driver dropdown depend on the selected season. As it is implemented now one could not use it for older seasons!
-    * Solution is to have a static json file that maps drivers for each season/year
-- Same applies for constructor championship standings but instead Constructor dropdown
-    * Similar solution to above
-- Add driver comparison