-
Notifications
You must be signed in to change notification settings - Fork 324
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
lightseq cross entropy kernel is slower than original torch kernel. #204
Comments
Have you tried our code for the speed test? |
nope. I just used |
time.time() is testing on the CPU side, because GPU executes
asynchronously, maybe you should ensure that the GPU task is finished. For
example, retrieve the result at the end.
…On Wed, Oct 13, 2021 at 7:53 PM Kevin Ko ***@***.***> wrote:
nope. I just used time.time() module.
I was wondering if my experiment was wrong, so I wanted to check any
benchmark result.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#204 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AELIZAPVTTLQZLNDUSLEOQDUGVXNNANCNFSM5F4JX2UQ>
.
|
Thank you for reply. Thanks. |
For example, printing the result to the console will ensure that the GPU
task is finished.
…On Wed, Oct 13, 2021 at 10:43 PM Kevin Ko ***@***.***> wrote:
Thank you for reply.
What does "retrieve the result at the end." means?
Thanks.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#204 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AELIZAODXKYT5ERUVXL73JTUGWLIXANCNFSM5F4JX2UQ>
.
|
So is there a simple way to benchmark the time spent on the GPU? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Are there any benchmark results for this kernel?
As a result of my experiments, it seems to be slower than the original torch kernel.
The text was updated successfully, but these errors were encountered: