Skip to content

Increase timeout for daily analysis job to 300 minutes - #1229

Closed
hkbmw1 wants to merge 1 commit into
ZhuLinsen:mainfrom
hkbmw1:main
Closed

Increase timeout for daily analysis job to 300 minutes#1229
hkbmw1 wants to merge 1 commit into
ZhuLinsen:mainfrom
hkbmw1:main

Conversation

@hkbmw1

@hkbmw1 hkbmw1 commented May 8, 2026

Copy link
Copy Markdown

Increased the timeout limit for the analysis job to prevent task failures.

PR Type

  • fix
  • feat
  • refactor
  • docs
  • chore
  • test

Background And Problem

请描述当前问题、影响范围与触发场景。
(EN) Describe the problem, its impact, and what triggers it.

Scope Of Change

请列出本 PR 修改的模块和文件范围。
(EN) List the modules and files changed in this PR.

Issue Link

必须填写以下之一 / Fill in one of:

  • Fixes #<issue_number>
  • Refs #<issue_number>
  • 无 Issue 时说明原因与验收标准 / If no issue, explain the motivation and acceptance criteria

Verification Commands And Results

请填写你实际执行过的命令和关键结果(不要只写"已测试")。
(EN) Paste the commands you actually ran and their key output (don't just write "tested"):

# example
./scripts/ci_gate.sh
python -m pytest -m "not network"

关键输出/结论 / Key output & conclusion:

Compatibility And Risk

请说明兼容性影响、潜在风险(如无请写 None)。
(EN) Describe compatibility impact and potential risks (write None if not applicable).

  • 若本 PR 修改第三方模型 / API 的兼容语义、请求参数、路由前缀或 provider fallback,请提供官方来源链接或公告,并说明这是长期约束、当前运行时约束还是临时兼容处理。
    (EN) If this PR changes third-party model/API compatibility, request parameters, routing prefixes, or provider fallback behavior, include an official source link or announcement and clarify whether the rule is permanent, runtime-specific, or a temporary compatibility workaround.)
  • 若本 PR 依赖特定运行时 / 锁定依赖窗口(例如 LiteLLM 版本范围、OpenAI-compatible 路由、YAML alias 行为),请写明当前验证过的兼容范围与覆盖路径。
    (EN) If this PR depends on a specific runtime or pinned dependency window (for example a LiteLLM version range, OpenAI-compatible routing, or YAML alias behavior), state the compatibility window you verified and which code paths were covered.)
  • 若本 PR 触及运行时配置保存、清理、迁移或回填逻辑,请明确说明旧配置是否会被自动改写、清空、迁移或保持不变,以及用户如何恢复原行为。
    (EN) If this PR touches runtime config save/cleanup/migration/backfill logic, explicitly describe whether existing config is rewritten, cleared, migrated, or left intact, and how users can restore the previous behavior.)

Rollback Plan

请至少写一句可执行的回滚方案(必填)。
(EN) Provide at least one actionable rollback step (required).

  • 如果是兼容性修复,默认应写出最小回滚方式(例如 revert this PR),并说明是否需要额外回滚配置或数据迁移。
    (EN) For compatibility fixes, include the minimal rollback path (for example revert this PR) and whether any additional config or data rollback is required.)

EXTRACT_PROMPT Change (if applicable)

若本 PR 修改了 src/services/image_stock_extractor.py 中的 EXTRACT_PROMPT,请在此处粘贴完整变更后的 prompt。
If this PR changes EXTRACT_PROMPT in src/services/image_stock_extractor.py, paste the full updated prompt here:

展开 / Expand: Full EXTRACT_PROMPT
(paste full prompt here)

