Raising exceptions instead of using assertions for few models #14219

ghost · 2021-10-31T10:02:03Z

What does this PR do?

The PR addresses issue Raise exceptions instead of using assertions for control flow #12789 in a few transformer scripts.
Didn't try to tinker a lot of the scripts for the same since it's my 1st PR

Fixes # (issue)

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?

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.

sgugger

Looks great! Thanks a lot for fixing those!

…gface#14219) * raising exceptions instead of using assertions for few models * fixed formatting issues * fixing copy inconsistencies

raising exceptions instead of using assertions for few models

7367795

ghost marked this pull request as draft October 31, 2021 10:24

pdqn added 2 commits October 31, 2021 17:34

fixed formatting issues

57f8a76

fixing copy inconsistencies

6f41ad0

ghost marked this pull request as ready for review October 31, 2021 14:07

sgugger approved these changes Oct 31, 2021

View reviewed changes

LysandreJik merged commit 33fb983 into huggingface:master Nov 1, 2021