Update

[jingyu-ml]

What does this PR do?

Type of change: ?

Usage

# Add a code snippet demonstrating how to use this

Testing

Before your PR is "Ready for review"

Make sure you read and follow Contributor guidelines and your commits are signed (git commit -s -S).

Make sure you read and follow the Security Best Practices (e.g. avoiding hardcoded trust_remote_code=True, torch.load(..., weights_only=False), pickle, etc.).

• Is this change backward compatible?: ✅ / ❌ / N/A
• If you copied code from any other sources or added a new PIP dependency, did you follow guidance in CONTRIBUTING.md: ✅ / ❌ / N/A
• Did you write any new necessary tests?: ✅ / ❌ / N/A
• Did you update Changelog?: ✅ / ❌ / N/A

Additional Information

Summary by CodeRabbit

• Refactor
  • Optimized the internal model parameter caching mechanism in the pipeline manager for improved code maintainability.

[copy-pr-bot]

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 31912891-da6c-469b-9124-03687fb8af82

📥 Commits

Reviewing files that changed from the base of the PR and between 5887410 and 5cc805f.

📒 Files selected for processing (1)

• examples/diffusers/quantization/pipeline_manager.py

---

📝 Walkthrough

Walkthrough

The caching logic for LTX-2 backbones in the pipeline manager was refactored to route model-parameter ledger getters through PipelineManager instance attributes instead of lambda closures that capture local variables, while maintaining the same initialization and caching behavior.

Changes

Cohort / File(s)	Summary
Caching Logic Refactoring examples/diffusers/quantization/pipeline_manager.py	Replaced closure-based variable capture with instance attribute references for transformer and video_decoder in model ledger getters. The cache initialization remains triggered once, but attribute access now routes through pm instance references rather than captured local variables.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check	❓ Inconclusive	The pull request title 'Update' is vague and does not convey meaningful information about the changeset, failing to summarize the main change.	Provide a more descriptive title that clearly summarizes the primary change, such as 'Refactor LTX-2 model caching to use instance attributes instead of closures' or similar.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Security Anti-Patterns	✅ Passed	Pull request adds only pipeline_manager.py with no security anti-patterns: no unsafe deserialization, no hardcoded trust_remote_code, no eval/exec, no nosec comments, no new dependencies.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch jingyux/fixed-ltx2-ootm

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.84%. Comparing base (c796611) to head (5cc805f).
⚠️ Report is 8 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1338      +/-   ##
==========================================
- Coverage   74.60%   73.84%   -0.77%     
==========================================
  Files         467      468       +1     
  Lines       50176    51431    +1255     
==========================================
+ Hits        37435    37980     +545     
- Misses      12741    13451     +710     

Flag	Coverage Δ
unit	52.71% <ø> (+0.35%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.