Skip to content

高尾線の京王ライナーを新宿~高尾山口の独立系統として追加 - #1724

Merged
TinyKitten merged 3 commits into
devfrom
feature/keio-takao-liner
Oct 16, 2026
Merged

TinyKitten merged 3 commits into
devfrom
feature/keio-takao-liner

Conversation

@TinyKitten

@TinyKitten TinyKitten commented Oct 16, 2026 •

Copy link
Copy Markdown
Member

概要

未収録だった高尾線の京王ライナーを追加し、新宿~高尾山口で独立した種別として手動選択・経路表示できるようにします。停車駅修正の #1723 とは別のPRです。#1723 のマージ後のdevを取り込み、既存監査テストも高尾線ライナーを含む32系統に更新しています。

変更の種類

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

変更内容

  • 種別413「京王ライナー(高尾山口方面)」と路線グループ1188を追加。
  • 新宿~北野~高尾山口の全38行を登録。北野は京王線・高尾線の両方の駅IDを収録。
  • 上下とも新宿・明大前・府中・分倍河原・聖蹟桜ヶ丘・高幡不動・北野・めじろ台・高尾・高尾山口に停車。京王片倉・山田・狭間は通過。
  • 既存の京王ライナーと同じ方向0・kind=2を使用し、Mt.TAKAO号とは別種別として提供。
  • 時間帯・方向別乗降制限があるため、requires_timetable=trueを設定。時刻表なしのconnectedRoutes・到達可能性検索から除外し、手動選択用には保持。
  • 全駅順・停車/通過・種別メタデータ・時刻表必須ルールを回帰テストで確認。
  • docs/route-service-audit.mdに根拠と運用上の扱いを追記。

公式資料(2026-10-16確認):

テスト

  • make fmt が通ること
  • make clippy が通ること(wasm32 ターゲットを含む)
  • make test が通ること
  • cargo run -p data_validator: No errors reported
  • cargo test -p data_validator --test keio_takao_liner: 1 passed
  • make ipa-audit: train_types 309件・未解決0件
  • git diff --check
  • 京王線系統の停車駅を照合し急行・京王ライナーの停車漏れを修正 #1723 統合後の既存停車駅監査と高尾線ライナー専用テストがともに成功

すべて手元のRust環境で実行・成功。実行時のフォールバックデータはcanonical CSVで、GTFS・N02を用いるgeneratedデータの再構築はCIで確認します。

関連Issue

ユーザー依頼「別PRで高尾線の京王ライナーも追加」。
関連: #1723

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

データ追加のためなし。

Summary by CodeRabbit

  • ドキュメント

    • 高尾線の京王ライナーを監査対象に追加し、停車パターンや駅IDの扱い、利用できる経路検索の範囲を記録しました。
    • 回帰検査の対象範囲に、高尾線の京王ライナーが含まれることを明記しました。
  • テスト

    • 高尾ライナーの種別、方向、停車駅、運行規則が期待どおりであることを確認する検証を追加しました。
    • 駅区間の監査対象にグループ1188を追加しました。

@github-actions github-actions Bot added feature 要望対応や課題解決 deploy-dev rust and removed feature 要望対応や課題解決 labels 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: dd28c8b0-f527-48cb-9f48-d6ded9f9876d
📥 Commits

Reviewing files that changed from the base of the PR and between 6931d52 and 473d402.

⛔ Files ignored due to path filters: 3
  • data/4!types.csv is excluded by !**/*.csv
  • data/5!station_station_types.csv is excluded by !**/*.csv
  • data/route_service_rules.csv is excluded by !**/*.csv
📒 Files selected for processing: 4
  • data_validator/tests/keio_stopping_patterns.rs
  • data_validator/tests/keio_takao_liner.rs
  • docs/keio-stopping-pattern-audit.md
  • docs/route-service-audit.md

Included review availability: This review used your included allowance. 3 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

路線グループ1188を停車パターン監査の対象に追加しました。種別413の駅設定と運行規則を検証するテストを追加し、関連する監査ドキュメントを更新しました。

Changes: 高尾山口方面の京王ライナー

Layer / File(s) Summary
停車パターンの検証と監査記録
data_validator/tests/keio_stopping_patterns.rs, data_validator/tests/keio_takao_liner.rs, docs/keio-stopping-pattern-audit.md, docs/route-service-audit.md
グループ1188を既存の駅区間と監査対象に加えました。種別413のメタデータ、駅一覧、停車設定、運行規則を検証するテストを追加しました。監査ドキュメントに停車駅、通過駅、時刻表要件、探索での扱いを記載しました。

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Feature

Merge Risk: ⚪ Minimal · up to 473d4

No outstanding issue was established in the changed validation tests or audit records. The change is mergeable after normal checks.

Pre-merge checks: Passed 4 · Failed 1

Failed Failed checks: 1 warning

Check name Status Explanation Resolution
Docstring Coverage Warning Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 4 functions across 2 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
Title check Passed タイトルは、高尾線の京王ライナーを独立系統として追加する主な変更を明確に示しています。
Description check Passed 概要、変更種類、変更内容、テスト、関連Issue、任意項目を含み、実施した検証結果と運用上の扱いも具体的に記載されています。
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.
Full details: Docstring Coverage

Explanation

Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 4 functions across 2 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

うさぎは時刻表をそっと開く
駅の並びをひとつずつ確かめる
止まる駅には丸をつけ
通過する駅も見落とさない
1188の列を見届けて
月明かりの下で耳を揺らす

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

@TinyKitten TinyKitten self-assigned this Oct 16, 2026
@github-actions

Copy link
Copy Markdown

🚉 停車駅の変更

1 件の新しい停車パターン

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

🆕 京王ライナー(高尾山口方面) (Keio Liner(for Takaosanguchi)) - 京王線 / 京王高尾線

type_cd=413, line_group_cd=1188

変更内容: 38 駅追加

全駅リスト (38駅)
# 駅名 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 北野 Kitano ●
33 京王片倉 Keio-Katakura ○
34 山田 Yamada ○
35 めじろ台 Mejirodai ●
36 狭間 Hazama ○
37 高尾 Takao ●
38 高尾山口 Takaosanguchi ●

@github-actions github-actions Bot added the feature 要望対応や課題解決 label Oct 16, 2026
@TinyKitten
TinyKitten merged commit 72b0e3a into dev Oct 16, 2026
14 checks passed
@TinyKitten
TinyKitten deleted the feature/keio-takao-liner branch October 16, 2026 00:44
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