Skip to content

Remove Places API and migrate to new CSV - #1

Merged
hengin-eer merged 1 commit into
mainfrom
agent/remove-places-api-update-csv
Jul 27, 2026
Merged

hengin-eer merged 1 commit into
mainfrom
agent/remove-places-api-update-csv

Conversation

@hengin-eer

Copy link
Copy Markdown
Collaborator

Summary

  • Remove all Google Places API integrations and related preprocessing/runtime code.
  • Migrate the new 1,729-row CSV to data/places.csv.
  • Add a project-local uv environment with a locked dependency set.

Changes

  • CSV preprocessing now normalizes title, memo, URL, tags, comments, latitude, longitude, and address.
  • Chroma documents use CSV-provided location and note fields.
  • Search and evaluation no longer perform or mention live Places lookups.
  • Add setup documentation and pytest coverage.

Validation

  • uv lock --check
  • uv sync --locked --offline
  • uv run pytest -q (6 passed)
  • CSV preprocessing: 1,729 records
  • Chroma build: 1,729 chunks
  • Retrieval smoke test: 3 candidates returned

Interactive Groq plan generation was not run because no API key was available.

@hengin-eer
hengin-eer requested a review from Copilot July 27, 2026 09:34
@hengin-eer
hengin-eer marked this pull request as ready for review July 27, 2026 09:34

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Google Places API 依存を完全に削除し、Google マップ保存リスト由来の data/places.csv のみをソースとして前処理→ベクトルDB構築→検索/生成を行う構成へ移行する PR です。uv によるプロジェクトローカル環境と pytest を追加し、CSVベース運用の手順をドキュメント化しています。

Changes:

  • Places API の前処理/実行時参照を削除し、検索コンテキストは CSV/Chroma のみで構築
  • 新 CSV 形式(タイトル/メモ/URL/タグ/コメント/緯度/経度/住所)を正規化して places_data.json を生成・DB化
  • uv セットアップ、.env.example、pytest を追加し、移行の回帰テストを追加

Reviewed changes

Copilot reviewed 15 out of 18 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
操作説明書.md uv 前提の実行手順と、CSV/ローカルDBのみで動作する旨へ更新
README.md セットアップ・実行・生成物の説明をCSV運用に合わせて刷新
pyproject.toml uv/パッケージング/依存関係/pytest を追加
.python-version Python 3.12 を固定
.gitignore .venv/.env/pytest cache/egg-info を追加し生成物を除外
.env.example GROQ_API_KEY の雛形を追加
fetch_places_data.py Places API を廃止し、新CSV→JSON変換・正規化ロジックへ変更
make_vector_db.py CSV由来フィールドのみで Document/metadata を構築するよう変更
nodes/search_node.py live lookup を削除し、Chroma 検索結果のみで context を構築
nodes/generate_node.py 営業時間に関する生成ルールを削除(根拠データが無いため)
nodes/self_eval_node.py 営業時間整合性の評価観点を削除(根拠データが無いため)
places_client.py Places API クライアントを削除
add_address_to_csv.py 住所付与用の Places API スクリプトを削除
tests/test_fetch_places_data.py 新CSVマッピング・座標バリデーション・実CSVの整合をテスト追加
tests/test_make_vector_db.py ページコンテンツ/メタデータがCSV由来のみであることをテスト追加
tests/test_search_node.py 検索ノードが live lookup なしで context を構築することをテスト追加

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread make_vector_db.py
Comment thread fetch_places_data.py
@hengin-eer
hengin-eer merged commit 276d57a into main Jul 27, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants