-
Notifications
You must be signed in to change notification settings - Fork 22.6k
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
[DataPipe] Count number of successful yields for IterDataPipe #79657
Conversation
[ghstack-poisoned]
🔗 Helpful links
✅ No Failures (0 Pending)As of commit 55bdafc (more details on the Dr. CI page): Expand to see more💚 💚 Looks good so far! There are no failures yet. 💚 💚 This comment was automatically generated by Dr. CI (expand for details).Please report bugs/suggestions to the (internal) Dr. CI Users group. |
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
…ipe" [ghstack-poisoned]
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.
I have two comments below about bumping and reducing the counter.
…ipe" This PR adds an attribute and logic to count the number of successful yields from `IterDataPipe`. This information can be useful to fast-forward a DataPipe (or the entire graph) back to a certain state. [ghstack-poisoned]
…ipe" This PR adds an attribute and logic to count the number of successful yields from `IterDataPipe`. This information can be useful to fast-forward a DataPipe (or the entire graph) back to a certain state. [ghstack-poisoned]
…ipe" This PR adds an attribute and logic to count the number of successful yields from `IterDataPipe`. This information can be useful to fast-forward a DataPipe (or the entire graph) back to a certain state. [ghstack-poisoned]
…ipe" This PR adds an attribute and logic to count the number of successful yields from `IterDataPipe`. This information can be useful to fast-forward a DataPipe (or the entire graph) back to a certain state. [ghstack-poisoned]
@pytorchbot merge |
@pytorchbot successfully started a merge job. Check the current status here |
@NivekT your PR has been successfully merged. |
#79657) Summary: This PR adds an attribute and logic to count the number of successful yields from `IterDataPipe`. This information can be useful to fast-forward a DataPipe (or the entire graph) back to a certain state. Pull Request resolved: #79657 Approved by: https://github.com/VitalyFedyunin Test Plan: contbuild & OSS CI, see https://hud.pytorch.org/commit/pytorch/pytorch/b8e50f512f735fb79b8b525cdb552e095a3873cb Reviewed By: b0noI Differential Revision: D37509831 Pulled By: NivekT fbshipit-source-id: ad9f1bbcb43fc04d1d45135ec45a806dff8ff018
Stack from ghstack:
This PR adds an attribute and logic to count the number of successful yields from
IterDataPipe
. This information can be useful to fast-forward a DataPipe (or the entire graph) back to a certain state.