feat: estimate whether users can catch nearby buses - #93
Open
steventeng2022 wants to merge 4 commits into
Open
steventeng2022 wants to merge 4 commits into
steventeng2022 wants to merge 4 commits into
Conversation
# Conflicts: # lib/screens/nearby_screen.dart
steventeng2022
force-pushed
the
codex/issue-81-catchability
branch
from
September 23, 2026 14:46
bea4c58 to
90780bc
Compare
Av1anJay
requested changes
Sep 23, 2026
Av1anJay
left a comment
Collaborator
There was a problem hiding this comment.
Code Review Summary
Verdict: Changes requested — 1 個會造成使用者誤判的 correctness 問題。
⚠️ Blocking
lib/screens/nearby_screen.dart:479— 這裡把原始的liveStop.sec傳給判斷器,但同一列的EtaBadge會透過effectiveStopEtaSeconds(stop),依stop.t扣掉資料取得後已經過的秒數。當即時資料稍舊時,兩者會直接矛盾:例如原始 ETA 5 分鐘、資料已過 4 分鐘、步行約 2 分鐘時,徽章會顯示約 1 分鐘/即將進站,但新標籤仍可能顯示「預估來得及,約剩餘 3 分鐘」。請改用effectiveStopEtaSeconds(liveStop)作為 estimator 輸入,並補一個帶舊t的回歸測試,確保標籤與 ETA 徽章使用同一個有效到站時間。
Checks
git diff --check origin/main...HEAD:通過- GitHub Actions
Verify(同一 head SHA):flutter analyze、flutter test、Chrome route geometry test 全部通過 - GitHub Actions builds:Android、Wear OS、Web、Windows、Linux、macOS、iOS 全部通過
其餘 estimator 邊界處理、保守緩衝與新增單元測試看起來合理。
Av1anJay
approved these changes
Sep 23, 2026
Av1anJay
left a comment
Collaborator
There was a problem hiding this comment.
Code Review Summary
Verdict: Approved
重新檢查目前 head aff53bc 後,前次阻擋問題已修正:catchability 現在與 EtaBadge 一樣使用 effectiveStopEtaSeconds(liveStop),且新增舊即時資料時間戳的回歸測試。完整 diff 與相關 ETA/附近站牌程式碼未發現其他具體阻擋缺陷。
Checks
git diff --check origin/main...HEAD:通過- GitHub Actions
Verify(目前 head):flutter analyze、flutter test、Chrome route geometry test 全部通過 - GitHub Actions builds:Android、Wear OS、Web、Windows、Linux、macOS、iOS 全部通過
- 本機沒有 Flutter SDK,因此未重跑 Flutter 指令;以上 Flutter 結果取自目前 head 的 Actions log
Reviewed by Hermes Agent
…bility # Conflicts: # lib/screens/nearby_screen.dart
This branch has not been 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.
痾 這好像版本不一樣