Skip to content

Fix the check of models supporting FA/SDPA not run#28202

Merged
ydshieh merged 3 commits intomainfrom
fix_support_list_test_not_run
Dec 22, 2023
Merged

Fix the check of models supporting FA/SDPA not run#28202
ydshieh merged 3 commits intomainfrom
fix_support_list_test_not_run

Conversation

@ydshieh
Copy link
Copy Markdown
Collaborator

@ydshieh ydshieh commented Dec 22, 2023

What does this PR do?

The original check (as test methods) in tests/utils/test_doc_samples.py won't run (and didn't run like in #28133) as that file is not impacted by the modeling files (in terms of import relation).

Those 2 checks don't need torch at all and could be done in the first stage of check (check_repository_consistency)

@ydshieh ydshieh changed the title Fix the check of the supported FA/SDPA models test not run Fix the check of models supporting FA/SDPA not run Dec 22, 2023
@ydshieh ydshieh force-pushed the fix_support_list_test_not_run branch from 8359110 to 7314d18 Compare December 22, 2023 10:20
@ydshieh ydshieh requested a review from amyeroberts December 22, 2023 10:32
@inkinworld
Copy link
Copy Markdown
Contributor

inkinworld commented Dec 22, 2023

i create the pr(#28201) check failed casue by tests/utils/test_doc_samples.py faild.
should i merge this commit, repush pr?

Copy link
Copy Markdown
Contributor

@amyeroberts amyeroberts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this so quickly!

@amyeroberts
Copy link
Copy Markdown
Contributor

@inkinworld Yes, once this PR has been merged into main then you can rebase on main or merge in the main branch into yours to include this fix and then push your branch again to trigger a re-run of the CI

@ydshieh ydshieh merged commit bb3bd44 into main Dec 22, 2023
@ydshieh ydshieh deleted the fix_support_list_test_not_run branch December 22, 2023 11:56
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.

3 participants