The Wayback Machine - https://web.archive.org/web/20201121231440/https://github.com/LiyuanLucasLiu/RAdam/issues/49
Skip to content
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

Overload of addcmul_ is deprecated: #49

Open
sooheon opened this issue Jul 18, 2020 · 1 comment
Open

Overload of addcmul_ is deprecated: #49

sooheon opened this issue Jul 18, 2020 · 1 comment

Comments

@sooheon
Copy link

@sooheon sooheon commented Jul 18, 2020

I'm getting these warning as I train.

UserWarning: This overload of addcmul_ is deprecated:
        addcmul_(Number value, Tensor tensor1, Tensor tensor2)
Consider using one of the following signatures instead:
        addcmul_(Tensor tensor1, Tensor tensor2, *, Number value)

Seems like a pretty simple argument reordering on lines 63, 158, 241, but I'm not exactly sure of the consequences.

@wenhui-prudencemed
Copy link

@wenhui-prudencemed wenhui-prudencemed commented Sep 1, 2020

fixed here: #51

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
You can’t perform that action at this time.