box-embeddings icon indicating copy to clipboard operation
box-embeddings copied to clipboard

Published 20 hours ago • verified publisher icon iesl

make loss.backward() work

Open allenhung1025 opened this issue 3 years ago • 1 comments

I was trying to use gumbel_intersection in my training code. However, when calling the loss.backward(), there would be an error message. Imgur which shows that there should not be an in-place operation with tunable parameter. After I made some changes in the intersection function, the code worked out.

allenhung1025 avatar Apr 20 '22 02:04 allenhung1025

Thanks for creating the PR. I am not entirely sure if this issue is coming up due to the box-embeddings library—we have been able to backprop successfully while using gumbel intersection. Would you be able to create a small test case that isolates this issue this issue?

dhruvdcoder avatar Apr 23 '22 18:04 dhruvdcoder