Skip to content

Conversation

@roll-away
Copy link
Contributor

Description

@paddle-bot
Copy link

paddle-bot bot commented Jan 27, 2026

Thanks for your contribution!

@JewelRoam
Copy link
Collaborator

JewelRoam commented Jan 27, 2026

hi 奕玮,

我来更新明确下本PR需要完成的事情:

  • 从eval_backend_perf中拆分出utils.timing即可,目前看来只包括CUDA、cpu计时相关的函数,大致是现有的measure_performance,至于get_hardward_name、get_compiler_version不应该是utils.timing中的内容。(我知道不一起放在utils.timing中可能确实会出现循环引用之类的问题,但仍然需要根据SRP原则拆分出去。)
  • 由于我们后续将把eval_backend_perf改造成只负责单次运行的TorchBackend,无关性能对比,所以不管你目前往里边放了什么,应该暂时是不需要拆出utils.comparison的。

辛苦调整

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.

2 participants