-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
add lazy ops docs #504
add lazy ops docs #504
Conversation
docs/tutorials/4_data_pipeline.md
Outdated
@@ -87,6 +87,39 @@ test_pipeline = [ | |||
] | |||
``` | |||
|
|||
We have supported some lazy operators and encourage users to apply them. | |||
Lazy ops record how the data should be processed, but it will postpone the processing on the raw data until the raw data forward `Fuse` stage. | |||
Specifically, lazy ops avoid frequent reading and modification operation on the raw data, but process the raw data once in the final Fuse stage, thus accelerating data preprocessing greatly. |
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.
Any numbers for greatly
?
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.
The effect on 320p K400 is obvious (but it could also be the memcache's efffect). I'll remove greatly
frist. A comparison result may be posted in the future.
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.
@dreamerlin Do you remember the experiment?
Codecov Report
@@ Coverage Diff @@
## master #504 +/- ##
=======================================
Coverage 84.73% 84.73%
=======================================
Files 120 120
Lines 8407 8407
Branches 1373 1373
=======================================
Hits 7124 7124
Misses 935 935
Partials 348 348
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
fix #418