Skip to content
This repository was archived by the owner on Sep 10, 2025. It is now read-only.

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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants