Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Text] Fix return types #191

Closed

Conversation

zpapakipos
Copy link
Contributor

Related Issue

Fixes #189

Summary

  • I have read CONTRIBUTING.md to understand how to contribute to this repository :)

Fixing the text augmentations' return types -- the output type is the same type as the input, which can be either a str or a List[str].

Unit Tests

See the test_python workflow.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Feb 18, 2022
@facebook-github-bot
Copy link
Contributor

@zpapakipos has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@zpapakipos has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@zpapakipos has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

Summary:
## Related Issue
Fixes facebookresearch#189

- [x] I have read CONTRIBUTING.md to understand how to contribute to this repository :)

Fixing the text augmentations' return types -- the output type is the same type as the input, which can be either a `str` or a `List[str]`.

## Unit Tests
See the `test_python` workflow.

Pull Request resolved: facebookresearch#191

Differential Revision: D34340331

Pulled By: zpapakipos

fbshipit-source-id: 87b6b2fe6f8e97bcc534b98326f0079153c49858
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D34340331

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Return type problem in text transformation functions
2 participants