-
Notifications
You must be signed in to change notification settings - Fork 299
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix SamplingDecodeTest and SamplingDecodeTest2 unittest failure #1874
Conversation
before the fix
after
|
lmdeploy/tests/csrc/unittests/test_sampling_layer.cu Lines 1035 to 1041 in 12e8609
Different with fastertransformer, we initialize |
ok |
Hi @irexyc Could you please review the latest code? Thanks. |
Motivation
fix SamplingDecodeTest and SamplingDecodeTest2 unittest failure
Hi @lzhangzz @irexyc @lvhan028 May you help review this pr? Thanks.
Modification
as titled
Checklist