智能文件打包和当前源码实现不一样么?貌似还是每个文件一个评审单元 #710
|
我前几天在公众号上看到相关文章,其中提到一点: |
Replies: 2 comments 1 reply
|
我们之前有个分支中有这部分的原因,由于相比 main 分支过时了太久,前几天被我删除了。没有上线的原因是外部版本尝试了使用用户配置 LLM 来进行分组,结论是分组分得不太稳定,其实最好是通过一个 embedding 模型进行分组,但是这个行为会让二进制产物的大小激增,所以这一块儿暂时性的搁置了,我认为在我们处理完社区高优先级的 PR 后会重新开展这项工作,为了避免现在 main 分支发展过快时的实验成本过高。如果您对源码比较感兴趣,我可以就前面说的依赖大模型进行分组的版本再推一份代码上去。 @Susanping |
|
@Susanping 来了来了,这个功能的实现刚推上去了:#808 目前的方案是用 LLM 来做语义分组(通过一个专门的 grouping_task prompt),会把关联文件(比如 .go + _test.go、头文件 + 实现文件、同模块的多个变更文件)归到同一个评审单元里,每个组作为一个独立的 subagent 并发审查。 还在跑评测调优中,欢迎看看代码提提意见~ |
我们之前有个分支中有这部分的原因,由于相比 main 分支过时了太久,前几天被我删除了。没有上线的原因是外部版本尝试了使用用户配置 LLM 来进行分组,结论是分组分得不太稳定,其实最好是通过一个 embedding 模型进行分组,但是这个行为会让二进制产物的大小激增,所以这一块儿暂时性的搁置了,我认为在我们处理完社区高优先级的 PR 后会重新开展这项工作,为了避免现在 main 分支发展过快时的实验成本过高。如果您对源码比较感兴趣,我可以就前面说的依赖大模型进行分组的版本再推一份代码上去。 @Susanping