fix(pricing): remove extra pipe chars breaking Ideogram and Krea heading rendering#1304
Closed
lin-bot23 wants to merge 4 commits into
Closed
fix(pricing): remove extra pipe chars breaking Ideogram and Krea heading rendering#1304lin-bot23 wants to merge 4 commits into
lin-bot23 wants to merge 4 commits into
Conversation
Two users asked about MLX support in ComfyUI on 2026-07-18 via the Mintlify AI assistant. The docs did not address whether ComfyUI uses MLX or how acceleration works on Apple Silicon. Adds an Accordion FAQ entry to the system requirements page clarifying that ComfyUI uses PyTorch MPS backend (not MLX) on Apple Silicon Macs, with a command to verify MPS availability. Source: Mintlify AI Assistant analytics data (2026-07-18)
…as array framework for Apple Silicon
…ng rendering The Ideogram and Krea sections in all 4 language pricing pages had extra leading pipe characters (e.g. ||## Ideogram instead of ## Ideogram), causing the markdown headings to be misinterpreted as table content. Removed the stray pipes to restore proper rendering.
📝 WalkthroughWalkthroughAdded an MLX support FAQ to system requirements documentation in four languages, updated translated source hashes, and corrected Ideogram pricing table Markdown in four localized pricing pages. ChangesMLX support FAQ
Ideogram pricing markup
Possibly related PRs
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
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. Comment |
Contributor
Author
|
Replaced by clean PR — only pricing fix commits |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@installation/system_requirements.mdx`:
- Line 72: Update the MLX description text to use “machine learning” without a
hyphen, preserving the rest of the sentence unchanged.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: edc8c3cc-5d8f-43a3-a535-3d7bf2e572fa
📒 Files selected for processing (8)
installation/system_requirements.mdxja/installation/system_requirements.mdxja/tutorials/partner-nodes/pricing.mdxko/installation/system_requirements.mdxko/tutorials/partner-nodes/pricing.mdxtutorials/partner-nodes/pricing.mdxzh/installation/system_requirements.mdxzh/tutorials/partner-nodes/pricing.mdx
| <Accordion title="Does ComfyUI support MLX (Apple's machine learning framework)?"> | ||
| ComfyUI does not use MLX directly. On Apple Silicon (M1/M2/M3/M4) Macs, ComfyUI uses PyTorch with the **MPS (Metal Performance Shaders)** backend, which leverages Apple's Metal graphics framework for GPU acceleration. | ||
|
|
||
| MLX is a separate machine-learning array framework designed for Apple Silicon. ComfyUI's PyTorch-based architecture uses MPS instead. |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value
Remove the hyphen in "machine-learning" for consistency.
A hyphen here is quite a fright, remove it so the text is right! It matches the title up above, for a consistency that we love. Let's make "machine learning" uniform across the page like a spell cast by a mischievous sage.
✨ Proposed fix to banish the hyphen
-MLX is a separate machine-learning array framework designed for Apple Silicon. ComfyUI's PyTorch-based architecture uses MPS instead.
+MLX is a separate machine learning array framework designed for Apple Silicon. ComfyUI's PyTorch-based architecture uses MPS instead.📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| MLX is a separate machine-learning array framework designed for Apple Silicon. ComfyUI's PyTorch-based architecture uses MPS instead. | |
| MLX is a separate machine learning array framework designed for Apple Silicon. ComfyUI's PyTorch-based architecture uses MPS instead. |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@installation/system_requirements.mdx` at line 72, Update the MLX description
text to use “machine learning” without a hyphen, preserving the rest of the
sentence unchanged.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The Ideogram and Krea sections in all 4 language pricing pages had extra leading pipe characters, causing markdown headings to be misinterpreted as table content.
Files
Change