-
Notifications
You must be signed in to change notification settings - Fork 31.1k
[Flash Attention] Fix flash attention integration
#40002
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
Merged
Changes from all commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
0ccc67d
fix flash attention
vasqu 0f97be0
i got a stroke reading that comment
vasqu 0a45416
change dropout kwarg back to before
vasqu 92e2075
rename _fa3... as it's used for multiple variants and should work as …
vasqu 22574dc
Merge branch 'main' into fix-fa-integration
vasqu 3581bd6
simplify imports and support kwargs for fa
vasqu 2f607ba
style
vasqu 49ce7ae
fix comments order
vasqu 5f7d937
small fix
vasqu d21095c
skip kernels test (causes cuda illegal memories w/o cleanup), fix fa …
vasqu 36bfffb
style
vasqu 9ec8c45
Merge branch 'main' into fix-fa-integration
vasqu 1612b56
allow fullgraph by preloading on init
vasqu ba8fd00
make globals "private"
vasqu 5240985
Merge branch 'main' into fix-fa-integration
vasqu 3dbf11a
ci pls be happy
vasqu d9d8ff7
change skip conditions based on backend flag (indicating missing mask…
vasqu ec0fbf3
move globals support to a function to prepare kwargs
vasqu a6996f5
style
vasqu ce0e586
Merge branch 'main' into fix-fa-integration
vasqu 86c9e81
generalize supported kwargs
vasqu 3ae14ec
small change to doc
vasqu 89b8f95
fix
vasqu 24512e4
Merge branch 'main' into fix-fa-integration
vasqu ec41ac5
Merge branch 'main' into fix-fa-integration
vasqu 74a8987
Merge branch 'main' into fix-fa-integration
vasqu 8f91219
add comments
vasqu 6a0a3a1
style
vasqu 54ed29e
Merge branch 'main' into fix-fa-integration
vasqu 7016548
revert prep during generate
vasqu d9da331
Merge branch 'main' into fix-fa-integration
vasqu a996bd5
style
vasqu 07fafe1
revert weird style changes
vasqu a98fac4
add fa kwarg prep during generate with fixes back
vasqu 9971d75
how did this even happen
vasqu 5e2d35f
how
vasqu 85cb1b1
Merge branch 'main' into fix-fa-integration
vasqu 4ad364c
add comment
vasqu cb89dbe
Merge branch 'main' into fix-fa-integration
vasqu 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
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.
Uh oh!
There was an error while loading. Please reload this page.