Checklist

  • 本 PR 有明确动机和业务价值 / This PR has a clear motivation and value
  • 已提供可复现的验证命令与结果 / Reproducible verification commands and results are included
  • 已评估兼容性与风险 / Compatibility and risk have been assessed
  • 已提供回滚方案 / A rollback plan is provided
  • 若涉及用户可见变更,已同步更新相关文档与 docs/CHANGELOG.mdREADME.md 仅在首页级信息变化时更新,细节优先写入 docs/*.md / If user-visible changes are included, relevant docs and docs/CHANGELOG.md are updated; README.md is updated only for homepage-level changes, with details kept in docs/*.md

Increased the timeout limit for the analysis job to prevent task failures.
@github-actions

github-actions Bot commented May 8, 2026

Copy link
Copy Markdown

🤖 自动审查报告

项目 结果
📊 变更文件 1 个
➕ 新增行数 1 行
➖ 删除行数 1 行
🔍 静态检查 ✅ 通过
🧠 AI 审查 ✅ 已完成

📁 修改的文件

  • 📝 .github/workflows/daily_analysis.yml (+1/-1)

🧠 AI 代码审查意见

结论

Not Ready

结构化结果

  • 必要性通过。PR 描述指出增加超时是为了“prevent task failures”,动机明确,旨在解决或预防日常分析任务因超时而失败的问题,具有业务价值。

  • 关联性不通过Issue Link 字段为空,未关联任何 Issue,也未说明无 Issue 时的动机与验收标准。这违反了审查规则中的强制要求。

  • 类型:建议 fixchore。目前选择 fix 是可以接受的,因为它旨在解决或预防任务失败。

  • 描述完整性不完整

    • 缺少 Background And Problem
    • 缺少 Scope Of Change(尽管文件列表已提供)。
    • 缺少 Issue Link
    • 缺少 Verification Commands And Results
    • 缺少 Rollback Plan
  • 风险级别。本次修改仅涉及 GitHub Actions workflow 的超时时间设置,将 timeout-minutes 从 30 增加到 300。此改动不影响核心业务逻辑、兼容性或安全性,因此评估为 None 是合理的。


必改项

  1. 缺少回滚方案。 Rollback Plan 是必填项,请至少提供一句可执行的回滚方案。例如:revert this PR
  2. 缺少 Issue 关联或说明。 Issue Link 字段是必填项。请关联一个 Issue (例如 Fixes #<issue_number>Refs #<issue_number>),或说明无 Issue 时的动机与验收标准。

建议项

  1. 补充背景和问题描述。 尽管更改简单,但为了 PR 记录的完整性,建议在 Background And Problem 中简要说明现有 30 分钟超时是否已导致实际失败,以及这些失败发生的频率或具体场景。
  2. 补充验证命令与结果。 对于此类 CI 变更,直接的本地验证可能有限。建议说明验证方式,例如“观察 daily_analysis workflow 后续运行情况,确认不再因超时失败”,或者提供模拟运行结果(如果可能)。
  3. 补充修改范围。 尽管修改文件列表已提供,建议在 Scope Of Change 中明确指出修改范围,例如“修改了 .github/workflows/daily_analysis.yml 文件中的 analyze job 超时配置”。
  4. 更新 PR 类型描述。 考虑将 PR Type 选项中的 [ ] fix 勾选,使其变为 [x] fix

💡 提示: 请确保代码已通过本地测试,并遵循项目代码规范。

@ZhuLinsen ZhuLinsen left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

评审结论

  • 必要性:通过,提升每日分析工作流超时时间与“避免分析任务超时失败”的目标一致。
  • 是否有对应 issue:无,PR 描述未检测到 Fixes/Refs #xxx,也未填写无 Issue 时的动机与验收标准。
  • PR 类型:fix,修改 .github/workflows/daily_analysis.ymltimeout-minutes 以修复/规避定时分析任务超时失败。
  • description 完整性:不完整,缺少背景触发场景、变更范围、Issue 或无 Issue 说明、实际验证命令与结果、兼容性/风险评估、回滚方案,Checklist 也未勾选。
  • 是否可直接合入:不可,当前 PR 存在冲突,且缺少 AGENTS.md 要求的回滚方案与必要 PR 描述信息。

主要问题

  1. [Process blocker] 当前 merge 状态为存在冲突,当前不能直接合并。需要先解决冲突后再进入合入判断。
  2. [Process blocker] PR 描述未填写回滚方案。AGENTS.md 将“缺少回滚方案”列为合入阻断条件;本次只改 workflow timeout,最小回滚可以是 revert 本 PR 或将 .github/workflows/daily_analysis.ymltimeout-minutes 恢复为原值。
  3. [Process blocker] PR 描述关键字段基本为空:未说明对应 issue 或无 issue 的验收标准,未列出实际验证命令与结果,未说明影响的是“每日股票分析”GitHub Actions 工作流。虽然当前 CI 为 success,但 PR 描述仍缺少本仓库规范要求的变更范围、验证和风险说明。
  4. [Nice to have] 结构化检测命中了 LLM/API 相关风险提示,但本地 merge-base diff 仅显示 timeout-minutes: 30 -> 300,没有新增或修改模型名、provider、Base URL、运行时配置迁移或清理逻辑;因此不作为外部 API 兼容性阻断。建议在风险说明中明确本 PR 不涉及模型/API 配置语义变更,避免误判。

🤖 此回复由 OpenReview Bot 自动生成,仅供参考。如有疑问请 @维护者。

@ZhuLinsen ZhuLinsen closed this May 8, 2026
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.

2 participants