Fixed gradient checkpoint bug for TimeSeriesTransformer

Open pmollerus23 opened this issue 2 years ago • 1 comments

What does this PR do?

Moved gradient checkpointing clause to above the decoder layer implementation. This should fix the bug this issue addresses.

Fixes # (21733)

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you read the contributor guideline, Pull Request section?
[x] Was this discussed/approved via a Github issue or the forum? Please add a link to it if that's the case. GitHub Issue
[ ] 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?

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.

@younesbelkada

Mar 20 '23 14:03 pmollerus23

The documentation is not available anymore as the PR was closed or merged.

Mar 20 '23 15:03 HuggingFaceDocBuilderDev

Strange, I will see if I can install Python for my environment then. Thank you for all your help, and thanks for running make for me!

Mar 23 '23 13:03 pmollerus23