Skip to content

feat: estimate whether users can catch nearby buses - #93

Open
steventeng2022 wants to merge 4 commits into
YetAnotherBusDeveloper:mainfrom
steventeng2022:codex/issue-81-catchability
Open

steventeng2022 wants to merge 4 commits into
YetAnotherBusDeveloper:mainfrom
steventeng2022:codex/issue-81-catchability

Conversation

@steventeng2022

Copy link
Copy Markdown
Contributor

痾 這好像版本不一樣

# Conflicts:
#	lib/screens/nearby_screen.dart
@steventeng2022
steventeng2022 force-pushed the codex/issue-81-catchability branch from bea4c58 to 90780bc Compare September 23, 2026 14:46

@Av1anJay Av1anJay left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 Av1anJay left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants