Skip to content

Baseline compile-model binary - #1680

Closed
elliottslaughter wants to merge 5 commits into
flexflow:masterfrom
elliottslaughter:bin-compile-model
Closed

Baseline compile-model binary#1680
elliottslaughter wants to merge 5 commits into
flexflow:masterfrom
elliottslaughter:bin-compile-model

Conversation

@elliottslaughter

@elliottslaughter elliottslaughter commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

This PR provides a basic compile-model binary that enables users to compile models. Currently, the only switch is the compilation method (passthrough, data_parallel, unity, or mcmc) and the actual compiler parameters are hard-coded. Passthrough means that the compiler is not invoked and the binary constructs a mapping of the CG for one device without introducing any parallelism.

The intention is that this will be subsequently extended to take more compiler parameters in the future.


This change is Reviewable

lockshaw added a commit that referenced this pull request Sep 9, 2026
This PR provides a basic compile-model binary that enables users to
compile models. Currently, the only switch is the compilation method
(passthrough, data_parallel, unity, or mcmc) and the actual compiler
parameters are hard-coded. Passthrough means that the compiler is not
invoked and the binary constructs a mapping of the CG for one device
without introducing any parallelism. A machine specification can be
provided through using the --machine-spec-json flag.

To allow passing a value (in this case the path) to the
--machine-spec-json flag, we add new a new CLINamedArgumentSpec type to
utils/cli along with corresponding parsing logic and tests.

We also update proj so that proj run no longer outputs build output to
stderr, allowing redirecting stdout to a file to work (important for,
e.g., export-model-arch).

The intention is that bin/compile-model will be extended to take more
compiler parameters in the future.

Co-authored-by: Colin Unger <lockshaw@lockshaw.net>
@lockshaw

lockshaw commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

Merged in bdcbed8308181b442121967969ad0cbf3d3078dc.

@lockshaw lockshaw closed this Sep 9, 2026
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