[Mlir-commits] [llvm] [mlir] [mlir python] Add nanobind support for standalone dialects. (PR #117922)

[Peter Hawkins]

hawkinsp wrote:

> Does nanobind have to be a required dependency?

Yes, that's correct. If it were just this PR, it would be reasonable for it to be optional. We'd certainly want to *test* it in CI, although it would be fine if a user didn't have it and didn't want it.

However, note https://github.com/llvm/llvm-project/pull/118583, which migrates the Python core code to nanobind, at which point it would certainly be mandatory. Indeed at that point the question is "can pybind11 be optional"?

Is there a way to specify dependencies that should be present for CI tests but are not mandatory for users?

https://github.com/llvm/llvm-project/pull/117922