Skip to content

Sync fbsync branch to main #2210

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 2 commits into from
Oct 12, 2023
Merged

Conversation

abhinavarora
Copy link
Contributor

No description provided.

henryhu6 and others added 2 commits October 12, 2023 13:12
Summary:
The loaded tensor could be on CUDA, whose data cannot to converted to a std::string.

This change moves tensor to CPU first. Then data can be converted.

Reviewed By: mortzur

Differential Revision: D48057806

fbshipit-source-id: ad495bb2e84f9aab5120d5462d4d1c381719d014
…riter.py

Summary: Python3 makes the use of `(object)` in class inheritance unnecessary. Let's modernize our code by eliminating this.

Reviewed By: meyering

Differential Revision: D48958009

fbshipit-source-id: aa42e55a827d83aadf1b10b2de79317c3761ab7a
@pytorch-bot
Copy link

pytorch-bot bot commented Oct 12, 2023

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/text/2210

Note: Links to docs will display an error until the docs builds have been completed.

❌ 9 New Failures, 3 Unrelated Failures

As of commit 24ca2e6 with merge base 45e4b8c (image):

NEW FAILURES - The following jobs have failed:

FLAKY - The following job failed but was likely due to flakiness present on trunk:

BROKEN TRUNK - The following jobs failed but was present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@abhinavarora abhinavarora merged commit 581a3a4 into pytorch:main Oct 12, 2023
@abhinavarora abhinavarora deleted the sync_fbcode branch October 12, 2023 20:23
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.

4 participants