Skip to content

Conversation

co63oc
Copy link
Contributor

@co63oc co63oc commented Sep 7, 2025

增加单测
test_draft_model_update
speculate_update_v2/speculate_update_v3 都为 speculate_update

Copy link

paddle-bot bot commented Sep 7, 2025

Thanks for your contribution!

@paddle-bot paddle-bot bot added the contributor External developers label Sep 7, 2025
@co63oc co63oc changed the title 【Hackathon 9th No.65】add test_draft_model_update 【Hackathon 9th No.61、65】add test_draft_model_update Sep 9, 2025
@co63oc
Copy link
Contributor Author

co63oc commented Sep 10, 2025

/re-run all-failed

@co63oc
Copy link
Contributor Author

co63oc commented Sep 10, 2025

@freeliuzc CI已完成需要review

@freeliuzc
Copy link
Collaborator

请问下这个测试为什么起名叫 V3 呢?

@co63oc
Copy link
Contributor Author

co63oc commented Sep 10, 2025

speculate_update_v3

NO.61 自定义算子 speculate_update_v3 单测补充
任务名称是这个,XPU算子名称也是speculate_update_v3,custom_ops/xpu_ops/src/ops/mtp_ops/speculate_update_v3.cc
如果需要改名也可以修改,主要是想和XPU一致
@freeliuzc

@freeliuzc
Copy link
Collaborator

speculate_update_v3

NO.61 自定义算子 speculate_update_v3 单测补充 任务名称是这个,XPU算子名称也是speculate_update_v3,custom_ops/xpu_ops/src/ops/mtp_ops/speculate_update_v3.cc 如果需要改名也可以修改,主要是想和XPU一致 @freeliuzc

好的,这里应该是没有及时更新,辛苦把单测名字去掉 v3 哈,目前只有一个稳定版本

@co63oc
Copy link
Contributor Author

co63oc commented Sep 12, 2025

好的,这里应该是没有及时更新,辛苦把单测名字去掉 v3 哈,目前只有一个稳定版本

已修改去掉v3

}


class TestSpeculateUpdateV3(unittest.TestCase):
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里名字辛苦也去掉下,别的没问题了

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

已修改

@luotao1 luotao1 merged commit ef4a1aa into PaddlePaddle:develop Sep 15, 2025
15 of 17 checks passed
@co63oc co63oc changed the title 【Hackathon 9th No.61、65】add test_draft_model_update 【Hackathon 9th No.61、65、41】add test_draft_model_update Sep 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants