特急こうのとりの停車駅を修正し新大阪直通に対応 - #1629
Conversation
川西池田・中山寺を通過から一部通過に修正し、東海道本線(JR京都線・ JR神戸線)側の新大阪・大阪・塚本・尼崎を line_group_cd=300 に追加した。 従来の新大阪は JR宝塚線側の廃止駅レコード(1162901, e_status=2)のみを 参照しており、クエリの e_status = 0 条件で常に除外されていた。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017XD5VJ7bJxoSXhmA2L1Y2G
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
🚉 停車駅の変更1 件の変更された停車パターン
✏️ こうのとり (Kounotori) - JR京都線 / JR宝塚線 / JR神戸線 / 福知山線 / 山陰本線 / 山陰本線type_cd=222, line_group_cd=300 変更内容: 6 駅追加 | 中山寺: ○ → △ | 川西池田: ○ → △ 全駅リスト (53駅)
|
概要
特急「こうのとり」(
type_cd=222/line_group_cd=300)の停車駅データの誤りを修正し、新大阪への直通運転を種別データ上でも表現できるようにしました。変更の種類
変更内容
対象は
data/5!station_station_types.csvのline_group_cd=300のみです。1. 停車駅の誤り修正
1162908):passを1(通過)→2(一部通過)1162909):passを1(通過)→2(一部通過)こうのとりは 2・4・19・21・23 号が川西池田・中山寺に停車するため、通過扱いは誤りでした。同グループ内で同じく「一部の列車が停車」となる西宮名塩・新三田・相野・谷川・黒井が既に
pass=2のため、値をそれに揃えています。石生・市島は 2011 年 3 月 11 日以降全列車通過のためpass=1のまま変更していません。2. 新大阪直通への対応
グループには新大阪の行が存在していましたが、参照先が JR宝塚線側の
1162901で、この駅レコードはdata/3!stations.csv上e_status=2(廃止)でした。列車種別・駅取得クエリはいずれもs.e_status = 0で絞り込むため(stationapi/src/infrastructure/train_type_repository.rsほか)、この行は常に結果から除外され、アプリ上で新大阪が扱えない状態でした。そこで東海道本線側(本データセットでは愛称名の JR京都線 / JR神戸線)の駅コードを追加しました。
station_cdpass116021311602141160301116030211629241160303AGENTS.mdの「直通する接続駅は路線ごとのstation_cdを全て登録する」ルールに従い、新大阪・大阪・尼崎で各路線側のレコードを揃えました。これにより JR京都線・JR神戸線を選択中でもこうのとりが種別として選択できます。pass=1)で追加しています。種別取得クエリはsst.pass <> 1で絞るため、塚本で種別候補には現れません。ORDER BY sst.id(= CSV の行順)が経路順になるため、新大阪 → 大阪 → 塚本 → 尼崎 → 塚口 … の順で挿入しています。line_group_cd=798)と同じ書き方に揃えました。1162901は残置しています。他グループにも同種の廃駅参照行が 269 件あり、既存の慣行に沿った扱いです。検証
cargo run -p data_validator→[VALID] No errors reported.テスト
データ(
data/**/*.csv)のみの変更でコード本体に変更が無いため、下記 3 項目のチェックは省略しています。代わりにcargo run -p data_validatorを実行し、エラーが無いことを確認済みです。cargo fmt --all -- --checkが通ることcargo clippy -- -D warningsが通ることcargo test(SQLX_OFFLINE=true)が通ること関連Issue
スクリーンショット(任意)