Commit c9893d7
committed
[MLIR][test] introduce has_py_module_ml_dtypes
Some python MLIR tests require the `ml_dtypes` module which doesn't
exist on Fedora for example. In order to avoid having to exclude a test
we would like to mark certain tests that require `ml_dtypes` with:
# REQUIRES: has_py_module_ml_dtypes
We noticed that `mlir/python/requirements.txt` lists `ml_dtypes` as a
requirement but when looking at the code in `mlir/python`, the
only `import` is guarded:
```python
try:
import ml_dtypes
except ModuleNotFoundError:
# The third-party ml_dtypes provides some optional low precision data-types for NumPy.
ml_dtypes = None
```
This makes `ml_dtypes` an optional dependency. Due to that reason I was
thinking if using a `pyproject.toml` could be a better fit to mark
dependencies as optional. But that's part of a different PR.1 parent 171d3ed commit c9893d7
File tree
4 files changed
+13
-1
lines changed- mlir/test
- python
4 files changed
+13
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
85 | 85 | | |
86 | 86 | | |
87 | 87 | | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
88 | 97 | | |
89 | 98 | | |
90 | 99 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
314 | 314 | | |
315 | 315 | | |
316 | 316 | | |
| 317 | + | |
| 318 | + | |
317 | 319 | | |
318 | 320 | | |
319 | 321 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
| 12 | + | |
12 | 13 | | |
13 | 14 | | |
14 | 15 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | | - | |
| 2 | + | |
3 | 3 | | |
4 | 4 | | |
5 | 5 | | |
| |||
0 commit comments