Skip to content

trainRouteのEstimatedモードと線路の長さによる到着時間推定を本番へ反映 - #1716

Merged
TinyKitten merged 5 commits into
masterfrom
dev
Oct 1, 2026
Merged

TinyKitten merged 5 commits into
masterfrom
dev

Conversation

@TinyKitten

Copy link
Copy Markdown
Member

概要

dev の変更を master(本番)へ反映します。

変更の種類

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

変更内容

デプロイ時の注意

  • 本番の estimateArrivalTimes の値が変わります。 線路の長さと求め直した較正で推定するためです。connectedRoutes の結果と trainRoute(model 省略時)の値は変わりません。
  • N02 は 駅間の線路の長さを返す機能を本番へ反映 #1706 の本番デプロイで取得済みです。キャッシュが無効なら、国土数値情報のサーバーから再取得します。
  • デプロイ後の確認: make schema で SDL が一致すること、trainRoute(..., model: Estimated) の arrivalCumulativeMinutes が埋まること、model を省いたときに null のままであることを確かめてください。

テスト

関連Issue

Refs #1709, #1704

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

🤖 Generated with Claude Code

TinyKitten and others added 5 commits October 1, 2026 01:05
* 到着時間推定を実際の所要時間と比べるベンチマークを追加し、CIで推定の悪化を検出できるようにした

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* 所要時間ベンチマークで記録済みの基準を推定できなくなったら失敗させ、generatedのデータでは記録と比べないようにした

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* 所要時間ベンチマークの記録を生成データで作り、build_worker.ymlで生成データを作ってから比べるようにした

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* 所要時間ベンチマークに典型的な所要時間の列を足し、回帰の見張りを範囲からの外れではなく典型値からのずれで判定するようにした

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…dRoutesを元の計算に戻した (#1714)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
@TinyKitten TinyKitten self-assigned this Oct 1, 2026
@coderabbitai

coderabbitai Bot commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Essentials

Run ID: 4b98ee10-2ed5-4551-a43d-fe99d71a2e55

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


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

@TinyKitten
TinyKitten merged commit 3a67638 into master Oct 1, 2026
13 checks passed

This branch was successfully deployed

1 active deployment
staging — 3dd6ab32 Deployed Oct 1, 2026 by TinyKitten via Build and deploy to staging #30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant