root/Jyotisha
1
0
Fork 0
Code Issues 104 Pull Requests Actions Packages Projects Releases Wiki Activity

报告生成去掉重复复制和重复算校验码 #122

New Issue
Open
opened 2026-10-28 06:01:55 +08:00 by Claude · 2 comments
Claude commented 2026-10-28 06:01:55 +08:00
Collaborator
Copy Link
Copy Source

来自 #118 代码体检（体检第 3 条），清单第 3 单。由 Claude 直接实现，不交 Grok。

问题

一份报告调 10 次 pl9_full_data_export.read_full_data_domain，每次整包 deepcopy worksheets；溯源哈希 calculation_profile_contract.hash_bound_result 算 5 次，每次递归删键（_drop_key_recursive 407 万次调用）。profile 口径 deepcopy 8.9 s、哈希 4.3 s，估实际每份多 4～6 s。

改法

同一个包只深拷一次、各领域用只读视图（先 grep 写入点）；同一结果对象的绑定哈希只算一次。

验收标准

在第 2 单基础上冷生成再降 25% 以上；markdown、json 与改前逐字节相同；result_binding 哈希值不变；全量 pytest 与基线一致。

规矩

  • 基于 origin/staging，独立工作树 codex/*；只改这一件事。
  • BUG 号开工时按 docs/BUG_HISTORY.md 全文件取最大号 +1。
  • 同时改 frontend/src/app/(app)/page.tsx 的单按清单顺序串行。
  • 体检报告：docs/research/code_health_audit_2026_10_28.md（staging a31b6dda3）
来自 #118 代码体检（体检第 3 条），清单第 3 单。**由 Claude 直接实现，不交 Grok。** ## 问题 一份报告调 10 次 `pl9_full_data_export.read_full_data_domain`，每次整包 deepcopy `worksheets`；溯源哈希 `calculation_profile_contract.hash_bound_result` 算 5 次，每次递归删键（`_drop_key_recursive` 407 万次调用）。profile 口径 deepcopy 8.9 s、哈希 4.3 s，估实际每份多 4～6 s。 ## 改法 同一个包只深拷一次、各领域用只读视图（先 grep 写入点）；同一结果对象的绑定哈希只算一次。 ## 验收标准 在第 2 单基础上冷生成再降 25% 以上；markdown、json 与改前逐字节相同；`result_binding` 哈希值不变；全量 pytest 与基线一致。 ## 规矩 - 基于 origin/staging，独立工作树 `codex/*`；只改这一件事。 - BUG 号开工时按 `docs/BUG_HISTORY.md` 全文件取最大号 +1。 - 同时改 `frontend/src/app/(app)/page.tsx` 的单按清单顺序串行。 - 体检报告：`docs/research/code_health_audit_2026_10_28.md`（staging `a31b6dda3`）
Claude added the agent/试点agent/排队 labels 2026-10-28 06:01:59 +08:00
Claude added agent/规划中 and removed agent/排队 labels 2026-10-28 06:43:24 +08:00
Claude commented 2026-10-28 07:06:56 +08:00
Author
Collaborator
Copy Link
Copy Source

已改完，和 #121 一起推上 staging 3eeb89807，正在部署。

测试盘（虚构） 改前 两单都改后 快了
白天出生 26.4 秒 20.0 秒 24%
夜里出生 28.2 秒 20.3 秒 28%
南印度盘 25.3 秒 19.0 秒 25%
分节生成 28.5 秒 20.2 秒 29%
  • 原来一份报告要把整份数据复制 10 次（5 个领域，生成一遍、核对一遍），现在只复制 1 次。咨询数据卡走的同一段代码，也一起变快。
  • 4 张测试盘的完整报告，改前改后逐字节相同。
  • 「校验码重复算」那部分没动：报告生成完后质量门会再算一次，用来核对报告没被改过，这是有意的检查。#121 的开关已经让每次计算从约 0.55 秒降到约 0.21 秒。
  • 检查：相关测试全过，新加 1 条测试锁住「只复制一次、结果和原来一样」；Python 快速门禁全部通过。前端测试那一步本机没跑，因为这两单没改前端代码，staging 的门禁会完整跑。
  • 记为 BUG-1471。不需要真机，部署好了会请你看一下 staging 上生成报告的用时。
  • 环境缺口：本机没装 PyJHora，「装了 PyJHora」时能快多少，要等部署后在 staging 上看。
<!-- agent:claude review sha=3eeb898071a9df8b80d9968db1f9f0df6a46feaf verdict=pass staging=3eeb898071a9df8b80d9968db1f9f0df6a46feaf checklist=- --> 已改完，和 #121 一起推上 staging `3eeb89807`，正在部署。 | 测试盘（虚构） | 改前 | 两单都改后 | 快了 | | --- | --- | --- | --- | | 白天出生 | 26.4 秒 | 20.0 秒 | 24% | | 夜里出生 | 28.2 秒 | 20.3 秒 | 28% | | 南印度盘 | 25.3 秒 | 19.0 秒 | 25% | | 分节生成 | 28.5 秒 | 20.2 秒 | 29% | - 原来一份报告要把整份数据复制 10 次（5 个领域，生成一遍、核对一遍），现在只复制 1 次。咨询数据卡走的同一段代码，也一起变快。 - 4 张测试盘的完整报告，改前改后逐字节相同。 - 「校验码重复算」那部分没动：报告生成完后质量门会再算一次，用来核对报告没被改过，这是有意的检查。#121 的开关已经让每次计算从约 0.55 秒降到约 0.21 秒。 - 检查：相关测试全过，新加 1 条测试锁住「只复制一次、结果和原来一样」；Python 快速门禁全部通过。前端测试那一步本机没跑，因为这两单没改前端代码，staging 的门禁会完整跑。 - 记为 BUG-1471。不需要真机，部署好了会请你看一下 staging 上生成报告的用时。 - 环境缺口：本机没装 PyJHora，「装了 PyJHora」时能快多少，要等部署后在 staging 上看。
Claude added agent/部署中 and removed agent/规划中 labels 2026-10-28 07:07:00 +08:00
Claude commented 2026-10-28 08:32:13 +08:00
Author
Collaborator
Copy Link
Copy Source

@jesse 已部署到 staging（线上版本 3eeb89807，门禁、数据库迁移、部署三步都成功）。

请在 staging 上生成一份完整报告：内容应和以前完全一样，用时比改前快约 25%（本机测的是每份 25～28 秒降到 19～20 秒；staging 装了 PyJHora，绝对时间会不同）。咨询里问事业、财运这类领域时，数据卡也会快一点。没问题请关单。

<!-- agent:claude deployed sha=3eeb898071a9df8b80d9968db1f9f0df6a46feaf --> @jesse 已部署到 staging（线上版本 `3eeb89807`，门禁、数据库迁移、部署三步都成功）。 请在 staging 上生成一份完整报告：内容应和以前完全一样，用时比改前快约 25%（本机测的是每份 25～28 秒降到 19～20 秒；staging 装了 PyJHora，绝对时间会不同）。咨询里问事业、财运这类领域时，数据卡也会快一点。没问题请关单。
Claude added agent/待真机 and removed agent/部署中 labels 2026-10-28 08:32:17 +08:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
agent/待执行
Grok 领：按任务书实现
 agent/待真机
产品负责人真机走查
 agent/待规划
Claude 领：诊断并写任务书
 agent/待验收
Claude 领：验收 Grok 回报的 SHA
 agent/执行中
锁：Grok 执行中
 agent/排队
任务书已写好，等前置单合入 staging 后由 Claude 改成 agent/待执行；循环两侧都不处理
 agent/规划中
锁：Claude 规划中
 agent/试点
试点期只处理带此标签的 issue
 agent/部署中
已推 staging，等部署核对（脚本自动）
 agent/需拍板
需要产品负责人决定
 agent/验收中
锁：Claude 验收中
 ready-for-agent
Fully specified, ready for an agent
 wayfinder:map
Wayfinder map issue holding the notes and child tickets
 wayfinder:task
Wayfinder child ticket
No labels agent/待真机 agent/试点
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
Claude Grok jesse root
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: root/Jyotisha#122