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

add lazy ops docs #504

Merged
merged 2 commits into from
Dec 30, 2020
Merged

add lazy ops docs #504

merged 2 commits into from
Dec 30, 2020

Conversation

congee524
Copy link
Contributor

fix #418

@@ -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.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any numbers for greatly?

Copy link
Contributor Author

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.

Copy link
Contributor Author

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
Copy link

codecov bot commented Dec 30, 2020

Codecov Report

Merging #504 (952c0a7) into master (5081643) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           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           
Flag Coverage Δ
unittests 84.72% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5081643...952c0a7. Read the comment docs.

@innerlee innerlee merged commit e40df88 into open-mmlab:master Dec 30, 2020
@congee524 congee524 deleted the add_lazy_ops_docs branch January 25, 2021 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Need docs about lazy ops
2 participants