- Notifications
You must be signed in to change notification settings - Fork 14.1k
llama : add gpt-oss#15091
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
Merged
+2,943 −228
Merged
llama : add gpt-oss #15091
Changes from all commits
Commits
Show all changes
48 commits Select commit Hold shift + click to select a range
81991fc oai moe
ngxson 917f923 compat with new checkpoint
ngxson a4ab869 add attn sink impl
ngxson 3801c36 add rope scaling yarn
ngxson 13f39f6 logits match with latest transformers code
ngxson b3594b3 wip chat template
ngxson bd57158 Merge branch 'master' into xsn/oai_moe
ngxson 089a7ab rm trailing space
ngxson 4d01b36 use ggml_scale_bias
ngxson f271cc8 Merge branch 'master' into xsn/oai_moe
ngxson 106b17e rm redundant is_swa_all
ngxson e2c1beb convert interleaved gate_up
ngxson 4431c82 Merge remote-tracking branch 'gg-public/master' into xsn/oai_moe-gg
ggerganov fe9b818 Merge remote-tracking branch 'gg-public/master' into xsn/oai_moe-gg
ggerganov 539c2b6 Merge remote-tracking branch 'gg-public/master' into xsn/oai_moe-gg
ggerganov 039a6f1 graph : fix activation function to match reference (#7)
ggerganov aa240b9 Merge branch 'master' into xsn/oai_moe-gg
ggerganov 32a654c Merge branch 'master' into xsn/oai_moe-gg
ggerganov 13f3568 vocab : handle o200k_harmony special tokens
ggerganov e59b2eb ggml : add attention sinks support (#1)
ggerganov 832dc26 repack mxfp4 upon conversion
ngxson c68069d clean up a bit
ngxson 423b191 enable thinking
ngxson 4dd479b add quick hack to render only some special tokens
ngxson ebc7da5 fix bf16 conversion
ngxson a543ddf remove vocab hack
ngxson 6b30372 webui ok
ngxson 44bdb75 support chat parsing for gpt-oss
ngxson 65b536f Merge branch 'master' into xsn/oai_moe
ggerganov 6197917 fix webui
ngxson 3c4725b direct mapping mxfp4, FINALLY
ngxson 04cfb6d force using mxfp4
ngxson 4cf69df properly use lazy tensor
ngxson ec95c0e ggml : add mxfp4
ggerganov 3ef6c8c ggml : add ggml_add_id (#13)
slaren cd514cc Merge branch 'master' into xsn/oai_moe
slaren 98c4be5 Merge branch 'xsn/oai_moe' into mxfp4-rebased
slaren fcb2339 Merge branch 'master' into gpt-oss-mxfp4
ngxson 98f3444 llama : fix compile error
ggerganov df8411e cuda : add fallback for __nv_cvt_e8m0_to_bf16raw
slaren 60ab08a cleanup
slaren 256fe66 sycl : fix supports_op for MXFP4
slaren cd8ed32 fix Unknown reasoning format
ngxson a3b291e ggml-cpu : fix AVX build
slaren 1ea3769 fix hip build
slaren 07d781e cuda : add mxfp4 dequantization support for cuBLAS
slaren b236c90 ggml-cpu : fix mxfp4 fallback definitions for some architectures
slaren d9d89b4 cuda : fix version required for __nv_cvt_e8m0_to_bf16raw
slaren File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Uh oh!
There was an error while loading. Please reload this page.
Jump to
Jump to file
Failed to load files.
Loading
Uh oh!
There was an error while loading. Please reload this page.
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Oops, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Oops, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Too late, but why was this split? Only adds extra ops on the graph...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The gate_up tensor is organized in a way that a row of gate is followed by a row of up, aka interleaving. While we can rearrange it to the expected layout for fused op, I think it's easier to just split it into gate and up independently
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ahhh, didn't catch that.