Skip to content

Remove pre-trained model weights from the repository#88

Merged
ivanbasov merged 1 commit into
mainfrom
ci/models-from-draft-release
Jul 9, 2026
Merged

Remove pre-trained model weights from the repository#88
ivanbasov merged 1 commit into
mainfrom
ci/models-from-draft-release

Conversation

@ivanbasov

@ivanbasov ivanbasov commented Jul 9, 2026

Copy link
Copy Markdown
Member

The pre-trained model weights are distributed via Hugging Face:

This PR removes the weight files from the repository:

  • Removed models/*.pt and all Git LFS machinery (.gitattributes rule, lfs: true checkouts, git lfs install, git-lfs packages); /models/ is now gitignored and remains the conventional local directory for downloaded weights.
  • The CI jobs whose tests load the weights (unit-tests, unit-tests-coverage, gpu-tests, gpu-coverage, ler-regression) download them at run time via the new .github/actions/fetch-models action.
  • README now points at the Hugging Face model cards with download instructions; test error messages updated; removed the unimplemented PREDECODER_MODEL_URL mention.

🤖 Generated with Claude Code

@ivanbasov ivanbasov force-pushed the ci/models-from-draft-release branch from cc729b4 to 99142dc Compare July 9, 2026 21:45
@ivanbasov ivanbasov requested a review from bmhowe23 July 9, 2026 21:47
The pre-trained model weights are distributed via Hugging Face:
  - https://huggingface.co/nvidia/Ising-Decoder-SurfaceCode-1-Fast
  - https://huggingface.co/nvidia/Ising-Decoder-SurfaceCode-1-Accurate

Changes:
  - Delete models/*.pt and the .gitattributes Git LFS rule; ignore /models/.
  - Add .github/actions/fetch-models, a composite action that downloads
    the model assets of the ci-models-v1 release into models/.
  - Wire the action into the five jobs whose tests load the weights
    (unit-tests, unit-tests-coverage, gpu-tests, gpu-coverage,
    ler-regression); those jobs grant the token the push access needed
    to resolve the release. All workflow triggers are same-repo
    (push/merge_group/schedule/dispatch), never fork pull_request.
  - Drop lfs: true, git lfs install and git-lfs apt packages everywhere.
  - Point README and test error messages at Hugging Face; remove the
    unimplemented PREDECODER_MODEL_URL mention.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@ivanbasov ivanbasov force-pushed the ci/models-from-draft-release branch from 99142dc to 4f54db6 Compare July 9, 2026 22:13
@ivanbasov ivanbasov changed the title Remove model weights from repo; CI fetches them from a permanent draft release Remove pre-trained model weights from the repository Jul 9, 2026
@ivanbasov ivanbasov merged commit 147c5fb into main Jul 9, 2026
17 checks passed
@ivanbasov ivanbasov deleted the ci/models-from-draft-release branch July 9, 2026 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants