Skip to content

chore: 移除 #1259 优化期 MBA profiling 脚手架 #1281

Description

@TATP-233

Parent: #1259

Depends on: #1279 / PR #1280(U2 已合入);#1278(U3 已完成 no-go 评估)

Blocks: #1263 最终 7-case 性能验收

一句话问题

#1255 / PR #1258#1256 / PR #1264 为定位和优化 MBA runtime,在正式 step/reset 热路径加入了细粒度 perf_counter、getter recorder、动态 timing dict、info["timing"] 键以及 benchmark 展示逻辑。相关评论已明确这些属于优化期临时脚手架;已激活的性能 child 现已全部完成,最终验收应基于移除 profiling 开销后的 production-like head。

最小交付结果

删除只服务于 #1255/#1256 定位的 MBA 细粒度 profiling 代码与配套测试/benchmark 字段,同时保留生产行为、现有粗粒度 env/backend timing、U1 state-read cache、reset transaction 与全部 manager/backend contract。

In scope

Non-goals

  • 不做新的 manager、task 或 backend 性能优化。
  • 不修改公共 manager lifecycle、SimBackend、NpEnv reset/step contract、policy I/O、配置或 sim2sim contract。
  • 不把 MotionCommand 正常 step、chunk tuner、tracking sensor 裁剪等条件候选塞入本清理 PR。
  • 不运行 Benchmark: 完成必选 MBA runtime 的最终 7-case 性能验收 #1263 的最终 7-case 验收。

Owner、规模与永久维护成本

  • Owner:MBA env/manager instrumentation boundary + collector benchmark 展示层。
  • 预计 10–15 个文件、净删除为主、一个 PR;若需要触及超过 15 个文件或保留/删除某 timing 已成为公共 contract,立即停止并重新拆分/确认。
  • 完成后永久维护成本下降:正式热路径不再维护一次性 profiling 键与闭合测试。

Acceptance criteria

Stop conditions

  • 删除项被发现是公开兼容 contract 或被仓库外 consumer 依赖。
  • 需要改变 manager/backend lifecycle、数值/RNG/reset 行为或新增公共接口。
  • 达到 15 文件、800 行净手写改动、一个 PR 上限,或出现第二个独立交付结果。

创建本 issue 只完成 tracking 与 scope 整合,不自动授权开始开发。

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions