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

[Not merge] Tests for quality #1805

Closed
wants to merge 8 commits into from
Closed

[Not merge] Tests for quality #1805

wants to merge 8 commits into from

Conversation

yiliu30
Copy link
Collaborator

@yiliu30 yiliu30 commented May 20, 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

Signed-off-by: yiliu30 <yi4.liu@intel.com>
Signed-off-by: yiliu30 <yi4.liu@intel.com>
Copy link

github-actions bot commented May 20, 2024

⚡ Required checks status: All passing 🟢

Groups summary

🟢 Code Scan Tests workflow
Check ID Status Error details
Code-Scan success
Code-Scan (Bandit Code Scan Bandit) success
Code-Scan (DocStyle Code Scan DocStyle) success
Code-Scan (Pylint Code Scan Pylint) success

These checks are required after the changes to neural_compressor/common/utils/constants.py.

🟢 Model Tests 3x workflow
Check ID Status Error details
Model-Test-3x success
Model-Test-3x (Generate Report GenerateReport) success
Model-Test-3x (Run PyTorch Model opt_125m_woq_gptq_int4) success
Model-Test-3x (Run PyTorch Model opt_125m_woq_gptq_int4_dq_bnb) success
Model-Test-3x (Run PyTorch Model opt_125m_woq_gptq_int4_dq_ggml) success

These checks are required after the changes to neural_compressor/common/utils/constants.py.

🟢 Unit Tests 3x-TensorFlow workflow
Check ID Status Error details
UT-3x-TensorFlow success
UT-3x-TensorFlow (Coverage Compare CollectDatafiles) success
UT-3x-TensorFlow (Unit Test 3x TensorFlow Unit Test 3x TensorFlow) success
UT-3x-TensorFlow (Unit Test 3x TensorFlow baseline Unit Test 3x TensorFlow baseline) success

These checks are required after the changes to neural_compressor/common/utils/constants.py.

🟢 Unit Tests 3x-PyTorch workflow
Check ID Status Error details
UT-3x-Torch success
UT-3x-Torch (Coverage Compare CollectDatafiles) success
UT-3x-Torch (Unit Test 3x Torch Unit Test 3x Torch) success
UT-3x-Torch (Unit Test 3x Torch baseline Unit Test 3x Torch baseline) success

These checks are required after the changes to neural_compressor/common/utils/constants.py.

🟢 Unit Tests 3x-ONNXRT workflow
Check ID Status Error details
UT-3x-ONNXRT success
UT-3x-ONNXRT (Coverage Compare CollectDatafiles) success
UT-3x-ONNXRT (Unit Test 3x ONNXRT Unit Test 3x ONNXRT) success
UT-3x-ONNXRT (Unit Test 3x ONNXRT baseline Unit Test 3x ONNXRT baseline) success

These checks are required after the changes to neural_compressor/common/utils/constants.py.


Thank you for your contribution! 💜

Note
This comment is automatically generated and will be updates every 180 seconds within the next 6 hours. If you have any other questions, contact chensuyue or XuehaoSun for help.

@yiliu30 yiliu30 self-assigned this May 20, 2024
@chensuyue
Copy link
Contributor

onnxrt ut failed due to onnxruntime 1.18.0 release, @mengniwang95 is working on fix that.

@yiliu30
Copy link
Collaborator Author

yiliu30 commented Jun 19, 2024

Torch

PR lines coverage: 5044/5829 (86.533%)
PR branches coverage: 1265/1728 (73.206%) # due to it include `amp` and `habana_fp8` folders which will be deprecated soon 

TF

PR lines coverage: 10799/11989 (90.074%)
PR branches coverage: 3354/4377 (76.628%) # due to it handle too many corner cases for OOB

ONC

The enhancements are submitted to ONC directly. https://github.com/onnx/neural-compressor

@yiliu30 yiliu30 closed this Jun 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants