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

Enable one of the npu-xrt/e2e tests #1908

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

Conversation

hunhoffe
Copy link
Collaborator

@hunhoffe hunhoffe commented Nov 5, 2024

I came across the set of test/npu-xrt/e2e tests more or less by accident today. I am specifically interested in (eventually) creating an end-to-end notebook, and looking at these tests seemed like a decent place to start exploring. These tests are all currently disabled.

In this PR:

  • I ported one of the tests to be supported (compiled with peano), which now runs correctly in the CI as far as I can tell
  • I removed all unsupported tests and the compiler util file that seems pretty outdated. I don't think there was an easy path to updating all these things.

As a note for the future, some of these examples and the compiler utils showed some neat/interesting ways to interact with the compiler in Python, which may be a good idea to revisit in the future!

@hunhoffe hunhoffe changed the title [WIP] Enable (some) npu-xrt/e2e Tests [WIP] Enable one of the npu-xrt/e2e tests Nov 5, 2024
@hunhoffe hunhoffe changed the title [WIP] Enable one of the npu-xrt/e2e tests Enable one of the npu-xrt/e2e tests Nov 5, 2024
@hunhoffe hunhoffe marked this pull request as ready for review November 5, 2024 21:04
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.

1 participant