Skip to content

fea(ci): Updated the fixtures based on changes in PR#2246#2262

Merged
regisss merged 1 commit into
huggingface:mainfrom
imangohari1:ig/ci-fixture-fix-09162025
Sep 17, 2025
Merged

fea(ci): Updated the fixtures based on changes in PR#2246#2262
regisss merged 1 commit into
huggingface:mainfrom
imangohari1:ig/ci-fixture-fix-09162025

Conversation

@imangohari1
Copy link
Copy Markdown
Contributor

@imangohari1 imangohari1 commented Sep 16, 2025

What does this PR do?

This PR is a follow up to the changes made in tests/test_text_generation_example.py in #2246
All the bf16 and fp8 the changes in fixture names are tests and they are now

Update 09/17/2025: I have tested the new tests with use_flex_attention=True, and they are all passing on g3 1.22-740.

> [!NOTE]

The throughput for tests using use_flex_attention (i.e. fixtures ending with -True) would need a revisit at a later time. For now they are set to be the same as use_flex_attention = False

Fixes # (issue)
#2259

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you make sure to update the documentation with your changes?
  • Did you write any new necessary tests?

@HuggingFaceDocBuilderDev
Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Copy link
Copy Markdown
Collaborator

@regisss regisss left a comment

Choose a reason for hiding this comment

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

LGTM

@regisss regisss merged commit 63be7d7 into huggingface:main Sep 17, 2025
2 of 5 checks passed
This was referenced Sep 17, 2025
gplutop7 pushed a commit to HabanaAI/optimum-habana-fork that referenced this pull request Oct 15, 2025
gplutop7 pushed a commit to HabanaAI/optimum-habana-fork that referenced this pull request Nov 6, 2025
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.

4 participants