Skip to content

Cleanup: 删除未接线的 BaseCollector 与 FastTD3Runner 孤儿模块 #993

Description

@TATP-233

Parent: #983
Integration base: dev/issue-983-code-cleanup(不得向 main 提 PR)
Work type: infra · Area: algos · Scope: Small

Owner summary

这次只删除两个仓库内零引用、未导出、未注册的孤儿模块:BaseCollectorFastTD3Runner。生产 collector/TD3 仍使用现有 worker 与 Hydra owner 构造,不改变训练路径、算法默认值或公共 registry;永久维护项是一条防止同名孤儿文件回流的 repo-hygiene 断言。

一句话问题

两个模块看似提供正式抽象/runner,但没有任何 import、子类、registry 或配置入口,已经与真实生产 owner 平行腐化。

最小交付结果

删除两个确定不可达的模块,并用静态回归证明生产引用与公开 export 不受影响。

In scope

  • 删除 algos/torch/common/base_collector.py
  • 删除 algos/torch/fast_td3/runner.py
  • 复核 fast_td3/__init__.py、registry、Hydra config、tests/benchmark/docs 无引用。
  • 在现有 repo-hygiene 测试中登记这些已删除孤儿路径。

Non-goals

  • 不删除或改造 FastTD3 learner、off-policy worker、DoubleBuffer runner。
  • 不处理 FastSAC/FlashSAC runner 或其 public export。
  • 不删除 ANE 预留模块,也不改算法默认参数。

Owner 与预计改动

删除 2 个 production 文件,修改 1 个测试文件;净删除约 150 行,1 PR。

Acceptance criteria

  • 全仓无 BaseCollectorFastTD3Runner 或对应模块 import。
  • FastTD3 learner 与 production off-policy TD3 配置测试仍通过。
  • repo-hygiene 定向测试和 make test-all 通过。

Stop conditions

若发现外部文档/插件承诺、正式 re-export 或需要修改 production TD3 构造,暂停并回到 #983

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