Skip to content

Conversation

@Alptimus
Copy link

What does this PR do?

Fixes # "NameError: name 'ftfy' is not defined" in Wan pipeline by directly importing ftfy.

Before submitting

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.

@asomoza and @sayakpaul

@a-r-r-o-w
Copy link
Contributor

How does directly importing it fix the problem of a missing package?

The NameError is expected if you don't have ftfy installed. If you do, the conditional import will work as expected and import it

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants