Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[pull] main from mlc-ai:main #302

Merged
merged 13 commits into from
Nov 16, 2024
Merged

[pull] main from mlc-ai:main #302

merged 13 commits into from
Nov 16, 2024

Conversation

pull[bot]
Copy link

@pull pull bot commented Nov 16, 2024

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

shyeonn and others added 13 commits October 31, 2024 10:34
Relative Path causes this error. #2600 

One can really only use an absolute path here
docs: Update model template link address
This PR migrates the implementation of the grammar engine to
an external library XGrammar. The implementation of XGrammar is
basically the same as our current version, but more modularized and
robust. XGrammar will be continuously enhanced in the future.

See https://github.com/mlc-ai/xgrammar for more details.
[docs] Updated path pointers in both docs files to point to the
`conversation_template` directory instead of linking to the
non-existent `conversation_template.py` file.

Co-authored-by: 涧波 <[email protected]>
This PR bumps the 3rdparty/tokenizer-cpp.

The SentencePiece tokenizer was disabled by default to
reduce the binary size of the built library, while it causes
some error when users expect to use the SentencePiece
tokenizer.

This PR enables it by default. And we will need to manually
disable it if we need to reduce its binary size.
This PR fixes the function table initialization, so that when
sliding window is enabled, we won't pick the FlashInfer attn
kernel.
This PR bumps the 3rdparty tokenizers-cpp to update the HuggingFace
tokenizers package dependency.
This PR adds the sentencepiece package as a Python installation
requirement for tokenizer coverage when running `gen_config`.
@pull pull bot added the ⤵️ pull label Nov 16, 2024
@pull pull bot merged commit 967fb76 into kp-forks:main Nov 16, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants