Skip to content
This repository was archived by the owner on Apr 17, 2026. It is now read-only.

Fix bug for peft compatibility#20

Open
YuliangXiu wants to merge 1 commit into
google:mainfrom
YuliangXiu:main
Open

Fix bug for peft compatibility#20
YuliangXiu wants to merge 1 commit into
google:mainfrom
YuliangXiu:main

Conversation

@YuliangXiu

@YuliangXiu YuliangXiu commented Jan 14, 2024

Copy link
Copy Markdown

With this quick fix, break-a-scene could be successfully trained with peft approaches, such as LoRA and BOFT, without following errors:

RuntimeError: one of the variables needed for gradient computation has been modified by an inplace operation: [torch.cuda.FloatTensor [10, 4096, 77]], which is output 0 of SoftmaxBackward0, is at version 1; expected version 0 instead. Hint: the backtrace further above shows the operation that failed to compute its gradient. The variable in question was changed in there or anywhere later. Good luck!

@google-cla

google-cla Bot commented Jan 14, 2024

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant