Skip to content

Build TF dependencies in Kokoro CI#269

Closed
GMNGeoffrey wants to merge 1 commit intoiree-org:masterfrom
GMNGeoffrey:ci-tags
Closed

Build TF dependencies in Kokoro CI#269
GMNGeoffrey wants to merge 1 commit intoiree-org:masterfrom
GMNGeoffrey:ci-tags

Conversation

@GMNGeoffrey
Copy link
Copy Markdown
Contributor

@GMNGeoffrey GMNGeoffrey commented Dec 26, 2019

Everything still builds without these tags. Let's see how well RBE handles building tensorflow

@GMNGeoffrey GMNGeoffrey changed the title Remove unnecessary nokokoro exclusions Build TF dependencies in Kokoro CI Dec 26, 2019
@GMNGeoffrey GMNGeoffrey marked this pull request as ready for review December 26, 2019 19:17
iree-copybara-bot pushed a commit that referenced this pull request Dec 26, 2019
The aliasing is fine but makes things really difficult to read. A (to be implemented) dead block argument removal pass should take care of this in cases where it does naturally occur.

Note that the new test in here demonstrates the current bad behavior of the remapping process (`["1->0", "0->1"]`).

FUTURE_COPYBARA_INTEGRATE_REVIEW=#269 from GMNGeoffrey:ci-tags 09ff710
PiperOrigin-RevId: 287233202
iree-copybara-bot pushed a commit that referenced this pull request Dec 26, 2019
The aliasing is fine but makes things really difficult to read. A (to be implemented) dead block argument removal pass should take care of this in cases where it does naturally occur.

Note that the new test in here demonstrates the current bad behavior of the remapping process (`["1->0", "0->1"]`).

FUTURE_COPYBARA_INTEGRATE_REVIEW=#269 from GMNGeoffrey:ci-tags 09ff710
PiperOrigin-RevId: 287233202
@GMNGeoffrey GMNGeoffrey deleted the ci-tags branch December 31, 2019 00:25
copybara-service Bot pushed a commit that referenced this pull request Apr 3, 2020
Prior to the changes all `_` in the path got replaced with `/`:

Before:
```
  Test  #30: iree/test/e2e/linalg/path:pw_add.mlir.test
  Test  #31: iree/test/e2e/linalg/path:pw_add_mul.mlir.test
  Test  #32: iree/test/e2e/linalg/path:pw_add_multiwg.mlir.test
  Test  #33: iree/test/e2e/linalg/path:reduce.mlir.test

  Test #268: iree/samples/custom/modules:custom_modules_test
  Test #269: iree/samples/custom/modules/dialect/test:conversion.mlir.test
  Test #270: iree/samples/custom/modules/dialect/test:custom_ops.mlir.test
  Test #271: iree/samples/simple/embedding:simple_embedding_test
```

After:
```
  Test  #30: iree/test/e2e/linalg_path:pw_add.mlir.test
  Test  #31: iree/test/e2e/linalg_path:pw_add_mul.mlir.test
  Test  #32: iree/test/e2e/linalg_path:pw_add_multiwg.mlir.test
  Test  #33: iree/test/e2e/linalg_path:reduce.mlir.test

  Test #268: iree/samples/custom_modules:custom_modules_test
  Test #269: iree/samples/custom_modules/dialect/test:conversion.mlir.test
  Test #270: iree/samples/custom_modules/dialect/test:custom_ops.mlir.test
  Test #271: iree/samples/simple_embedding:simple_embedding_test
```

Closes #1360

COPYBARA_INTEGRATE_REVIEW=#1360 from iml130:cmake-test-labels 34dc25d
PiperOrigin-RevId: 304524688
stellaraccident pushed a commit that referenced this pull request Sep 24, 2023
Add more pure user flow if no dev is needed. The autogeneration of pip
packages not yet automatic, but this flow should work.
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.

3 participants