fix: Use os.environ[] instead of .get() for DATABASE_URL
Some checks failed
CI / lint-and-test (push) Has been cancelled
Some checks failed
CI / lint-and-test (push) Has been cancelled
Fixes Pylance type error - create_engine() expects str, not str | None. Using direct access raises KeyError if not set, which is correct behavior. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
@@ -38,7 +38,7 @@
|
|||||||
"# Load .env from project root\n",
|
"# Load .env from project root\n",
|
||||||
"load_dotenv('../../.env')\n",
|
"load_dotenv('../../.env')\n",
|
||||||
"\n",
|
"\n",
|
||||||
"engine = create_engine(os.environ.get('DATABASE_URL'))\n",
|
"engine = create_engine(os.environ['DATABASE_URL'])\n",
|
||||||
"\n",
|
"\n",
|
||||||
"query = \"\"\"\n",
|
"query = \"\"\"\n",
|
||||||
"SELECT\n",
|
"SELECT\n",
|
||||||
|
|||||||
@@ -29,7 +29,7 @@
|
|||||||
"execution_count": null,
|
"execution_count": null,
|
||||||
"metadata": {},
|
"metadata": {},
|
||||||
"outputs": [],
|
"outputs": [],
|
||||||
"source": "import pandas as pd\nfrom sqlalchemy import create_engine\nfrom dotenv import load_dotenv\nimport os\n\n# Load .env from project root\nload_dotenv('../../.env')\n\nengine = create_engine(os.environ.get('DATABASE_URL'))\n\nquery = \"\"\"\nSELECT\n neighbourhood_name,\n parks_index,\n schools_index,\n transit_index,\n amenity_index,\n amenity_tier\nFROM public_marts.mart_neighbourhood_amenities\nWHERE year = (SELECT MAX(year) FROM public_marts.mart_neighbourhood_amenities)\nORDER BY amenity_index DESC\n\"\"\"\n\ndf = pd.read_sql(query, engine)\nprint(f\"Loaded {len(df)} neighbourhoods\")"
|
"source": "import pandas as pd\nfrom sqlalchemy import create_engine\nfrom dotenv import load_dotenv\nimport os\n\n# Load .env from project root\nload_dotenv('../../.env')\n\nengine = create_engine(os.environ['DATABASE_URL'])\n\nquery = \"\"\"\nSELECT\n neighbourhood_name,\n parks_index,\n schools_index,\n transit_index,\n amenity_index,\n amenity_tier\nFROM public_marts.mart_neighbourhood_amenities\nWHERE year = (SELECT MAX(year) FROM public_marts.mart_neighbourhood_amenities)\nORDER BY amenity_index DESC\n\"\"\"\n\ndf = pd.read_sql(query, engine)\nprint(f\"Loaded {len(df)} neighbourhoods\")"
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
"cell_type": "markdown",
|
"cell_type": "markdown",
|
||||||
|
|||||||
@@ -38,7 +38,7 @@
|
|||||||
"# Load .env from project root\n",
|
"# Load .env from project root\n",
|
||||||
"load_dotenv('../../.env')\n",
|
"load_dotenv('../../.env')\n",
|
||||||
"\n",
|
"\n",
|
||||||
"engine = create_engine(os.environ.get('DATABASE_URL'))\n",
|
"engine = create_engine(os.environ['DATABASE_URL'])\n",
|
||||||
"\n",
|
"\n",
|
||||||
"query = \"\"\"\n",
|
"query = \"\"\"\n",
|
||||||
"SELECT\n",
|
"SELECT\n",
|
||||||
|
|||||||
@@ -38,7 +38,7 @@
|
|||||||
"# Load .env from project root\n",
|
"# Load .env from project root\n",
|
||||||
"load_dotenv('../../.env')\n",
|
"load_dotenv('../../.env')\n",
|
||||||
"\n",
|
"\n",
|
||||||
"engine = create_engine(os.environ.get('DATABASE_URL'))\n",
|
"engine = create_engine(os.environ['DATABASE_URL'])\n",
|
||||||
"\n",
|
"\n",
|
||||||
"query = \"\"\"\n",
|
"query = \"\"\"\n",
|
||||||
"SELECT\n",
|
"SELECT\n",
|
||||||
|
|||||||
@@ -38,7 +38,7 @@
|
|||||||
"# Load .env from project root\n",
|
"# Load .env from project root\n",
|
||||||
"load_dotenv('../../.env')\n",
|
"load_dotenv('../../.env')\n",
|
||||||
"\n",
|
"\n",
|
||||||
"engine = create_engine(os.environ.get('DATABASE_URL'))\n",
|
"engine = create_engine(os.environ['DATABASE_URL'])\n",
|
||||||
"\n",
|
"\n",
|
||||||
"query = \"\"\"\n",
|
"query = \"\"\"\n",
|
||||||
"SELECT\n",
|
"SELECT\n",
|
||||||
|
|||||||
@@ -38,7 +38,7 @@
|
|||||||
"# Load .env from project root\n",
|
"# Load .env from project root\n",
|
||||||
"load_dotenv('../../.env')\n",
|
"load_dotenv('../../.env')\n",
|
||||||
"\n",
|
"\n",
|
||||||
"engine = create_engine(os.environ.get('DATABASE_URL'))\n",
|
"engine = create_engine(os.environ['DATABASE_URL'])\n",
|
||||||
"\n",
|
"\n",
|
||||||
"query = \"\"\"\n",
|
"query = \"\"\"\n",
|
||||||
"SELECT\n",
|
"SELECT\n",
|
||||||
|
|||||||
@@ -38,7 +38,7 @@
|
|||||||
"# Load .env from project root\n",
|
"# Load .env from project root\n",
|
||||||
"load_dotenv('../../.env')\n",
|
"load_dotenv('../../.env')\n",
|
||||||
"\n",
|
"\n",
|
||||||
"engine = create_engine(os.environ.get('DATABASE_URL'))\n",
|
"engine = create_engine(os.environ['DATABASE_URL'])\n",
|
||||||
"\n",
|
"\n",
|
||||||
"query = \"\"\"\n",
|
"query = \"\"\"\n",
|
||||||
"SELECT\n",
|
"SELECT\n",
|
||||||
|
|||||||
@@ -38,7 +38,7 @@
|
|||||||
"# Load .env from project root\n",
|
"# Load .env from project root\n",
|
||||||
"load_dotenv('../../.env')\n",
|
"load_dotenv('../../.env')\n",
|
||||||
"\n",
|
"\n",
|
||||||
"engine = create_engine(os.environ.get('DATABASE_URL'))\n",
|
"engine = create_engine(os.environ['DATABASE_URL'])\n",
|
||||||
"\n",
|
"\n",
|
||||||
"# City-wide average rent by year\n",
|
"# City-wide average rent by year\n",
|
||||||
"query = \"\"\"\n",
|
"query = \"\"\"\n",
|
||||||
|
|||||||
@@ -38,7 +38,7 @@
|
|||||||
"# Load .env from project root\n",
|
"# Load .env from project root\n",
|
||||||
"load_dotenv('../../.env')\n",
|
"load_dotenv('../../.env')\n",
|
||||||
"\n",
|
"\n",
|
||||||
"engine = create_engine(os.environ.get('DATABASE_URL'))\n",
|
"engine = create_engine(os.environ['DATABASE_URL'])\n",
|
||||||
"\n",
|
"\n",
|
||||||
"query = \"\"\"\n",
|
"query = \"\"\"\n",
|
||||||
"SELECT\n",
|
"SELECT\n",
|
||||||
|
|||||||
@@ -38,7 +38,7 @@
|
|||||||
"# Load .env from project root\n",
|
"# Load .env from project root\n",
|
||||||
"load_dotenv('../../.env')\n",
|
"load_dotenv('../../.env')\n",
|
||||||
"\n",
|
"\n",
|
||||||
"engine = create_engine(os.environ.get('DATABASE_URL'))\n",
|
"engine = create_engine(os.environ['DATABASE_URL'])\n",
|
||||||
"\n",
|
"\n",
|
||||||
"query = \"\"\"\n",
|
"query = \"\"\"\n",
|
||||||
"SELECT\n",
|
"SELECT\n",
|
||||||
|
|||||||
@@ -29,7 +29,7 @@
|
|||||||
"execution_count": null,
|
"execution_count": null,
|
||||||
"metadata": {},
|
"metadata": {},
|
||||||
"outputs": [],
|
"outputs": [],
|
||||||
"source": "import pandas as pd\nfrom sqlalchemy import create_engine\nfrom dotenv import load_dotenv\nimport os\n\n# Load .env from project root\nload_dotenv('../../.env')\n\nengine = create_engine(os.environ.get('DATABASE_URL'))\n\nquery = \"\"\"\nSELECT\n neighbourhood_id,\n neighbourhood_name,\n geometry,\n year,\n livability_score,\n safety_score,\n affordability_score,\n amenity_score,\n population,\n median_household_income\nFROM public_marts.mart_neighbourhood_overview\nWHERE year = (SELECT MAX(year) FROM public_marts.mart_neighbourhood_overview)\nORDER BY livability_score DESC\n\"\"\"\n\ndf = pd.read_sql(query, engine)\nprint(f\"Loaded {len(df)} neighbourhoods\")"
|
"source": "import pandas as pd\nfrom sqlalchemy import create_engine\nfrom dotenv import load_dotenv\nimport os\n\n# Load .env from project root\nload_dotenv('../../.env')\n\nengine = create_engine(os.environ['DATABASE_URL'])\n\nquery = \"\"\"\nSELECT\n neighbourhood_id,\n neighbourhood_name,\n geometry,\n year,\n livability_score,\n safety_score,\n affordability_score,\n amenity_score,\n population,\n median_household_income\nFROM public_marts.mart_neighbourhood_overview\nWHERE year = (SELECT MAX(year) FROM public_marts.mart_neighbourhood_overview)\nORDER BY livability_score DESC\n\"\"\"\n\ndf = pd.read_sql(query, engine)\nprint(f\"Loaded {len(df)} neighbourhoods\")"
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
"cell_type": "markdown",
|
"cell_type": "markdown",
|
||||||
|
|||||||
@@ -38,7 +38,7 @@
|
|||||||
"# Load .env from project root\n",
|
"# Load .env from project root\n",
|
||||||
"load_dotenv('../../.env')\n",
|
"load_dotenv('../../.env')\n",
|
||||||
"\n",
|
"\n",
|
||||||
"engine = create_engine(os.environ.get('DATABASE_URL'))\n",
|
"engine = create_engine(os.environ['DATABASE_URL'])\n",
|
||||||
"\n",
|
"\n",
|
||||||
"query = \"\"\"\n",
|
"query = \"\"\"\n",
|
||||||
"SELECT\n",
|
"SELECT\n",
|
||||||
|
|||||||
@@ -38,7 +38,7 @@
|
|||||||
"# Load .env from project root\n",
|
"# Load .env from project root\n",
|
||||||
"load_dotenv('../../.env')\n",
|
"load_dotenv('../../.env')\n",
|
||||||
"\n",
|
"\n",
|
||||||
"engine = create_engine(os.environ.get('DATABASE_URL'))\n",
|
"engine = create_engine(os.environ['DATABASE_URL'])\n",
|
||||||
"\n",
|
"\n",
|
||||||
"query = \"\"\"\n",
|
"query = \"\"\"\n",
|
||||||
"SELECT\n",
|
"SELECT\n",
|
||||||
|
|||||||
@@ -38,7 +38,7 @@
|
|||||||
"# Load .env from project root\n",
|
"# Load .env from project root\n",
|
||||||
"load_dotenv('../../.env')\n",
|
"load_dotenv('../../.env')\n",
|
||||||
"\n",
|
"\n",
|
||||||
"engine = create_engine(os.environ.get('DATABASE_URL'))\n",
|
"engine = create_engine(os.environ['DATABASE_URL'])\n",
|
||||||
"\n",
|
"\n",
|
||||||
"query = \"\"\"\n",
|
"query = \"\"\"\n",
|
||||||
"SELECT\n",
|
"SELECT\n",
|
||||||
|
|||||||
@@ -38,7 +38,7 @@
|
|||||||
"# Load .env from project root\n",
|
"# Load .env from project root\n",
|
||||||
"load_dotenv('../../.env')\n",
|
"load_dotenv('../../.env')\n",
|
||||||
"\n",
|
"\n",
|
||||||
"engine = create_engine(os.environ.get('DATABASE_URL'))\n",
|
"engine = create_engine(os.environ['DATABASE_URL'])\n",
|
||||||
"\n",
|
"\n",
|
||||||
"query = \"\"\"\n",
|
"query = \"\"\"\n",
|
||||||
"SELECT\n",
|
"SELECT\n",
|
||||||
|
|||||||
Reference in New Issue
Block a user