Skip to content

京王線系統の停車駅を照合し急行・京王ライナーの停車漏れを修正 - #1723

Merged
TinyKitten merged 2 commits into
devfrom
feature/keio-stopping-pattern-audit
Oct 16, 2026
Merged

TinyKitten merged 2 commits into
devfrom
feature/keio-stopping-pattern-audit

Conversation

@TinyKitten

@TinyKitten TinyKitten commented Oct 16, 2026 •

Copy link
Copy Markdown
Member

概要

京王線系統の既存31路線グループ・936行を公式53駅の停車種別と照合し、急行と京王ライナーの停車漏れ6行を修正します。都営新宿線直通・相模原線・高尾線・競馬場線・動物園線も対象です。

変更の種類

  • バグ修正
  • 新機能
  • データの修正・追加
  • リファクタリング
  • ドキュメント
  • CI/CD
  • その他

変更内容

  • 急行285・757・760のつつじヶ丘を通過から停車に修正。70・414・763・766は既に停車であり維持。
  • 京王ライナー65の明大前・北野、66の明大前を通過から停車に修正。
  • 公式53駅の参照URL・確認日付きfixtureを追加。各行の種別で照合し、都営線内各停→京王区間急行などの種別変更も検査。
  • 既存31グループの存在・全駅ID列・順序・接続駅両側のIDを回帰テストで検証。
  • Mt.TAKAO上下別停車パターン、東府中の特急一部停車と競馬場直通停車は維持。
  • 根拠と確認範囲は docs/keio-stopping-pattern-audit.md に記載。

根拠:

補足: 公式案内には高尾線の京王ライナーもありますが、現行CSVには専用路線グループがありません。本変更は既存パターンの停車駅修正であり、その系統追加は含みません。

テスト

  • Pythonで公式fixtureと既存31グループ・936行を全件照合。不一致0件。
  • Pythonで各グループの駅ID列・接続駅・順序を照合。
  • Pythonで全 station_station_types の駅ID・種別IDの参照整合性を確認。
  • make fmt が通ること(CIの cargo fmt --all -- --check)
  • make clippy が通ること(CIのnative/wasm32 Clippy)
  • make test が通ること(CIのnative/workerテスト)

ローカルにはcargoがなく実行不可(command not found)のためCIで確認。最終コミット 7c197f6 で Check・Rustfmt・Clippy・Test Suite・Verify pushed migration data が成功。追加した京王線回帰テスト、cargo run -p data_validator も成功。Build data and worker は実行中。

関連Issue

ユーザーからの京王線全体の停車駅整合性確認依頼。

スクリーンショット(任意)

データ修正のためなし。

Summary by CodeRabbit

  • ドキュメント
    • 京王線の停車パターン監査結果と、公式情報との照合方法を記録した文書を追加しました。
    • READMEから監査・回帰チェックの文書にアクセスできるようになりました。
  • テスト
    • 京王線の停車駅データについて、対象グループの駅順と停車パターンを検証するチェックを追加しました。

@github-actions github-actions Bot added feature 要望対応や課題解決 deploy-dev rust labels Oct 16, 2026
@TinyKitten TinyKitten self-assigned this Oct 16, 2026
@coderabbitai

coderabbitai Bot commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: c83ada20-1555-492b-9ca2-1bd1c9b05987
📥 Commits

Reviewing files that changed from the base of the PR and between 9ab519b and 7c197f6.

⛔ Files ignored due to path filters: 2
  • data/5!station_station_types.csv is excluded by !**/*.csv
  • data_validator/tests/fixtures/keio_stops.csv is excluded by !**/*.csv
📒 Files selected for processing: 3
  • README.md
  • data_validator/tests/keio_stopping_patterns.rs
  • docs/keio-stopping-pattern-audit.md

Included review availability: This review used your included allowance. 4 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.

📝 Walkthrough

京王線系統の停車パターンを照合する回帰テストを追加しました。監査結果と検査範囲を文書に記録し、READMEから参照できるようにしました。

Changes: 京王停車パターン監査

Layer / File(s) Summary
停車パターン回帰検査
data_validator/tests/keio_stopping_patterns.rs
公式停車パターン53件を読み込み、対象グループの停車値、グループ集合、駅IDの順序を検証するテストを追加しました。グループ758と駅番号23の特急には個別の期待値を設定しています。
監査記録と文書リンク
docs/keio-stopping-pattern-audit.md, README.md
文書に既存31グループ・936行の照合結果、記録された停車設定の修正内容、検査方法と対象範囲を記載しました。READMEに文書へのリンクを追加しました。

Priority: ➖ Normal

Estimated code review effort: 3 (Moderate) | ~20 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 7c197

