Repository navigation
駅間の線路の長さを返す機能を本番へ反映 - #1706
Merged
Merged
駅間の線路の長さを返す機能を本番へ反映#1706
Conversation
* 隣り合う駅のあいだの線路の長さを国土数値情報から求め、Station.trackDistanceFromPreviousで返すようにした Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * N02の取得にタイムアウトを明示し、CIでN02をキャッシュするようにした Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Essentials Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
4 of 9 tasks
5 of 10 tasks
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
dev の変更を master(本番)へ反映します。
Station.trackDistanceFromPreviousで返します(隣り合う駅のあいだの線路の長さを返せるようにした #1705)。変更の種類
変更内容
Station/StationNestedにtrackDistanceFromPrevious: Floatを追加しました。スキーマへの追加だけで、既存のフィールドは変わりません。lineStations・lineGroupStations・trainRouteだけです。nullになります。trainRouteのdistanceFromPrevious(直線距離)は変えていません。generated/connections.csvに書き出します。connections.binを二分探索して値を引きます。起動時に索引は作りません(wasm は約 130KB 増えます)。data/8!connections.csvを線路の長さの手修正に使うようにし、data_validatorで検査します。actions/cacheにキャッシュします。取得に失敗すると preprocessor が失敗します。デプロイ時の注意
actions/cacheは、devで作られたキャッシュをmasterから使えません。そのため初回の本番デプロイでは、N02(約 13MB)を国土数値情報のサーバーから取得します。サーバーが応答しないとデプロイは失敗します。その場合は再実行してください。lineStations(lineId: 11302) { name trackDistanceFromPrevious }で、大崎がnull、五反田が 806、目黒が 1190 になることを確かめてください。テスト
make fmtが通ることmake clippyが通ること(wasm32 ターゲットを含む)make testが通ること反映する内容は 隣り合う駅のあいだの線路の長さを返せるようにした #1705 のブランチ(f5db66f)と同じです(
git diff origin/dev feature/track-distanceは差分なし)。そのブランチで上の 3 項目とcargo run -p data_validator([VALID] No errors reported.)を通しています。隣り合う駅のあいだの線路の長さを返せるようにした #1705 の CI もすべて成功しています。worker-build --release→wrangler devで、make schemaの一致(32 型)と、lineStations/trainRoute/stationの応答を確認しています。関連Issue
Refs #1704
スクリーンショット(任意)
🤖 Generated with Claude Code