text-generation-inference icon indicating copy to clipboard operation
text-generation-inference copied to clipboard

Published 20 hours ago • verified publisher icon huggingface

Attempting to fix torch leak.

Open Narsil opened this issue 11 months ago • 1 comments

What does this PR do?

Fixes # (issue)

Before submitting

  • [ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • [ ] Did you read the contributor guideline, Pull Request section?
  • [ ] Was this discussed/approved via a Github issue or the forum? Please add a link to it if that's the case.
  • [ ] Did you make sure to update the documentation with your changes? Here are the documentation guidelines, and here are tips on formatting docstrings.
  • [ ] Did you write any new necessary tests?

Who can review?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag members/contributors who may be interested in your PR.

Narsil avatar Aug 12 '23 07:08 Narsil

@Narsil So it seems like theres multiple causal_lm files so you would need it to all of them not just the one I mentioned. So any file that ends with causal_lm needs this change as it includes the if stopped: function. Let me know if i need to be more clear

Ichigo3766 avatar Aug 19 '23 04:08 Ichigo3766