-
Notifications
You must be signed in to change notification settings - Fork 212
Add support for Iterable auto dataset + resolve a bug for Preprocess Transforms. #227
Conversation
Co-authored-by: Kaushik B <45285388+kaushikb11@users.noreply.github.com>
Hello @tchaton! Thanks for updating this PR.
Comment last updated at 2021-04-19 14:17:03 UTC |
Codecov Report
@@ Coverage Diff @@
## master #227 +/- ##
==========================================
- Coverage 86.61% 86.49% -0.13%
==========================================
Files 57 57
Lines 2750 2829 +79
==========================================
+ Hits 2382 2447 +65
- Misses 368 382 +14
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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.
LGTM 😃
self, | ||
data: Union[Iterable, Any], | ||
running_stage: RunningStage = None, | ||
use_iterable_auto_dataset: bool = False |
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.
use_iterable_auto_dataset: bool = False | |
use_iterable_auto_dataset: bool = False, |
What does this PR do?
Fixes # (issue)
Before submitting
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