Skip to content

Allow disabling copy_missing_tensors_from_source via env var, default enabled#1623

Open
xin3he wants to merge 6 commits intomainfrom
xinhe/2-26b
Open

Allow disabling copy_missing_tensors_from_source via env var, default enabled#1623
xin3he wants to merge 6 commits intomainfrom
xinhe/2-26b

Conversation

@xin3he
Copy link
Contributor

@xin3he xin3he commented Mar 26, 2026

Description

For develop usage, AR_DISABLE_COPY_MTP_WEIGHTS for tiny model

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Performance improvement
  • Code refactoring
  • Other (please specify):

Related Issues

Fixes or relates to #1615

Checklist Before Submitting

  • My code has been tested locally.
  • Documentation has been updated as needed.
  • New or updated tests are included where applicable.

Co-authored-by: Yi Liu <yi4.liu@intel.com>
Copilot AI review requested due to automatic review settings March 26, 2026 08:44
Co-authored-by: Xin He <xin3.he@intel.com>
@xin3he
Copy link
Contributor Author

xin3he commented Mar 26, 2026

/azp run Unit-Test-CUDA-AutoRound

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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

Adds an environment-variable-controlled switch to optionally skip copy_missing_tensors_from_source during model export (intended for dev/tiny-model workflows).

Changes:

  • Gate copy_missing_tensors_from_source(...) in save_model() behind an env var (default enabled).
  • Register a new environment variable in auto_round.envs.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
auto_round/export/utils.py Adds env-var gating around missing-tensor copy during export.
auto_round/envs.py Introduces a new env var entry for disabling the missing-tensor copy behavior.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@xin3he
Copy link
Contributor Author

xin3he commented Mar 26, 2026

/azp run Unit-Test-CUDA-AutoRound

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@chensuyue chensuyue added this to the 0.12.0 milestone Mar 26, 2026
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.

4 participants