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 woq example and update document for v1.19.0 #2097

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

xin3he
Copy link
Contributor

@xin3he xin3he commented Dec 23, 2024

Type of Change

feature or bug fix or documentation or validation or others
API changed or not

Description

detail description

Expected Behavior & Potential Risk

the expected behavior that triggered by this PR

How has this PR been tested?

how to reproduce the test (including hardware information)

Dependency Change?

any library dependency introduced or removed

Copy link
Contributor

@thuang6 thuang6 left a comment

Choose a reason for hiding this comment

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

LGTM

@XuehaoSun XuehaoSun added this to the 3.2 milestone Dec 23, 2024
@xin3he
Copy link
Contributor Author

xin3he commented Dec 24, 2024

image
it's a fix for accuracy, bf16 cannot get correct accuracy as below, no idea but bf16 is not necessary so remove it.
inc_gaudi_test #150 [Jenkins]

@xin3he
Copy link
Contributor Author

xin3he commented Dec 24, 2024

Signed-off-by: Xin He <[email protected]>
@xin3he
Copy link
Contributor Author

xin3he commented Dec 24, 2024

@XuehaoSun please help trigger the test of opt_125m_woq_autoround_int4_hpu, not opt_125m_woq_autoround_int4, after removing environment variables and using requirement-autoround-hpu.txt in validation codes.

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.

5 participants