停車駅・種別・経路のデータ不備をまとめて修正 - #1663
Conversation
|
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 |
🚉 停車駅の変更12 件の変更された停車パターン | 4 件の削除された停車パターン
🗑️ 各駅停車 (Local) - 小田急多摩線type_cd=101, line_group_cd=779 変更内容: 8 駅削除 ✏️ マリンライナー (Marine Liner) - 宇野線 / 瀬戸大橋線 / 予讃線type_cd=123, line_group_cd=400 変更内容: 備前西市: △ → ▲ 全駅リスト (23駅)
🗑️ あずさ (Azusa) - 中央線(快速) / 中央本線 / 篠ノ井線 / 北アルプス線type_cd=210, line_group_cd=140 変更内容: 4 駅削除 全駅リスト (106駅)
🗑️ あずさ (Azusa) - 総武本線 / 中央・総武線 / 中央線(快速) / 中央本線 / 篠ノ井線 / 北アルプス線type_cd=210, line_group_cd=904 変更内容: 4 駅削除 全駅リスト (116駅)
✏️ 急行 (Express) - 東京メトロ千代田線 / 小田急線 / 小田急多摩線type_cd=304, line_group_cd=154 変更内容: 黒川: ○ → ● | 五月台: ○ → ● | はるひ野: ○ → ● 全駅リスト (46駅)
✏️ 急行 (Express) - 東京メトロ千代田線 / 小田急線 / 小田急多摩線type_cd=304, line_group_cd=771 変更内容: 黒川: ○ → ● | 五月台: ○ → ● | はるひ野: ○ → ● 全駅リスト (47駅)
✏️ 準急 (Semi Express) - 小田急線type_cd=315, line_group_cd=125 変更内容: 喜多見: ○ → ● | 和泉多摩川: ○ → ● 全駅リスト (36駅)
✏️ 準急 (Semi Express) - 東京メトロ千代田線 / 小田急線type_cd=315, line_group_cd=155 変更内容: 喜多見: ○ → ● | 和泉多摩川: ○ → ● 全駅リスト (51駅)
✏️ 準急 (Semi Express) - 東京メトロ千代田線 / 小田急線type_cd=315, line_group_cd=772 変更内容: 喜多見: ○ → ● | 和泉多摩川: ○ → ● 全駅リスト (52駅)
✏️ 快速急行 (Rapid Express) - 小田急線type_cd=322, line_group_cd=269 変更内容: 開成: ○ → ● 全駅リスト (47駅)
✏️ 急行 (Express) - 小田急線 / 小田急多摩線type_cd=327, line_group_cd=128 変更内容: 黒川: ○ → ● | 五月台: ○ → ● | はるひ野: ○ → ● 全駅リスト (31駅)
🗑️ 急行 (Express) - 小田急線type_cd=327, line_group_cd=779 変更内容: 23 駅削除 ✏️ 特急 (Special Express) - 京王線type_cd=354, line_group_cd=71 変更内容: 東府中: ○ → ▲ 全駅リスト (32駅)
✏️ 特急 (Special Express) - 京王線 / 京王高尾線type_cd=354, line_group_cd=415 変更内容: 東府中: ○ → ▲ 全駅リスト (38駅)
✏️ 特急 (Special Express) - 京王線 / 京王競馬場線type_cd=354, line_group_cd=764 変更内容: 東府中: ○ → ● | 東府中: ○ → ● 全駅リスト (23駅)
✏️ ふじさん (Mt. Fuji) - 小田急線 / 御殿場線type_cd=533, line_group_cd=1008 変更内容: 新松田: ● → ○ 全駅リスト (48駅)
|
概要
Issue #1656 で報告された停車駅・種別・経路のデータ不備のうち、
data/5!station_station_types.csvで修正できる 6 項目を反映する。あわせて調査中に見つかった快速急行の開成停車(同じダイヤ改正の反映漏れ)も直す。大半は 2025-03-15 ダイヤ改正の未反映によるもの。変更の種類
変更内容
小田急(2025-03-15 改正の反映漏れ)
line_group_cd= 128 新宿↔唐木田 / 771 北綾瀬↔唐木田 / 154 取手↔唐木田)line_group_cd= 779 を削除 — 128 と同じ新宿↔唐木田の急行を「新百合ヶ丘から各駅停車へ種別変更」として重複表現していたもの。改正後の実態は急行のまま多摩線内各駅停車なので、種別変更としての表現は誤り。128 を直したことで役目が無くなったline_group_cd= 125 / 155 / 772)line_group_cd= 269 の開成を停車へ(Issue には未記載だが同じ改正の反映漏れ)line_group_cd= 1008)。ふじさんの小田急線内の停車駅は秦野の次が松田(御殿場線)で、新松田には停車しないJR
line_group_cd= 140 / 904)。2025-03-15 改正で南小谷乗り入れが廃止されたため、信濃森上・白馬大池・千国・南小谷の 4 駅を削除pass= 2)から一部停車(pass= 5)へ(line_group_cd= 400)。実態は上り 8・10 号のみの停車で、基本は通過側京王
line_group_cd= 71 新宿↔京王八王子 / 415 新宿↔高尾山口)。東京競馬開催日に上下とも臨時停車があるline_group_cd= 764 の東府中(京王線側・競馬場線側の両方)を通過から停車へ。競馬場線へ入る系統が分岐駅を通過扱いにしていた誤り検証
cargo run -p data_validator→[VALID] No errors reported.対応していない項目
Issue #1656 の以下 5 項目は、本番 API を直接叩いた限り StationAPI 側では再現しなかった。MobileApp 側の切り分けが必要なため本 PR には含めない。
lineGroupStations(599)は武豊→大府→名古屋→岐阜の 25 駅を正しく返すlineGroupStations(499)は品川→原ノ町の 77 駅を正しく返し、他の常磐線系統(38 / 39 / 116 / 591)と同じ構成line_group_cd= 91、池袋〜方南町)はデータ上存在するline_group_cd= 361 / 362 とも「国際会館」で統一されているstation_cd= 2900401 で正しく設定されているテスト
make fmtが通ることmake clippyが通ること(wasm32 ターゲットを含む)make testが通ることコード変更が無いためチェック欄は OFF。実際には
make fmt/make clippy/make test(11 tests passed)とcargo run -p data_validatorを実行し、いずれも成功している。関連Issue
Refs #1656
(Issue の 11 項目のうち 6 項目を解決。残る 5 項目は上記のとおり StationAPI 側では再現しないため、Issue はクローズしない)
スクリーンショット(任意)