-
Notifications
You must be signed in to change notification settings - Fork 60
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
Single Ontology Data Augmentation #717
Conversation
…Split, DistReplace
Codecov Report
@@ Coverage Diff @@
## master #717 +/- ##
==========================================
- Coverage 81.36% 81.34% -0.03%
==========================================
Files 242 243 +1
Lines 18291 18338 +47
==========================================
+ Hits 14883 14917 +34
- Misses 3408 3421 +13
Continue to review full report at Codecov.
|
forte/processors/data_augment/algorithms/back_translation_op.py
Outdated
Show resolved
Hide resolved
forte/processors/data_augment/algorithms/back_translation_op.py
Outdated
Show resolved
Hide resolved
forte/processors/data_augment/algorithms/dictionary_replacement_op.py
Outdated
Show resolved
Hide resolved
forte/processors/data_augment/algorithms/back_translation_op.py
Outdated
Show resolved
Hide resolved
forte/processors/data_augment/algorithms/back_translation_op.py
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can provide better documentation in a way to help developers:
https://asyml-forte--717.org.readthedocs.build/en/717/code/data_aug.html
Check the generated documentation to see if you will find it useful from a developer's angle.
Description of changes
The augmentation framework introduced in #621 and #685 introduce a way to perform augmentation on multiple ontologies. This PR works on creating a method to simplify the operations of the
BaseDataAugmentationOp
by allowing augmentation of only one ontology at a time.Possible influences of this PR.
This PR will have two uses:
Test Conducted
All implementations and tests conducted for existing DA methods in Forte were updated to work with the new Op structure and the format required by
SingleTokenAugmentationOp