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 · 2 comments
Open

Overload of addcmul_ is deprecated: #49

sooheon opened this issue Jul 18, 2020 · 2 comments

Comments

@sooheon
Copy link

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

fixed here: #51

@brando90
Copy link

brando90 commented Aug 4, 2021

I also get it:

/home/miranda9/RAdam/radam/radam.py:58: 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) (Triggered internally at  /opt/conda/conda-bld/pytorch_1623448238472/work/torch/csrc/utils/python_arg_parser.cpp:1025.)
  exp_avg_sq.mul_(beta2).addcmul_(1 - beta2, grad, grad)

any progress on the PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants