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

Skip the verified config #1775

Merged
merged 4 commits into from
May 8, 2024
Merged

Skip the verified config #1775

merged 4 commits into from
May 8, 2024

Conversation

yiliu30
Copy link
Contributor

@yiliu30 yiliu30 commented May 8, 2024

Type of Change

API changed or not: None

How has this PR been tested?

Pre-CI
Test

 pytest -sv ./test/3x/common/test_common.py  -k test_config_loader_skip_verified_config

Dependency Change?

None

Signed-off-by: yiliu30 <[email protected]>
Copy link

github-actions bot commented May 8, 2024

⛈️ Required checks status: Has failure 🔴

Warning
If you do not have the access to re-run the Probot, please contact XuehaoSun for help. If you push a new commit, all of the workflow will be re-triggered.

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/base_config.py, neural_compressor/common/base_tuning.py, neural_compressor/common/tuning_param.py, neural_compressor/torch/quantization/algorithm_entry.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/base_config.py, neural_compressor/common/base_tuning.py, neural_compressor/common/tuning_param.py, neural_compressor/torch/quantization/algorithm_entry.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/base_config.py, neural_compressor/common/base_tuning.py, neural_compressor/common/tuning_param.py.

🔴 Unit Tests 3x-PyTorch workflow
Check ID Status Error details
UT-3x-Torch failure
UT-3x-Torch (Coverage Compare CollectDatafiles) no_status
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) failure download

These checks are required after the changes to neural_compressor/common/base_config.py, neural_compressor/common/base_tuning.py, neural_compressor/common/tuning_param.py, neural_compressor/torch/quantization/algorithm_entry.py, test/3x/torch/quantization/weight_only/test_mixed_algos.py, test/3x/common/test_common.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/base_config.py, neural_compressor/common/base_tuning.py, neural_compressor/common/tuning_param.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 changed the title Skip verified config Skip the verified config May 8, 2024
@@ -10,11 +10,8 @@


def run_fn(model):
# GPTQ uses ValueError to reduce computation when collecting input data of the first block
# It's special for UTs, no need to add this wrapper in examples.
with pytest.raises(ValueError):
Copy link
Contributor Author

@yiliu30 yiliu30 May 8, 2024

Choose a reason for hiding this comment

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

@XuehaoSun RTN won't raise ValueError

@XuehaoSun XuehaoSun merged commit 7b8aec0 into master May 8, 2024
33 of 36 checks passed
@XuehaoSun XuehaoSun deleted the skip_config branch May 8, 2024 06:34
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