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

TOSA importer failed after 08-17-2023 integrate #14775

Closed
hanhanW opened this issue Aug 22, 2023 · 5 comments · Fixed by #14810
Closed

TOSA importer failed after 08-17-2023 integrate #14775

hanhanW opened this issue Aug 22, 2023 · 5 comments · Fixed by #14810
Assignees

Comments

@hanhanW
Copy link
Contributor

hanhanW commented Aug 22, 2023

See #14725 and https://github.com/openxla/iree/actions/runs/5898571518/job/16000106665

Failed Tests (11):
  TENSORFLOW_TESTS :: iree_tfl_tests/east_text_detector.run
  TENSORFLOW_TESTS :: iree_tfl_tests/gpt2.run
  TENSORFLOW_TESTS :: iree_tfl_tests/llvmcpu_mobilenet_v1.run
  TENSORFLOW_TESTS :: iree_tfl_tests/llvmcpu_mobilenet_v3-large_uint8.run
  TENSORFLOW_TESTS :: iree_tfl_tests/llvmcpu_resnet_50_int8.run
  TENSORFLOW_TESTS :: iree_tfl_tests/mnasnet.run
  TENSORFLOW_TESTS :: iree_tfl_tests/mobilenet_v3.run
  TENSORFLOW_TESTS :: iree_tfl_tests/person_detect.run
  TENSORFLOW_TESTS :: iree_tfl_tests/vmvx_mobilebert_tf2_quant.run
  TENSORFLOW_TESTS :: iree_tfl_tests/vmvx_person_detect.run
  TENSORFLOW_TESTS :: iree_tfl_tests/vulkan_mobilenet_v1.run
@hcindyl
Copy link
Contributor

hcindyl commented Aug 23, 2023

FYI, tf-nightly 2.15.0.dev20230817 picked up the required fix. Not sure if anyone has raised a PR to re-enable the tflite tests.

@hanhanW
Copy link
Contributor Author

hanhanW commented Aug 23, 2023

@jpienaar I've seen you bumped tf-nightly couple times, could you help bump tf-nightly version and re-enable the tests and benchmarks?

@jpienaar
Copy link
Member

@rsuderman is already working on this AFAIK

@hanhanW
Copy link
Contributor Author

hanhanW commented Aug 24, 2023

Hey @rsuderman I'd like check in what's the status about this? I have a couple PRs that needs to test if there are regression. It would be awesome if this is prioritized.

@hanhanW
Copy link
Contributor Author

hanhanW commented Aug 24, 2023

I'm trying to bump it. It looks like we need to bump python to 3.9 as well.

image

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 a pull request may close this issue.

4 participants