The corrected stops are covered by the new regression test. No issue requiring a fix before merge was identified; normal CI checks remain appropriate.

Pre-merge checks: Passed 4 · Failed 1

Failed Failed checks: 1 warning

Check name Status Explanation Resolution
Docstring Coverage Warning Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 1 files. (2 skipped: 2… Write docstrings for the functions missing them to satisfy the coverage threshold.
Passed Passed checks: 4 passed
Check name Status Explanation
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Title check Passed タイトルは、京王線系統の停車駅照合と急行・京王ライナーの停車漏れ修正という主要変更を明確に示しています。
Description check Passed 概要、変更種類、変更内容、テスト、関連Issue、スクリーンショットの各テンプレート項目を記載しています。テストはローカル未実施ですが、CIでの確認結果と実行できなかった理由を明記しています。
Full details: Docstring Coverage

Explanation

Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 1 files. (2 skipped: 2 unsupported.)

  • Fix all pre-merge checks with AI
✨ Finishing Touches: 💡 1
📝 Generate docstrings: 💡
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

うさぎは駅の表を見比べる
ぴょんと停車駅を数えてみる
順序もグループも確かめた
監査の文書へ道しるべ
月夜に回帰テストを祝う

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

🚉 停車駅の変更

5 件の変更された停車パターン

● 停車 / ○ 通過 / △ 一部通過 / ◆ 平日停車 / ◇ 休日停車 / ▲ 一部停車

✏️ 急行 (Express) - 京王新線 / 京王線 / 京王相模原線

type_cd=382, line_group_cd=285

変更内容: つつじヶ丘: ○ → ●

全駅リスト (31駅)
# 駅名 Station 停車
1 新線新宿 Shinjuku ●
2 初台 Hatsudai ●
3 幡ヶ谷 Hatagaya ●
4 笹塚 Sasazuka ●
5 笹塚 Sasazuka ●
6 代田橋 Daitabashi ○
7 明大前 Meidaimae ●
8 下高井戸 Shimo-takaido ○
9 桜上水 Sakurajosui ●
10 上北沢 Kami-kitazawa ○
11 八幡山 Hachiman-yama ○
12 芦花公園 Roka-koen ○
13 千歳烏山 Chitose-karasuyama ●
14 仙川 Sengawa ○
15 つつじヶ丘 Tsutsujigaoka ●
16 柴崎 Shibasaki ○
17 国領 Kokuryo ○
18 布田 Fuda ○
19 調布 Chofu ●
20 調布 Chofu ●
21 京王多摩川 Keio-tamagawa ○
22 京王稲田堤 Keio-inadazutumi ●
23 京王よみうりランド Keio-yomiuri-land ○
24 稲城 Inagi ○
25 若葉台 Wakabadai ○
26 京王永山 Keio-nagayama ●
27 京王多摩センター Keio-tama-center ●
28 京王堀之内 Keio-horinouchi ○
29 南大沢 Minami-osawa ●
30 多摩境 Tamasakai ○
31 橋本 Hashimoto ●

✏️ 急行 (Express) - 京王新線 / 京王線 / 京王相模原線

type_cd=382, line_group_cd=757

変更内容: つつじヶ丘: ○ → ●

全駅リスト (31駅)
# 駅名 Station 停車
1 新線新宿 Shinjuku ●
2 初台 Hatsudai ●
3 幡ヶ谷 Hatagaya ●
4 笹塚 Sasazuka ●
5 笹塚 Sasazuka ●
6 代田橋 Daitabashi ○
7 明大前 Meidaimae ●
8 下高井戸 Shimo-takaido ○
9 桜上水 Sakurajosui ●
10 上北沢 Kami-kitazawa ○
11 八幡山 Hachiman-yama ○
12 芦花公園 Roka-koen ○
13 千歳烏山 Chitose-karasuyama ●
14 仙川 Sengawa ○
15 つつじヶ丘 Tsutsujigaoka ●
16 柴崎 Shibasaki ○
17 国領 Kokuryo ○
18 布田 Fuda ○
19 調布 Chofu ●
20 調布 Chofu ●
21 京王多摩川 Keio-tamagawa ○
22 京王稲田堤 Keio-inadazutumi ●
23 京王よみうりランド Keio-yomiuri-land ○
24 稲城 Inagi ○
25 若葉台 Wakabadai ○
26 京王永山 Keio-nagayama ●
27 京王多摩センター Keio-tama-center ●
28 京王堀之内 Keio-horinouchi ○
29 南大沢 Minami-osawa ●
30 多摩境 Tamasakai ○
31 橋本 Hashimoto ●

✏️ 急行 (Express) - 京王線 / 京王相模原線

type_cd=382, line_group_cd=760

変更内容: つつじヶ丘: ○ → ●

全駅リスト (28駅)
# 駅名 Station 停車
1 新宿 Shinjuku ●
2 笹塚 Sasazuka ●
3 代田橋 Daitabashi ○
4 明大前 Meidaimae ●
5 下高井戸 Shimo-takaido ○
6 桜上水 Sakurajosui ●
7 上北沢 Kami-kitazawa ○
8 八幡山 Hachiman-yama ○
9 芦花公園 Roka-koen ○
10 千歳烏山 Chitose-karasuyama ●
11 仙川 Sengawa ○
12 つつじヶ丘 Tsutsujigaoka ●
13 柴崎 Shibasaki ○
14 国領 Kokuryo ○
15 布田 Fuda ○
16 調布 Chofu ●
17 調布 Chofu ●
18 京王多摩川 Keio-tamagawa ○
19 京王稲田堤 Keio-inadazutumi ●
20 京王よみうりランド Keio-yomiuri-land ○
21 稲城 Inagi ○
22 若葉台 Wakabadai ○
23 京王永山 Keio-nagayama ●
24 京王多摩センター Keio-tama-center ●
25 京王堀之内 Keio-horinouchi ○
26 南大沢 Minami-osawa ●
27 多摩境 Tamasakai ○
28 橋本 Hashimoto ●

✏️ 京王ライナー(京王八王子方面) (Keio Liner(for Keiō-hachiōji)) - 京王線

type_cd=400, line_group_cd=65

変更内容: 北野: ○ → ● | 明大前: ○ → ●

全駅リスト (32駅)
# 駅名 Station 停車
1 新宿 Shinjuku ●
2 笹塚 Sasazuka ○
3 代田橋 Daitabashi ○
4 明大前 Meidaimae ●
5 下高井戸 Shimo-takaido ○
6 桜上水 Sakurajosui ○
7 上北沢 Kami-kitazawa ○
8 八幡山 Hachiman-yama ○
9 芦花公園 Roka-koen ○
10 千歳烏山 Chitose-karasuyama ○
11 仙川 Sengawa ○
12 つつじヶ丘 Tsutsujigaoka ○
13 柴崎 Shibasaki ○
14 国領 Kokuryo ○
15 布田 Fuda ○
16 調布 Chofu ○
17 西調布 Nishi-chofu ○
18 飛田給 Tobitakyu ○
19 武蔵野台 Musashinodai ○
20 多磨霊園 Tama-reien ○
21 東府中 Higashi-fuchu ○
22 府中 Fuchu ●
23 分倍河原 Bubaigawara ●
24 中河原 Nakagawara ○
25 聖蹟桜ヶ丘 Seiseki-sakuragaoka ●
26 百草園 Mogusaen ○
27 高幡不動 Takahatafudo ●
28 南平 Minamidaira ○
29 平山城址公園 Hirayamajoshi-koen ○
30 長沼 Naganuma ○
31 北野 Kitano ●
32 京王八王子 Keio-hachioji ●

✏️ 京王ライナー(橋本方面) (Keio Liner(for Hashimoto)) - 京王線 / 京王相模原線

type_cd=401, line_group_cd=66

変更内容: 明大前: ○ → ●

全駅リスト (28駅)
# 駅名 Station 停車
1 新宿 Shinjuku ●
2 笹塚 Sasazuka ○
3 代田橋 Daitabashi ○
4 明大前 Meidaimae ●
5 下高井戸 Shimo-takaido ○
6 桜上水 Sakurajosui ○
7 上北沢 Kami-kitazawa ○
8 八幡山 Hachiman-yama ○
9 芦花公園 Roka-koen ○
10 千歳烏山 Chitose-karasuyama ○
11 仙川 Sengawa ○
12 つつじヶ丘 Tsutsujigaoka ○
13 柴崎 Shibasaki ○
14 国領 Kokuryo ○
15 布田 Fuda ○
16 調布 Chofu ○
17 調布 Chofu ○
18 京王多摩川 Keio-tamagawa ○
19 京王稲田堤 Keio-inadazutumi ○
20 京王よみうりランド Keio-yomiuri-land ○
21 稲城 Inagi ○
22 若葉台 Wakabadai ○
23 京王永山 Keio-nagayama ●
24 京王多摩センター Keio-tama-center ●
25 京王堀之内 Keio-horinouchi ○
26 南大沢 Minami-osawa ●
27 多摩境 Tamasakai ○
28 橋本 Hashimoto ●

@TinyKitten
TinyKitten merged commit 6931d52 into dev Oct 16, 2026
14 checks passed
@TinyKitten
TinyKitten deleted the feature/keio-stopping-pattern-audit branch October 16, 2026 00:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deploy-dev feature 要望対応や課題解決 rust

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant