Skip to content

Upgrade Jinja to 1.3.0 - #226

Merged
pcuenca merged 2 commits into
huggingface:mainfrom
tpae:jinja-upgrade
Sep 9, 2025
Merged

Upgrade Jinja to 1.3.0#226
pcuenca merged 2 commits into
huggingface:mainfrom
tpae:jinja-upgrade

Conversation

@tpae

@tpae tpae commented Sep 8, 2025

Copy link
Copy Markdown
Contributor

Thanks to @johnmai-dev, we were able to make minor performance optimizations on Swift Jinja package, more information can be found here: huggingface/swift-jinja#24.

Benchmark Results:

Phase Before (ms) After (ms) Change % Improvement
Tokenize avg 69.525 68.887 -0.638 ~0.9% faster
Parse avg 0.498 0.457 -0.041 ~8.2% faster
Run avg 0.116 0.099 -0.017 ~14.7% faster
Template.render 0.176 0.142 -0.034 ~19.3% faster

@FL33TW00D

Copy link
Copy Markdown
Contributor

CC: @pcuenca
We are also looking into a better solution for the 429's in CI so that the tests can pass!

@pcuenca

pcuenca commented Sep 9, 2025

Copy link
Copy Markdown
Member

Apologies for the failed CI run, we are in the process of fixing it. Will verify that tests pass locally and then merge!

@pcuenca
pcuenca merged commit f000aa7 into huggingface:main Sep 9, 2025
1 of 2 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.

3 participants