-
Notifications
You must be signed in to change notification settings - Fork 116
fix: unpin xgrammar when bug is fixed #990
Copy link
Copy link
Open
Labels
area/backendsProvider-specific work: Ollama, HF, LiteLLM, OpenAI, Bedrock, vLLMProvider-specific work: Ollama, HF, LiteLLM, OpenAI, Bedrock, vLLMarea/intrinsicsGranite intrinsic adapters: RAG, Guardian, CoreGranite intrinsic adapters: RAG, Guardian, Corearea/stdlibCore abstractions: Context, MOT, SamplingStrategy, formatters, serializationCore abstractions: Context, MOT, SamplingStrategy, formatters, serializationbugSomething isn't workingSomething isn't working
Description
Metadata
Metadata
Assignees
Labels
area/backendsProvider-specific work: Ollama, HF, LiteLLM, OpenAI, Bedrock, vLLMProvider-specific work: Ollama, HF, LiteLLM, OpenAI, Bedrock, vLLMarea/intrinsicsGranite intrinsic adapters: RAG, Guardian, CoreGranite intrinsic adapters: RAG, Guardian, Corearea/stdlibCore abstractions: Context, MOT, SamplingStrategy, formatters, serializationCore abstractions: Context, MOT, SamplingStrategy, formatters, serializationbugSomething isn't workingSomething isn't working
Type
Fields
Give feedbackNo fields configured for issues without a type.
Issue in xgrammar repo: mlc-ai/xgrammar#617
The xgrammar issue has a repro script that we can use to test. Within our own code, we can running
test/formatters/granite/test_intrinsics_formatters.py::test_run_transformerswithout CICD set, resulting in:Pinning to v0.1.33 fixed the issue.