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

p.add_(..., inplace=True) error #1201

Open
markussteindl opened this issue Oct 23, 2023 · 1 comment
Open

p.add_(..., inplace=True) error #1201

markussteindl opened this issue Oct 23, 2023 · 1 comment

Comments

@markussteindl
Copy link

commit: c7392f2
line: automl/lion/lion_pytorch.py:81

When calling Lion.step(), I get this error message:

TypeError: add_() received an invalid combination of arguments - got unrecognized keyword arguments: inplace

I think p.add_() is already the inplace version, while p.add() has an optional parameter inplace.
Or am I using the wrong pytorch version? I have 2.0.1+cu118.

@fbotp
Copy link

fbotp commented Oct 25, 2023

@Stonatus I found some info about Tensor.add_ method in pytorch document, and it means add_ is an inplace method, so I think it might be okey to just remove inplace argument in line: automl/lion/lion_pytorch.py:81, and it seems work fine

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

2 participants