Skip to content

Fix memory leak related to int8.#14697

Merged
comfyanonymous merged 1 commit into
masterfrom
comfyanonymous-patch-2
Jun 30, 2026
Merged

Fix memory leak related to int8.#14697
comfyanonymous merged 1 commit into
masterfrom
comfyanonymous-patch-2

Conversation

@comfyanonymous

Copy link
Copy Markdown
Member

No description provided.

@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedpypi/​comfy-kitchen@​0.2.15 ⏵ 0.2.16100100100100100 +20

View full report

@coderabbitai

coderabbitai Bot commented Jun 30, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 02368ff0-2028-4f2e-986e-33cb498ac27e

📥 Commits

Reviewing files that changed from the base of the PR and between 8fe0243 and 26a22de.

📒 Files selected for processing (1)
  • requirements.txt

📝 Walkthrough

Walkthrough

This change updates the pinned version of the comfy-kitchen dependency in requirements.txt from 0.2.15 to 0.2.16. No other dependencies or files are modified.

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title mentions an int8 memory leak fix, but the PR only bumps a dependency version in requirements.txt. Rename the title to reflect the dependency update, or include the actual memory-leak fix if that is the intended change.
Description check ❓ Inconclusive No pull request description was provided, so there is no meaningful description to evaluate. Add a brief description of what the dependency update is meant to change or fix.
✅ Passed checks (3 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@comfyanonymous comfyanonymous merged commit d395813 into master Jun 30, 2026
21 checks passed
@comfyanonymous comfyanonymous deleted the comfyanonymous-patch-2 branch June 30, 2026 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants