Skip to content

Updating TorchData DataPipe API usages #1663

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

Merged
merged 1 commit into from
Mar 29, 2022
Merged

Conversation

NivekT
Copy link
Contributor

@NivekT NivekT commented Mar 25, 2022

For better consistency among its API names, TorchData deprecated/renamed a few DataPipes' methods in its beta release. This PR update torchtext's usages to align with the latest TorchData API.

Basically, we are updating .read_from_ARCHIVE() to .load_from_ARCHIVE().

There should be no functional changes for torchtext at all. For details about the renaming and deprecation, see the following issue.


The lint failure in CI is pre-existing.
The issue with "ci/circleci: unittests" failing during builds is fixed by this PR:

Copy link
Contributor

@Nayef211 Nayef211 left a comment

Choose a reason for hiding this comment

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

Changes LGTM. I just kicked off another run for the failing workflows in CI. Let's wait for those to complete before merging this in :)

@NivekT
Copy link
Contributor Author

NivekT commented Mar 28, 2022

Changes LGTM. I just kicked off another run for the failing workflows in CI. Let's wait for those to complete before merging this in :)

Do I need to rebase this branch? Otherwise, I don't think the CI will pick up the change to install torchdata nightly instead?

@Nayef211
Copy link
Contributor

Changes LGTM. I just kicked off another run for the failing workflows in CI. Let's wait for those to complete before merging this in :)

Do I need to rebase this branch? Otherwise, I don't think the CI will pick up the change to install torchdata nightly instead?

Yup I think you need to rebase on the latest master

@NivekT NivekT force-pushed the UpdateDeprecation branch from 2b195ad to 4667670 Compare March 28, 2022 20:18
@Nayef211
Copy link
Contributor

I don't think the CI failures are related to your PR. Feel free to merge this 😄

@NivekT NivekT merged commit d4656e2 into pytorch:main Mar 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants