Skip to content
This repository was archived by the owner on Jul 18, 2024. It is now read-only.

[DIEN]Update DIEN readme #10

Merged
merged 5 commits into from
Sep 2, 2022
Merged

[DIEN]Update DIEN readme #10

merged 5 commits into from
Sep 2, 2022

Conversation

xuechendi
Copy link
Contributor

No description provided.

@xuechendi xuechendi force-pushed the dien branch 2 times, most recently from a855182 to d301a53 Compare August 29, 2022 22:58
@xuechendi xuechendi added the jenkins test Trigger for Jenkins tests label Aug 29, 2022
@xuechendi xuechendi requested a review from Jian-Zhang August 30, 2022 02:19
@@ -13,18 +13,33 @@ export path_to_e2eaiok=`pwd`/e2eAIOK
mkdir -p ${path_to_e2eaiok_dataset}
```

## Install
## Install End to end AIOK Kit for DIEN
Copy link
Contributor

Choose a reason for hiding this comment

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

should use full name: Intel® End-to-End AI Optimization Kit.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Jian-Zhang I fixed naming issue

Signed-off-by: Xue, Chendi <[email protected]>
@Jian-Zhang Jian-Zhang changed the title [DIEN]Update DIEN readme removing sensitive info [DIEN]Update DIEN readme Aug 31, 2022
@Jian-Zhang
Copy link
Contributor

Test failed, looks it trying to reload the model, is this expected behavior? A single doc update shouldn't lead to it, may be an issue of CI/CD system itself? @zigzagcai @xuechendi.

@zigzagcai
Copy link
Contributor

zigzagcai commented Aug 31, 2022

Test failed, looks it trying to reload the model, is this expected behavior? A single doc update shouldn't lead to it, may be an issue of CI/CD system itself? @zigzagcai @xuechendi.

I checked jenkins log. It was because some files were renamed from aidk to e2eaiok in this repo, but some were not changed, resulting in file not found error. I have fixed this issue in PR#7.
This CICD build was before my PR merged. It will be okay now.

image

@xuechendi
Copy link
Contributor Author

@Jian-Zhang , I re-triggered test, and now UT are passed.

@Jian-Zhang Jian-Zhang merged commit a67a7a5 into intel:main Sep 2, 2022
tianyil1 added a commit to tianyil1/e2eAIOK that referenced this pull request Dec 5, 2023
* Refine the main README for deltatuner

* Refine the deltatuner README

* remove the docker folder

* refine the deltatuner code example
xuechendi pushed a commit that referenced this pull request Dec 7, 2023
* delete

* [V1.2] Refine the Deltatuner README (#10)

* Refine the main README for deltatuner

* Refine the deltatuner README

* remove the docker folder

* refine the deltatuner code example

* refine example readme

* add a new package name

* refine format and fix typo

* update api

---------

Co-authored-by: Tianyi Liu <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
jenkins test Trigger for Jenkins tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants