Skip to content
This repository has been archived by the owner on Oct 25, 2024. It is now read-only.

Commit

Permalink
update owner and CI
Browse files Browse the repository at this point in the history
Signed-off-by: Wenxin Zhang <wenxin.zhang@intel.com>
  • Loading branch information
VincyZhang committed Aug 11, 2023
1 parent 4038579 commit f34b97f
Show file tree
Hide file tree
Showing 4 changed files with 41 additions and 19 deletions.
12 changes: 11 additions & 1 deletion .github/workflows/script/formatScan/nlp_dict.txt
Original file line number Diff line number Diff line change
Expand Up @@ -2262,4 +2262,14 @@ Cpplint
DCO
cloc
jira
servicedesk
servicedesk
Bas
GHA
clangformat
dco
gtest
inteltf
jenk
mr
sign off
formatScan
32 changes: 22 additions & 10 deletions docs/CI_introduction.md
Original file line number Diff line number Diff line change
@@ -1,17 +1,29 @@
# CI Introduction
Intel Extension for Transformers use Github Action (GHA) for CI test, if you are new to GHA, please refer to [GHA](https://docs.github.com/en/actions)
All the test scripts will be maintained under [.github](../.github/workflows).


Generally We use [Azure Cloud](https://azure.microsoft.com/en-us/pricing/purchase-options/pay-as-you-go) to deploy CI on Es-v5, Bas-v2.

| Test Name | Test Pass Criteria | Checks |

| Test Name | Test Scope | Test Pass Criteria |
|-------------------------------|-----------------------------------------------|---------------|
| Format Scan | Pylint/Cpplint/bandit/cloc Pass | 4 |
| Spell Check | Check Pass | 1 |
| Copyright Check | Add License on top of code/scripts | 1 |
| DCO | Use git commit -s to signoff | 1 |
| Unit Test | a. No failure, no core dump, no segmentation fault b. No coverage drop| 5 |
| Kernel Benchmark | No failure, No performance regression | 1 |
| Model Test | a. Fp32/Int8 inference no regression b. API no fucntionality success | 5 |
| CPP Graph Test | No failure, No performance regression | 1 |
| Format Scan | Pylint/Cpplint/bandit/cloc/clangformat Pass | PASS |
| Spell Check | Spelling Check | PASS |
| Copyright Check | Copyright Check | PASS |
| [DCO](https://github.com/apps/dco/) | Use git commit -s to sign off | PASS |
| Unit Test | pytest + coverage + gtest | PASS(No failure, No core dump, No segmentation fault, No coverage drop) |
| Kernel Benchmark | Benchmark [Details](../intel_extension_for_transformers/backends/neural_engine/test/kernels/benchmark) | PASS(No performance regression) |
| Model Test | Pytorch + Tensorflow + Neural Engine + IPEX + CPP Graph | PASS(FP32/INT8 No performance regression) |

# FAQ
1. How to add examples

(1) Still need to pass detected CI tests

(2) Use [Extension Test](https://inteltf-jenk.sh.intel.com/view/nlp-toolkit-validation/job/nlp-toolkit-validation-top-mr-extension/) to cover new examples. If you do not have permission for extension test, please contact [Maintainers](inc.maintainers@intel.com)


2. How to test specific component version

(1) Use [Extension Test](https://inteltf-jenk.sh.intel.com/view/nlp-toolkit-validation/job/nlp-toolkit-validation-top-mr-extension/) to specify the version. If you do not have permission for extension test, please contact [Maintainers](inc.maintainers@intel.com)

14 changes: 7 additions & 7 deletions docs/component_owner.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,10 @@ Contributions to Intel Extension for Transformers specific component must get ap

| Component | Owner | Backup |
|--------------------|-----------------|-----------------|
| Kernel | Hengyu | Luo yu |
| Engine | Dong bo | Zhenwei |
| Pytorch | Penghui | He xin |
| Tensorflow | Lv liang | Penghui |
| Neuralchat | Lv liang | Kaokao |
| Workflow | Lv liang | Haihao |
| Tests | Wenxin | Xuehao |
| Kernel | airMeng | luoyu-intel |
| Engine | a32543254 | zhenwei-intel |
| Pytorch | PenghuiCheng | xin3he |
| Tensorflow | lvliang-intel | PenghuiCheng |
| Neuralchat | lvliang-intel | lkk12014402 |
| Workflow | lvliang-intel | kevinintel |
| Tests | VincyZhang | XuehaoSun |
2 changes: 1 addition & 1 deletion docs/contributions.md
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ See [PR template](../.github/PULL_REQUEST_TEMPLATE/pull_request_template.md)

(3) ALL conversations solved

(4) Third-party dependency license compliable
(4) Third-party dependency license Compatible


## CI Introduction
Expand Down

0 comments on commit f34b97f

Please sign in to comment.