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

Fix torch-mlir-import-onnx entry point #3965

Merged
merged 1 commit into from
Jan 17, 2025

Conversation

marbre
Copy link
Member

@marbre marbre commented Jan 17, 2025

Due to the wrong entry point, calling torch-mlir-import-onnx currently fails with

$ torch-mlir-import-onnx
Traceback (most recent call last):
  File "venv-torch/bin/torch-mlir-import-onnx", line 5, in <module>
    from torch_mlir.tools.import_onnx import _cli_main
ImportError: cannot import name '_cli_main' from 'torch_mlir.tools.import_onnx' (unknown location)

@marbre marbre requested review from ScottTodd and zjgarvey January 17, 2025 13:47
@marbre marbre force-pushed the torch-mlir-import-onnx-entry-point branch from bf5e03e to c9fa5bc Compare January 17, 2025 18:09
Due to the wrong entry point, calling `torch-mlir-import-onnx` currently
fails with
```
$ torch-mlir-import-onnx
Traceback (most recent call last):
  File "venv-torch/bin/torch-mlir-import-onnx", line 5, in <module>
    from torch_mlir.tools.import_onnx import _cli_main
ImportError: cannot import name '_cli_main' from 'torch_mlir.tools.import_onnx' (unknown location)
```
@marbre marbre force-pushed the torch-mlir-import-onnx-entry-point branch from c9fa5bc to db58183 Compare January 17, 2025 19:09
@marbre marbre enabled auto-merge (squash) January 17, 2025 19:09
@marbre marbre merged commit b17cf23 into llvm:main Jan 17, 2025
3 checks passed
@marbre marbre deleted the torch-mlir-import-onnx-entry-point branch January 20, 2025 08:24
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