Skip to content

Conversation

Seven-Streams
Copy link
Collaborator

This is a rebased version of #383.
This PR refactors the pipeline. The functors in grammar_functor are divided into three types:

  • grammar_normalizer: When a grammar is constructed, they should be called immediately.
  • grammar_optimizer: When a grammar is going to be compiled, they will be called.
  • grammar_constructor: They are used to construct a new grammar, like constructing the union of two grammars.

@Seven-Streams Seven-Streams force-pushed the main-dev/2025-09-11/refac_pipeline branch from 226177e to a2bc8e6 Compare September 13, 2025 11:07
@Seven-Streams Seven-Streams force-pushed the main-dev/2025-09-11/refac_pipeline branch from a2bc8e6 to d2b510a Compare September 20, 2025 12:23
Copy link
Collaborator

@Ubospica Ubospica left a comment

Choose a reason for hiding this comment

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

Thanks

refac the file structure.

add files.

finish the basic refactor.

finish the basic refac.

add the optimized.

fix the most tests.

finish all the tests.

Signed-off-by: Yuchuan <[email protected]>

finish.

Signed-off-by: Yuchuan <[email protected]>

format.

Signed-off-by: Yuchuan <[email protected]>

fix cpp tests.

normalize.

fix.

format.

Signed-off-by: Yuchuan <[email protected]>

refac.

fix.

refac testing.

fix.

fix test.

tests.

format.

test, bump version.

format.

fix test.

fix stag converter test.

fix tests.

Signed-off-by: Yuchuan <[email protected]>

doc.

functor.
Signed-off-by: Yuchuan <[email protected]>
@Seven-Streams Seven-Streams force-pushed the main-dev/2025-09-11/refac_pipeline branch from efecca4 to 958f457 Compare September 21, 2025 10:19
Signed-off-by: Yuchuan <[email protected]>
@Seven-Streams Seven-Streams merged commit 9a15539 into mlc-ai:main Sep 25, 2025
65 of 68 checks 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