Skip to content

Add CLAUDE.md onboarding guide for Claude Code#37

Merged
haok1402 merged 1 commit into
mlc-ai:mainfrom
MasterJH5574:05-12-claude
May 12, 2026
Merged

Add CLAUDE.md onboarding guide for Claude Code#37
haok1402 merged 1 commit into
mlc-ai:mainfrom
MasterJH5574:05-12-claude

Conversation

@MasterJH5574
Copy link
Copy Markdown
Member

CLAUDE.md is auto-loaded into Claude Code's context for every session in this repo, so it doubles as a quick onboarding map for humans. It covers the common commands (lint, tests, training, dataset prep, memory estimation), the DualPipeV five-stage layer split with file pointers, the FP8 and 4D-parallelism wiring, and the agent workflows shipped under .claude/skills/.

CLAUDE.md is auto-loaded into Claude Code's context for every session
in this repo, so it doubles as a quick onboarding map for humans. It
covers the common commands (lint, tests, training, dataset prep,
memory estimation), the DualPipeV five-stage layer split with file
pointers, the FP8 and 4D-parallelism wiring, and the agent workflows
shipped under `.claude/skills/`.
Copy link
Copy Markdown

@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 introduces a CLAUDE.md file to provide comprehensive guidance for AI agents working within the PithTrain repository, covering setup, testing, and architecture. Feedback focused on correcting technical inaccuracies in the documentation, specifically regarding the DualPipeV.step() method's lack of a forward_only argument and the implementation details of the SlottedDefault base class.

Comment thread CLAUDE.md
Comment thread CLAUDE.md
Comment thread CLAUDE.md
@haok1402 haok1402 merged commit 6f332a7 into mlc-ai:main May 12, 2026
1 check passed
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