Skip to content

Clean up Voxtral Realtime HF weights after export#18289

Merged
mergennachin merged 1 commit intomainfrom
cleanup-voxtral-model-weights
Mar 18, 2026
Merged

Clean up Voxtral Realtime HF weights after export#18289
mergennachin merged 1 commit intomainfrom
cleanup-voxtral-model-weights

Conversation

@mergennachin
Copy link
Contributor

The downloaded model_weights/ directory (~20GB) was not removed
after export, causing it to be uploaded to S3 as part of the CI
artifact. Delete it after copying out the tokenizer since the
weights are fully baked into the .pte at that point.

The downloaded model_weights/ directory (~20GB) was not removed
after export, causing it to be uploaded to S3 as part of the CI
artifact. Delete it after copying out the tokenizer since the
weights are fully baked into the .pte at that point.
Copilot AI review requested due to automatic review settings March 18, 2026 13:44
@pytorch-bot
Copy link

pytorch-bot bot commented Mar 18, 2026

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/18289

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

❌ 1 Awaiting Approval, 2 New Failures, 51 Pending

As of commit 0186941 with merge base 5e265e6 (image):

AWAITING APPROVAL - The following workflow needs approval before CI can run:

NEW FAILURES - The following jobs have failed:

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

@meta-cla meta-cla 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 Mar 18, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes the large HuggingFace-downloaded Voxtral Realtime model_weights/ directory after export so it doesn’t get included in CI artifacts/S3 uploads, while still preserving the tokenizer needed at runtime.

Changes:

  • Copy tekken.json tokenizer into the output artifact directory.
  • Delete the downloaded model_weights/ directory after export for voxtral_realtime.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@github-actions
Copy link

This PR needs a release notes: label

If your change should be included in the release notes (i.e. would users of this library care about this change?), please use a label starting with release notes:. This helps us keep track and include your important work in the next release notes.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "release notes: none"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

@mergennachin mergennachin merged commit d9b394a into main Mar 18, 2026
364 of 374 checks passed
@mergennachin mergennachin deleted the cleanup-voxtral-model-weights branch March 18, 2026 14:24
@mergennachin mergennachin temporarily deployed to upload-benchmark-results March 18, 2026 14:44 — with GitHub Actions Inactive
@manuelcandales
Copy link
Contributor

@pytorchbot cherry-pick --onto release/1.2 -c release

pytorchbot pushed a commit that referenced this pull request Mar 20, 2026
The downloaded model_weights/ directory (~20GB) was not removed
after export, causing it to be uploaded to S3 as part of the CI
artifact. Delete it after copying out the tokenizer since the
weights are fully baked into the .pte at that point.

(cherry picked from commit d9b394a)
@pytorchbot
Copy link
Collaborator

Cherry picking #18289

The cherry pick PR is at #18357 The following tracker issues are updated:

Details for Dev Infra team Raised by workflow job

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/cuda ciflow/metal CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants