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

Test on ORT 1.17 and 1.18 #78

Closed
wants to merge 4 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 10 additions & 2 deletions .github/workflows/array-api.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,12 @@ jobs:
name: Array API test
timeout-minutes: 90
runs-on: ubuntu-latest
strategy:
fail-fast: false
matrix:
environment:
- ort117py312
- ort118py312
steps:
- name: Checkout branch
uses: actions/checkout@v4
Expand All @@ -21,10 +27,12 @@ jobs:
submodules: recursive
- name: Set up pixi
uses: prefix-dev/setup-pixi@ba3bb36eb2066252b2363392b7739741bb777659
with:
environments: ${{ matrix.environment }}
- name: Install repository
run: pixi run postinstall
run: pixi run -e ${{ matrix.environment }} postinstall
- name: Run Array API tests
run: pixi run arrayapitests
run: pixi run -e ${{ matrix.environment }} arrayapitests
- name: Upload Array API tests report
uses: actions/upload-artifact@v4
with:
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,8 @@ jobs:
environment:
- py310
- py311
- py312
- np1x
- ort117py312
- ort118py312
steps:
- name: Checkout branch
uses: actions/checkout@v4
Expand Down
5 changes: 5 additions & 0 deletions docs/intros/modelconversion.rst
Original file line number Diff line number Diff line change
Expand Up @@ -110,3 +110,8 @@ We can use an ONNX backend like onnxruntime to run our model. Here we use onnxru

print(out)
# array([0, 0, 0, 1, 2, 2, 0, 1, 2, 1, 0, 2, 1, 1, 1, 2, 2, 0, 0, 1, 0, 1, 1, 0, 0, 2, 2, 2, 2, 2])

.. note::

ONNX backends may not always support the entire ONNX specification and can sometimes miss kernel implementations for specific data types.
Since onnxruntime is such a common backend, ndonnx ensures proactively that any model generated using :func:`ndonnx.build` is compatible with at least the two latest minor onnxruntime releases on conda-forge.
2 changes: 2 additions & 0 deletions ndonnx/_build.py
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,8 @@ def build(
-------
out: onnx.ModelProto
ONNX model

Note: the generated ONNX model should be runnable on the latest two minor versions of onnxruntime documented in an ndonnx release.
"""

def collect_vars(name: str, arr: _CoreArray | Array):
Expand Down
Loading