master<-dev#1523
Merged
Merged
Conversation
`build_stop_route_mapping()` の `prev_chain` / `next_chain` 再帰 CTE が `depth < 10` の固定値で打ち切られていたため、メイン系統に戻るまでに 11 停留所以上ある variant 経路ではアンカー解決に失敗し、`estimated_seq` が末尾フォールバック (`mtms.max_seq + 9999`) や「next のみ解決」分岐に 落ちて停留所順が崩れていた。 route ごとの `variant_only_with_neighbors` の行数から `max_depth` を 計算する `variant_chain_limit` CTE を追加し、`prev_chain` / `next_chain` の上限をこれに置き換える。`visited` 配列で各 variant 停留所は 1 度しか 踏まないことが保証されているため、stop 数 + 1 はサイクル防止と整合した タイトな上限になる。 長い variant チェーン (M01 → V01..V12 → M15) のシナリオで V02..V11 が 両端を再帰解決して中央付近に挿入されることを検証する SQL 回帰テスト `test_build_stop_route_mapping_long_variant_chain_dynamic_limit` を追加。 Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* variant補間の再帰深さ上限をroute毎の停留所数から動的に算出 (Closes #1513) `build_stop_route_mapping()` の `prev_chain` / `next_chain` 再帰 CTE が `depth < 10` の固定値で打ち切られていたため、メイン系統に戻るまでに 11 停留所以上ある variant 経路ではアンカー解決に失敗し、`estimated_seq` が末尾フォールバック (`mtms.max_seq + 9999`) や「next のみ解決」分岐に 落ちて停留所順が崩れていた。 route ごとの `variant_only_with_neighbors` の行数から `max_depth` を 計算する `variant_chain_limit` CTE を追加し、`prev_chain` / `next_chain` の上限をこれに置き換える。`visited` 配列で各 variant 停留所は 1 度しか 踏まないことが保証されているため、stop 数 + 1 はサイクル防止と整合した タイトな上限になる。 長い variant チェーン (M01 → V01..V12 → M15) のシナリオで V02..V11 が 両端を再帰解決して中央付近に挿入されることを検証する SQL 回帰テスト `test_build_stop_route_mapping_long_variant_chain_dynamic_limit` を追加。 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * ECSデプロイワークフローの paths フィルタに docker/** を追加 (Closes #1500) Dockerfile や docker/ 配下のアセットを変更してもステージング/本番のデプロイが自動トリガーされなかったため、両ワークフローの on.push.paths に docker/** を追加する。 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.7 (1M context) <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: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
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
1 participant
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 へ反映するリリース PR。
variant補間の再帰深さ上限の動的算出、build_stop_route_mappingの SQL 回帰テスト基盤、ECS デプロイワークフローの整備を含む。変更の種類
変更内容
variant補間の再帰深さ上限を route ごとの停留所数から動的に算出 (variant補間の再帰深さ上限をroute毎の停留所数から動的に算出 (Closes #1513) #1520)build_stop_route_mappingの SQL 回帰テスト基盤を整備 (build_stop_route_mapping のSQL回帰テスト基盤を整備 #1519)pathsフィルタにdocker/**を追加 (ECSデプロイワークフローの paths フィルタに docker/** を追加 #1521)usesをコミット SHA にピン留め (GitHub ActionsのusesをコミットSHAにピン留め (Closes #1499) #1522)テスト
cargo fmt --all -- --checkが通ることcargo clippy -- -D warningsが通ることcargo test（SQLX_OFFLINE=true）が通ること関連Issue
#1499 #1513 #1514
スクリーンショット（任意）