Skip to content

update ar models#1110

Open
gushiqiao wants to merge 1 commit into
mainfrom
gsq/dev-seko-ar-2
Open

update ar models#1110
gushiqiao wants to merge 1 commit into
mainfrom
gsq/dev-seko-ar-2

Conversation

@gushiqiao
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request refactors the KV cache management system by removing several quantization schemes (such as Sage, TurboQuant, and LongLive FP4) along with their calibration logic, leaving only the KIVI and FP schemes. It also cleans up the runners by removing live-streaming functionality and optimizing VAE decoding. Additionally, synchronization has been improved in the offloaded rolling cache pool to prevent overwriting shared staging buffers. The primary feedback is to avoid hardcoding absolute, user-specific local paths in shared configuration files and scripts to ensure environment portability.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

"vae_cpu_offload": false,
"use_image_encoder": false,
"dit_original_ckpt": "path/to/lingbot_world_fast/",
"dit_original_ckpt": "/data/nvme4/models/lingbot-world-base-cam/lingbot_world_fast",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

Avoid hardcoding absolute, user-specific local paths (e.g., /data/nvme4/gushiqiao/...) in shared configuration files. This makes the configuration non-portable and will fail on other environments. Consider using a relative path or a placeholder/environment variable instead.

Comment on lines +20 to +24
"dit_quantized_ckpt": "/data/nvme5/gushiqiao/models/SekoTalk-Distill-AR/converted_fp8.safetensors",
"dit_quant_scheme": "fp8-sgl",
"adapter_quantized": true,
"adapter_quant_scheme": "fp8",
"adapter_model_path": "/models/seko_ar/-audio_adapter_fp8.pt",
"adapter_model_path": "/data/nvme5/gushiqiao/models/SekoTalk-Distill-AR/-audio_adapter_fp8.pt",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

Avoid hardcoding absolute, user-specific local paths (e.g., /data/nvme5/gushiqiao/...) in shared configuration files. This makes the configuration non-portable and will fail on other environments. Consider using relative paths or placeholders/environment variables instead.

Comment on lines +4 to +5
lightx2v_path=/data/nvme4/gushiqiao/new/LightX2V
model_path=/data/nvme4/models/mgv2
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

Avoid hardcoding absolute, user-specific local paths (e.g., /data/nvme4/gushiqiao/...) in shared scripts. This makes the script non-portable and will fail on other environments. Consider using relative paths or placeholders/environment variables instead.

Comment on lines +5 to +6
lightx2v_path=/data/nvme4/gushiqiao/new/LightX2V
model_path=/data/nvme4/models/mgv2
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

Avoid hardcoding absolute, user-specific local paths (e.g., /data/nvme4/gushiqiao/...) in shared scripts. This makes the script non-portable and will fail on other environments. Consider using relative paths or placeholders/environment variables instead.

Comment on lines +3 to +4
lightx2v_path=/data/nvme4/gushiqiao/new/LightX2V
model_path=/data/nvme5/gushiqiao/models/SekoTalk-Distill-AR/
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

Avoid hardcoding absolute, user-specific local paths (e.g., /data/nvme4/gushiqiao/...) in shared scripts. This makes the script non-portable and will fail on other environments. Consider using relative paths or placeholders/environment variables instead.

Comment on lines +4 to +5
lightx2v_path=/data/nvme4/gushiqiao/new/LightX2V
model_path=/data/nvme0/gushiqiao/models/official_models/Wan2.1-T2V-1.3B/
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

Avoid hardcoding absolute, user-specific local paths (e.g., /data/nvme4/gushiqiao/...) in shared scripts. This makes the script non-portable and will fail on other environments. Consider using relative paths or placeholders/environment variables instead.

--task i2v \
--model_path $model_path \
--config_json /data/nvme4/gushiqiao/new/LightX2V/configs/lingbot_fast/lingbot_fast_i2v_kv_turboquant.json \
--config_json /data/nvme4/gushiqiao/new/LightX2V/configs/lingbot_fast/lingbot_fast_i2v.json \
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

Avoid hardcoding absolute, user-specific local paths (e.g., /data/nvme4/gushiqiao/...) in shared scripts. This makes the script non-portable and will fail on other environments. Consider using relative paths or placeholders/environment variables instead.

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.

1 participant