Skip to content

Commit

Permalink
🐝 Remove the None option for the new insighter
Browse files Browse the repository at this point in the history
  • Loading branch information
larsyencken committed Sep 18, 2024
1 parent 45483e5 commit 6cdd73a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 9 deletions.
8 changes: 1 addition & 7 deletions apps/wizard/app_pages/datainsight_robot.py
Original file line number Diff line number Diff line change
@@ -1,17 +1,11 @@
import base64
from pathlib import Path
from typing import cast
from urllib import parse

import pandas as pd
import requests
import streamlit as st
from owid.catalog.charts import Chart

from apps.utils.gpt import OpenAIWrapper
from apps.wizard.app_pages.insights import (
fetch_chart_data,
fetch_data,
get_grapher_thumbnail,
get_thumbnail_url,
list_charts,
Expand Down Expand Up @@ -169,7 +163,7 @@ def get_trajectory_prompt(base_prompt: str, slug: str) -> str:
all_charts = list_charts(conn)
slugs = st.multiselect(
label="Grapher slug",
options=[None] + all_charts,
options=all_charts,
help="Introduce the URL to a Grapher URL. Query parameters work!",
key="tab2_url",
)
Expand Down
4 changes: 2 additions & 2 deletions apps/wizard/app_pages/insights.py
Original file line number Diff line number Diff line change
@@ -1,14 +1,14 @@
import base64
import datetime as dt
import json
from pathlib import Path
from typing import Literal
from urllib import parse

import pandas as pd
import requests
from dateutil.parser import parse as date_parse

from etl.db import get_connection


def get_thumbnail_url(grapher_url: str) -> str:
"""
Expand Down

0 comments on commit 6cdd73a

Please sign in to comment.