Implementing Character Flipping Data Augmentation#536
Implementing Character Flipping Data Augmentation#536hunterhector merged 9 commits intoasyml:masterfrom
Conversation
hunterhector
left a comment
There was a problem hiding this comment.
Overall seems good. Probably try to improve the documentation, make it easy for users to understand the op easily.
Codecov Report
@@ Coverage Diff @@
## master #536 +/- ##
==========================================
+ Coverage 79.58% 79.62% +0.03%
==========================================
Files 222 224 +2
Lines 15917 15968 +51
==========================================
+ Hits 12668 12714 +46
- Misses 3249 3254 +5
Continue to review full report at Codecov.
|
hunterhector
left a comment
There was a problem hiding this comment.
overall LGTM, can you also add the work to documentation like here: https://github.com/asyml/forte/pull/532/files#diff-bc9096de1a1a4569bb3196a29b0a79ae09a65c60bbda10b53c4fc8da775375c8R77
hunterhector
left a comment
There was a problem hiding this comment.
should be good if all tests pass
Description of changes
Added Character Flipping Data Augmentation to randomly flip certain characters in the text to other similar-looking characters.
Possible influences of this PR.
Additional Data Augmentation Task is added to Forte by this PR
Test Conducted
Asserts if the generated and expected texts are equal.