-
Notifications
You must be signed in to change notification settings - Fork 3.3k
[GPU] Enable dynamic quantization for MXFP8 & regular FP8 dtypes #35283
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
p-durandin
merged 56 commits into
openvinotoolkit:master
from
tkrupa-intel:tkrupa/mxfp8
Jun 25, 2026
Merged
Changes from all commits
Commits
Show all changes
56 commits
Select commit
Hold shift + click to select a range
395ff23
[GPU] Enable dynamic quantization for MXFP8 & regular FP8 dtypes
tkrupa-intel fb6ff10
Code formatting
tkrupa-intel 2556a2b
Remove unnecessary matcher macro
tkrupa-intel 15807e8
Apply review suggestions
tkrupa-intel a926796
Merge branch 'master' into tkrupa/mxfp8
tkrupa-intel 6e5833b
Fix potential memory issue
tkrupa-intel 2d59ee8
CI out of resources issue test
tkrupa-intel 60db5d3
Revert "CI out of resources issue test"
tkrupa-intel 3befb0e
CI out of resources issue test - nested node search
tkrupa-intel 355b0fa
Revert "CI out of resources issue test - nested node search"
tkrupa-intel c953fce
Bring back block write macro
tkrupa-intel e22b817
Revert "Bring back block write macro"
tkrupa-intel a4a70e2
Inline expensive wrapper functions
tkrupa-intel 71777fd
Get rid of expensive convert wrappers for non-fp dtypes
tkrupa-intel 5312577
Remove unnecessary preprocessor defines
tkrupa-intel 5295b31
Revert "Remove unnecessary preprocessor defines"
tkrupa-intel d776d23
Test block write macro impact
tkrupa-intel 35ceecd
Correct logical mistake
tkrupa-intel db90ea6
Revert "Test block write macro impact"
tkrupa-intel 3dea2a7
Check utils file include
tkrupa-intel 2f049ca
Retry removing extra defines
tkrupa-intel 90660f0
Revert "Check utils file include"
tkrupa-intel 2f62b62
Revert "Retry removing extra defines"
tkrupa-intel c2213f5
Revert pre_post_process.cpp changes
tkrupa-intel 893c827
Merge branch 'master' into tkrupa/mxfp8
tkrupa-intel c20a01c
Check if execution of new tests is connected to the issue
tkrupa-intel 8a5bf36
For real now
tkrupa-intel e57ea38
Frfr
tkrupa-intel 355c1d2
Merge branch 'master' into tkrupa/mxfp8
tkrupa-intel e50d868
Revert "Revert pre_post_process.cpp changes"
tkrupa-intel 3a9f2fe
Bring back tests
tkrupa-intel 2ab0ac1
Merge branch 'master' into tkrupa/mxfp8
tkrupa-intel 3c8e92b
Apply review suggestions
tkrupa-intel 6e847c1
Merge remote-tracking branch 'main/master' into HEAD
tkrupa-intel 360cc3e
Add missing test modifications
tkrupa-intel 89b7d28
Double check that it's not unit tests' fault
tkrupa-intel 3d2c6a9
Comment out troubleshooting part 1
tkrupa-intel 64fdb2c
Merge remote-tracking branch 'main/master' into HEAD
tkrupa-intel 619c0d6
Bring back cpp part
tkrupa-intel 721ba4f
Bring back reorder code and f8_utils
tkrupa-intel e7c3157
Remove just reorder and leave f8_utils
tkrupa-intel a7bed09
Merge branch 'master' into tkrupa/mxfp8
tkrupa-intel a771d04
Bring back everything and move f8_utils out of batch headers
tkrupa-intel 451790c
Remove unnecessary changes from transformations_pipeline
tkrupa-intel de0dfc7
Add more tests
tkrupa-intel fe59426
Remove unnecessary common.cl code
tkrupa-intel dbb2fd4
Merge branch 'master' into tkrupa/mxfp8
tkrupa-intel f0006bb
Merge correction
tkrupa-intel 59dba85
Code style & copyright
tkrupa-intel 712e6e0
Implement review suggestions
tkrupa-intel 6af3fa7
Move detection of guard patterns to a common function
tkrupa-intel 22758b1
Apply review suggestions
tkrupa-intel abf2d23
Code style
tkrupa-intel 8b7ac91
Apply review suggestions
tkrupa-intel c475d81
Remove unnecessary check
tkrupa-intel 850ea41
Fix zp handling
tkrupa-intel File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
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
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.
Oops, something went wrong.
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.
Why is required?
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.
Please see #35283 (comment)
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.
In addition, it is needed for correct model caching work:
visit_attributesis used during ov::Model serialization/deserializationUh oh!
There was an error while loading. Please reload this page.
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.
But serialization cannot just be added if attribute is not officially supported
If it was not serialized before it was not official Added in IR and must be reverted
If this change means the attribute is adde in IR it must be reverted.
Or it is used for internal serialization only?