PR #43121 [`FA`] Generalize fa config checks and fix flags

[`FA`] Generalize fa config checks and fix flags #43121

vasqu merged 19 commits into huggingface:main from vasqu:fix-general-fa-checks

fix fa checks

4da5af25

oops

864fd87b

yea makes sense

f402a7a7

fix

fe29ce8e

did not intend to add this

c5b39474

fixup tests and wrong flags

a231a2be

vasqu changed the title ~~[`FA`] Generalize fa config checks~~ [`FA`] Generalize fa config checks and fix flags 13 days ago

vasqu marked this pull request as ready for review 13 days ago

github-actions requested a review from

ArthurZucker 13 days ago

github-actions requested a review from

Rocketknight1 13 days ago

vasqu removed review request from

Rocketknight1 13 days ago

vasqu requested a review from

Cyrilvallez 13 days ago

fix

3570ca15

style

df5b8f18

restrict target guided attn

4e36b73b

remove sam and sam hq (dont support fa anyways), make it fallback as …

42078a8b

huggingface deleted a comment from github-actions on 2026-01-07

Merge branch 'main' into fix-general-fa-checks

2b3b7022

missed this one

b809d62a

Cyrilvallez commented on 2026-01-12

Merge branch 'main' into fix-general-fa-checks

a979f953

move check to separate function

df83111d

style

4af38061

Merge branch 'main' into fix-general-fa-checks

d05b5812

sync changes with main

4931796e

vasqu requested a review from

Cyrilvallez 6 days ago

Cyrilvallez approved these changes on 2026-01-15

force non ambiguity + nit

e034e49c

Merge branch 'main' into fix-general-fa-checks

6c512918

vasqu merged 9eea1b0b into main 4 days ago

vasqu deleted the fix-general-fa-checks branch 4 days ago

Reviewers

Cyrilvallez

ArthurZucker

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

transformers [`FA`] Generalize fa config checks and fix flags #43121 Merged

[`FA`] Generalize fa config checks and fix flags #43121

transformers
[`FA`] Generalize fa config checks and fix flags
#43121

Merged