Skip to content
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

Fix fa_version_unsupported_reason and more #46

Closed
wants to merge 2 commits into from

Conversation

waltforme
Copy link

@waltforme waltforme commented Feb 11, 2025

Looks like there are some bugs in vllm_flash_attn/flash_attn_interface.py. This PR tries to fix them.

@waltforme waltforme changed the title Fix fa_version_unsupported_reason Fix fa_version_unsupported_reason and more Feb 12, 2025
Signed-off-by: Jun Duan <[email protected]>
@hmellor
Copy link

hmellor commented Feb 17, 2025

Superseded by #48 (sorry I didn't see your PR before making mine)

@waltforme
Copy link
Author

Superseded by #48 (sorry I didn't see your PR before making mine)

No problem. I will ask for a coffee for this if some day we meet offline (just kidding).

@waltforme waltforme closed this Feb 17, 2025
@waltforme waltforme deleted the fa2-unsupported branch February 17, 2025 17:18
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.

2 participants