Skip to content

fix torchtune in Genie#480

Merged
SLR722 merged 1 commit into
pytorch:mainfrom
SLR722:export-D59404942
Jul 5, 2024
Merged

fix torchtune in Genie#480
SLR722 merged 1 commit into
pytorch:mainfrom
SLR722:export-D59404942

Conversation

@SLR722

@SLR722 SLR722 commented Jul 5, 2024

Copy link
Copy Markdown
Contributor

Summary:
fixes torchtune in Genie (all torchtune in Genie jobs are blocked by importlib.metadata.PackageNotFoundError: No package metadata was found for torch)

check is_fbcode before checking torch version. since python package version only works in OSS

Differential Revision: D59404942

@pytorch-bot

pytorch-bot Bot commented Jul 5, 2024

Copy link
Copy Markdown

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/ao/480

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

✅ No Failures

As of commit 719cabf with merge base a35a1cd (image):
💚 Looks good so far! There are no failures yet. 💚

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

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 5, 2024
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D59404942

Summary:
Pull Request resolved: pytorch#480

fixes torchtune in Genie (all torchtune in Genie jobs are blocked by importlib.metadata.PackageNotFoundError: No package metadata was found for torch)

check is_fbcode before checking torch version. since python package version only works in OSS

Differential Revision: D59404942
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D59404942

@SLR722 SLR722 force-pushed the export-D59404942 branch from 32ef5ce to 719cabf Compare July 5, 2024 19:30
@msaroufim msaroufim self-requested a review July 5, 2024 20:36
@SLR722 SLR722 merged commit ec73788 into pytorch:main Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants