Skip to content

[tests] set overwrite_modular_index=True in tests - #14999

Open
sayakpaul wants to merge 1 commit into
mainfrom
overwrite-modular-index-true
Open

sayakpaul wants to merge 1 commit into
mainfrom
overwrite-modular-index-true

Conversation

@sayakpaul

Copy link
Copy Markdown
Member

Otherwise, we get the following warning

FutureWarning: The default of `overwrite_modular_index` in `ModularPipeline.save_pretrained` changed from 
  `False` to `True`: the saved `modular_model_index.json` now points each saved component at the destination (the 
  save directory, or `repo_id` when pushing to the Hub). Pass `overwrite_modular_index=False` to keep the 
  previously recorded loading specs, or pass `True` explicitly to silence this warning.

Also, add a test to ensure that the warning is raised.

@github-actions github-actions Bot added tests size/S PR with diff < 50 LOC labels Oct 9, 2026
@sayakpaul
sayakpaul requested a review from yiyixuxu October 9, 2026 07:07

This branch has not been deployed

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

Labels

size/S PR with diff < 50 LOC tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant