Skip to content

[Doc] Install docs add docker install method#961

Merged
LeiWang1999 merged 1 commit intotile-ai:mainfrom
BBuf:install_docs_add_docker_install_method
Oct 10, 2025
Merged

[Doc] Install docs add docker install method#961
LeiWang1999 merged 1 commit intotile-ai:mainfrom
BBuf:install_docs_add_docker_install_method

Conversation

@BBuf
Copy link
Contributor

@BBuf BBuf commented Oct 9, 2025

I encountered multiple issues when building TileLang from source on B200 using the tutorial. For instance, the PyTorch version in the default Docker environment doesn't support the sm100 architecture, requiring manual creation of a miniconda environment to upgrade PyTorch and install all dependencies before compiling TVM/TileLang. This process essentially equals building Docker using TileLang's provided Dockerfile. The Docker build approach is much more convenient and streamlined, completing the entire environment setup without any extra compilation overhead. Therefore, I've added a new installation method that allows users to easily set up a clean TileLang development environment on any GPU.

图片

Summary by CodeRabbit

  • Documentation
    • Added a recommended Docker-based installation path with prerequisites, step-by-step build/run/verify instructions and command examples.
    • Renumbered installation methods: Source install is now Method 2, Bundled TVM Submodule is Method 3, and Provided Script is Method 4.
    • Updated headings, anchors, and cross-references to reflect the new order.
    • Expanded and clarified instructions across methods to streamline setup and verification.

@github-actions
Copy link

github-actions bot commented Oct 9, 2025

👋 Hi! Thank you for contributing to the TileLang project.

Please remember to run bash format.sh in the root directory of the project to ensure your changes are properly linted and formatted. This will help ensure your contribution passes the format check.

We appreciate you taking this step! Our team will review your contribution, and we look forward to your awesome work!

🚀

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 9, 2025

Walkthrough

The installation guide is restructured to introduce Docker as the new Method 1 (Recommended), with detailed steps for prerequisites, clone/build/run, and verification. Existing methods are renumbered: previous Methods 1–3 become Methods 2–4. Anchors, headings, and cross-references are updated accordingly.

Changes

Cohort / File(s) Summary of Changes
Docs: Installation restructuring
docs/get_started/Installation.md
Added full Docker-based installation workflow as Method 1 (Recommended); renumbered prior methods (Source→Method 2, TVM Submodule→Method 3, Script→Method 4); updated anchors/headings; expanded commands and verification steps; adjusted cross-references.

Sequence Diagram(s)

sequenceDiagram
    autonumber
    actor U as User
    participant D as Installation Docs
    participant G as Git/Repo
    participant DE as Docker Engine
    participant C as Container
    participant V as Verification

    Note over D: New Method 1 (Docker) — recommended
    U->>D: Read Docker-based instructions
    U->>G: git clone repo
    U->>DE: docker build image
    DE-->>U: Build logs
    U->>DE: docker run container
    DE->>C: Start container
    C-->>U: Runtime ready
    U->>C: Execute verification command(s)
    C->>V: Produce output
    V-->>U: Confirm installation
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I stacked my crates, a docker’d nest,
With gentle paws, I ran the test.
Four paths now hop, but one leads quick—
A carrot-marked, container trick.
I twitch my whiskers, logs aglow—
It builds, it runs, we’re good to go! 🥕🐇

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title clearly identifies that the pull request is a documentation update to add a Docker installation method and uses the “[Doc]” prefix to indicate its scope, accurately summarizing the primary change without introducing unrelated details.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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 and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 10adb79 and b16d0cd.

📒 Files selected for processing (1)
  • docs/get_started/Installation.md (3 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.18.1)
docs/get_started/Installation.md

73-73: Link fragments should be valid

(MD051, link-fragments)


74-74: Link fragments should be valid

(MD051, link-fragments)


75-75: Link fragments should be valid

(MD051, link-fragments)


76-76: Link fragments should be valid

(MD051, link-fragments)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: build-test-amd

Comment on lines +73 to +76
1. [Install Using Docker](#install-method-1) (Recommended)
2. [Install from Source (using your own TVM installation)](#install-method-2)
3. [Install from Source (using the bundled TVM submodule)](#install-method-3)
4. [Install Using the Provided Script](#install-method-4)
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

Fix broken section links in methods list.

The fragment identifiers #install-method-1#install-method-4 do not match any generated IDs, so each link in the list ends up broken (see markdownlint MD051). Please point them at the actual heading slugs (or use {ref} syntax if you want myst anchors). A minimal fix is:

-1. [Install Using Docker](#install-method-1) (Recommended)
-2. [Install from Source (using your own TVM installation)](#install-method-2)
-3. [Install from Source (using the bundled TVM submodule)](#install-method-3)
-4. [Install Using the Provided Script](#install-method-4)
+1. [Install Using Docker](#method-1-install-using-docker-recommended) (Recommended)
+2. [Install from Source (Using Your Own TVM Installation)](#method-2-install-from-source-using-your-own-tvm-installation)
+3. [Install from Source (Using the Bundled TVM Submodule)](#method-3-install-from-source-using-the-bundled-tvm-submodule)
+4. [Install Using the Provided Script](#method-4-install-using-the-provided-script)
📝 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
1. [Install Using Docker](#install-method-1) (Recommended)
2. [Install from Source (using your own TVM installation)](#install-method-2)
3. [Install from Source (using the bundled TVM submodule)](#install-method-3)
4. [Install Using the Provided Script](#install-method-4)
1. [Install Using Docker](#method-1-install-using-docker-recommended) (Recommended)
2. [Install from Source (Using Your Own TVM Installation)](#method-2-install-from-source-using-your-own-tvm-installation)
3. [Install from Source (Using the Bundled TVM Submodule)](#method-3-install-from-source-using-the-bundled-tvm-submodule)
4. [Install Using the Provided Script](#method-4-install-using-the-provided-script)
🧰 Tools
🪛 markdownlint-cli2 (0.18.1)

73-73: Link fragments should be valid

(MD051, link-fragments)


74-74: Link fragments should be valid

(MD051, link-fragments)


75-75: Link fragments should be valid

(MD051, link-fragments)


76-76: Link fragments should be valid

(MD051, link-fragments)

🤖 Prompt for AI Agents
In docs/get_started/Installation.md around lines 73-76, the list uses broken
fragment IDs (#install-method-1 … #install-method-4); update each link to the
actual heading slug (e.g. #install-using-docker,
#install-from-source-using-your-own-tvm-installation,
#install-from-source-using-the-bundled-tvm-submodule,
#install-using-the-provided-script) or replace with MyST cross-reference syntax
like {ref}`Install Using Docker <install-using-docker>` (or the equivalent for
each heading) so the anchors match the generated IDs and the links resolve.

@LeiWang1999
Copy link
Member

@BBuf Thanks for your contribution! The difficulty in installing packages on the B200 is likely due to its ARM CPU processor. For instance, the PyTorch version in the default Docker environment doesn't support the sm100 architecture, which necessitates the manual creation of a Miniconda environment to upgrade PyTorch and install all dependencies before compiling TVM/TileLang. By 'default Docker environment,' do you mean your own environment, or the Dockerfiles we provide?

@BBuf
Copy link
Contributor Author

BBuf commented Oct 10, 2025

@BBuf Thanks for your contribution! The difficulty in installing packages on the B200 is likely due to its ARM CPU processor. For instance, the PyTorch version in the default Docker environment doesn't support the sm100 architecture, which necessitates the manual creation of a Miniconda environment to upgrade PyTorch and install all dependencies before compiling TVM/TileLang. By 'default Docker environment,' do you mean your own environment, or the Dockerfiles we provide?

I just follow the install doc step in ubuntu host:

图片

I did not use TileLang's Dockerfile to build docker image first, so I encountered above issues.

@LeiWang1999
Copy link
Member

I see, overall lgtm, thanks.

@LeiWang1999 LeiWang1999 merged commit 6031416 into tile-ai:main Oct 10, 2025
7 of 8 checks passed
@coderabbitai coderabbitai bot mentioned this pull request Oct 11, 2025
21 tasks
RubiaCx pushed a commit to RubiaCx/tilelang that referenced this pull request Nov 24, 2025
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.

2 participants