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

Bump IREE to latest commit on main. #722

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

ScottTodd
Copy link
Member

@ScottTodd ScottTodd commented Dec 20, 2024

We're approaching our 3.1.0 release across all IREE packages, so bumping to build/test closer to HEAD.

PR #721 updates to the latest nightly, but we can also update to the latest commit, which includes multiple changes to https://github.com/iree-org/iree/tree/main/runtime/src/iree/hal/drivers/hip that we should test soon.

Commit range relative to #721: iree-org/iree@iree-3.1.0rc20241220...0a0483e

@ScottTodd
Copy link
Member Author

From these logs: https://github.com/nod-ai/shark-ai/actions/runs/12439009356/job/34732288777?pr=722#step:7:253, these errors:

ERROR tests/apps/sd/e2e_test.py::test_sd_server - RuntimeError: Timeout waiting for server start
ERROR tests/apps/sd/e2e_test.py::test_sd_server_bs4_dense - RuntimeError: Timeout waiting for server start
ERROR tests/apps/sd/e2e_test.py::test_sd_server_bs8_percall - RuntimeError: Timeout waiting for server start
ERROR tests/apps/sd/e2e_test.py::test_sd_server_bs4_dense_fpd2 - RuntimeError: Timeout waiting for server start
ERROR tests/apps/sd/e2e_test.py::test_sd_server_bs8_dense_fpd8 - RuntimeError: Timeout waiting for server start

seem to be new regressions, but I would expect more helpful error output, ideally from a unit test.

@ScottTodd
Copy link
Member Author

FYI @AWoloszyn @MaheshRavishankar . The only new IREE commits here are from both of you. Maybe the test failures make more sense to one of you 🤷

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