Skip to content

ci: migrate test models to pruna-test#385

Merged
gsprochette merged 4 commits intomainfrom
ci/migrate-test-models
Oct 6, 2025
Merged

ci: migrate test models to pruna-test#385
gsprochette merged 4 commits intomainfrom
ci/migrate-test-models

Conversation

@gsprochette
Copy link
Copy Markdown
Collaborator

@gsprochette gsprochette commented Oct 3, 2025

Description

This PR enacts the transfer of all test models previously stored in PrunaAI to the pruna-test organisation.

  • PrunaAI/model-name in the tests were renamed to pruna-test/model-name
  • same for loulou2/model-name

This PR also changes the condition for running the push-to-hub tests on pruna side in the nightlies so that it actually runs.

Related Issue

This fixes the following errors when testing the push-to-hub functionalities: huggingface_hub._upload_large_folder:_upload_large_folder.py:395 Failed to commit

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Re-ran the nightly tests.

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Additional Notes


Note

Updates test model repo IDs to pruna-test and removes HF token skip to run push-to-hub tests.

  • Tests:
    • Model refs: Update Hugging Face repo IDs from PrunaAI/* and loulou2/* to pruna-test/* in tests/engine/test_load.py and tests/fixtures.py.
    • Push-to-hub: Remove @pytest.mark.skipif("HF_TOKEN" not in os.environ, ...) to allow running hub upload tests in tests/engine/test_save.py.

Written by Cursor Bugbot for commit 836d748. This will update automatically on new commits. Configure here.

Copy link
Copy Markdown
Member

@davidberenstein1957 davidberenstein1957 left a comment

Choose a reason for hiding this comment

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

LFTM, however, is there a reason we are not transferring other models we are using to our test organisation?

@johnrachwan123
Copy link
Copy Markdown
Member

bugbot run

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no bugs!


@gsprochette gsprochette merged commit 2c52649 into main Oct 6, 2025
7 checks passed
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.

3 participants