You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by "jwfromm (via GitHub)" <gi...@apache.org> on 2023/09/22 22:22:24 UTC
[GitHub] [tvm] jwfromm opened a new pull request, #15811: [Unity][Frontend][NN] Switch to always using fp32 for timestep embedding calculation
jwfromm opened a new pull request, #15811:
URL: https://github.com/apache/tvm/pull/15811
It turns out that using datatypes besides `float32` when calculating timestep embeddings leads to substantial errors. The diffusers library just hard casts to float32 to avoid this and we should too.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] junrushao merged pull request #15811: [Unity][Frontend][NN] Switch to always using fp32 for timestep embedding calculation
Posted by "junrushao (via GitHub)" <gi...@apache.org>.
junrushao merged PR #15811:
URL: https://github.com/apache/tvm/pull/15811
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org