Skip to content

feat: support | first filter#14

Merged
ochafik merged 2 commits intoochafik:mainfrom
zhaob1n:first-filter
Dec 20, 2025
Merged

feat: support | first filter#14
ochafik merged 2 commits intoochafik:mainfrom
zhaob1n:first-filter

Conversation

@zhaob1n
Copy link
Copy Markdown

@zhaob1n zhaob1n commented Dec 17, 2025

the chat template in unsloth/Qwen3-Next-80B-A3B-Thinking-GGUF uses | first

{%- set reasoning_content = ((content.split('</think>')|first).rstrip('\n').split('<think>')|last).lstrip('\n') %} {%- set content = (content.split('</think>')|last).lstrip('\n') %}

@CISC
Copy link
Copy Markdown
Collaborator

CISC commented Dec 17, 2025

That's strange, @danielhanchen why would you replace [0] and [-1] from the original template?

@xcpky Anyway, it's an obvious omission from minja, thanks for the submission! :)

Copy link
Copy Markdown
Owner

@ochafik ochafik left a comment

Choose a reason for hiding this comment

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

Thanks @xcpky !

@ochafik ochafik merged commit 88a7210 into ochafik:main Dec 20, 2025
@danielhanchen
Copy link
Copy Markdown

Oh hi sorry didn't see this - this was probs ages ago to support LM Studio not parsing [0], [-1] correctly, hence the |first and |last

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.

4 participants