[tuner] Add CI for all tuning PR #88

RattataKing · 2024-08-01T20:20:52Z

Add pre-commit.yml and pytest.yml
Remove test_walk_mlir_op() in test_tune.py, which has dependencies on iree compiler, and its test data (template input mlir) is hard to maintain
Remove test-data contents

.github/workflows/pre-commit.yml

.github/workflows/pytest.yml

.github/workflows/pre-commit.yml

kuhar

Looks good but let's wait for @ScottTodd to approve

.github/workflows/pytest.yml

.github/workflows/pre-commit.yml

.github/workflows/pytest.yml

kuhar

Could you paste a link to a successful run on of this workflow on your fork?
Ah no, I can see it actually ran here already but failed.

ScottTodd

Workflows LGTM after changing * to **. Probably want the tests to pass before merging this.

.github/workflows/pre-commit.yml

Co-authored-by: Jakub Kuderski <[email protected]>

Co-authored-by: Scott Todd <[email protected]>

kuhar

All tests passed, LGTM

RattataKing force-pushed the add_ci_remote branch 3 times, most recently from e70c329 to 93d1177 Compare August 1, 2024 22:09

RattataKing requested review from kuhar and ScottTodd August 1, 2024 22:09

RattataKing changed the title ~~[tuner] Enable pre-commit test for all branches and PR~~ [tuner] Enable pre-commit test for all PR Aug 1, 2024

RattataKing changed the title ~~[tuner] Enable pre-commit test for all PR~~ [tuner] Run pre-commit test for all PR Aug 1, 2024

ScottTodd reviewed Aug 2, 2024

View reviewed changes

.github/workflows/pre-commit.yml Outdated Show resolved Hide resolved

.github/workflows/pre-commit.yml Outdated Show resolved Hide resolved

.github/workflows/pre-commit.yml Show resolved Hide resolved

RattataKing requested a review from ScottTodd August 2, 2024 20:54

RattataKing force-pushed the add_ci_remote branch from 4c3169a to dde6af5 Compare August 2, 2024 22:07

RattataKing changed the title ~~[tuner] Run pre-commit test for all PR~~ [tuner] Add CI for all tuning PR Aug 2, 2024

RattataKing force-pushed the add_ci_remote branch from dde6af5 to b95019d Compare August 2, 2024 22:11

kuhar reviewed Aug 2, 2024

View reviewed changes

.github/workflows/pytest.yml Outdated Show resolved Hide resolved

.github/workflows/pytest.yml Outdated Show resolved Hide resolved

.github/workflows/pre-commit.yml Outdated Show resolved Hide resolved

RattataKing requested a review from kuhar August 2, 2024 22:19

kuhar approved these changes Aug 2, 2024

View reviewed changes

kuhar reviewed Aug 2, 2024

View reviewed changes

.github/workflows/pytest.yml Outdated Show resolved Hide resolved

ScottTodd reviewed Aug 5, 2024

View reviewed changes

.github/workflows/pre-commit.yml Outdated Show resolved Hide resolved

.github/workflows/pytest.yml Outdated Show resolved Hide resolved

.github/workflows/pytest.yml Outdated Show resolved Hide resolved

.github/workflows/pytest.yml Outdated Show resolved Hide resolved

kuhar self-requested a review August 6, 2024 16:04

RattataKing requested a review from ScottTodd August 6, 2024 16:04

kuhar reviewed Aug 6, 2024

View reviewed changes

ScottTodd reviewed Aug 6, 2024

View reviewed changes

.github/workflows/pre-commit.yml Outdated Show resolved Hide resolved

RattataKing added 9 commits August 7, 2024 11:18

Add action

fd365d8

Replace explicit commands by action

e4fafa1

Update

742b983

Update

ab7f473

Update

e8d9c56

Update

f239466

Update

db23d4b

Update

e1cf706

Update

1418bf6

RattataKing and others added 16 commits August 7, 2024 11:18

Update

6381eb3

Update

3913994

Update

bccbeb9

Update

ce24c20

Update

9512792

Add ci tests

ea56be4

Update .github/workflows/pytest.yml

b76882f

Co-authored-by: Jakub Kuderski <[email protected]>

Update .github/workflows/pre-commit.yml

fb51470

Co-authored-by: Jakub Kuderski <[email protected]>

Update .github/workflows/pytest.yml

b73ebcd

Co-authored-by: Jakub Kuderski <[email protected]>

Update PR checking path

f01bdba

Co-authored-by: Scott Todd <[email protected]>

Update PR checking path

eec008a

Co-authored-by: Scott Todd <[email protected]>

Update .yml job name

339088a

Co-authored-by: Scott Todd <[email protected]>

Drop venv in pytest.yml

c16fbf7

Change to use nightly ver of ireec

db68c98

Add iree runtime

7653d74

Remove iree parsing test

6831547

RattataKing force-pushed the add_ci_remote branch from 51611c1 to ca236b2 Compare August 7, 2024 16:20

Change CI work path

824ca57

RattataKing force-pushed the add_ci_remote branch from ca236b2 to 824ca57 Compare August 7, 2024 16:21

kuhar approved these changes Aug 7, 2024

View reviewed changes

kuhar merged commit 4b8aa30 into main Aug 7, 2024
2 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[tuner] Add CI for all tuning PR #88

[tuner] Add CI for all tuning PR #88

RattataKing commented Aug 1, 2024 •

edited

Loading

kuhar left a comment

kuhar left a comment •

edited

Loading

ScottTodd left a comment

kuhar left a comment •

edited

Loading

[tuner] Add CI for all tuning PR #88

[tuner] Add CI for all tuning PR #88

Conversation

RattataKing commented Aug 1, 2024 • edited Loading

kuhar left a comment

Choose a reason for hiding this comment

kuhar left a comment • edited Loading

Choose a reason for hiding this comment

ScottTodd left a comment

Choose a reason for hiding this comment

kuhar left a comment • edited Loading

Choose a reason for hiding this comment

RattataKing commented Aug 1, 2024 •

edited

Loading

kuhar left a comment •

edited

Loading

kuhar left a comment •

edited

Loading