Skip to content

Make ONNX support optional#13

Open
theHamsta wants to merge 2 commits intorustnn:mainfrom
theHamsta:onnx-optional
Open

Make ONNX support optional#13
theHamsta wants to merge 2 commits intorustnn:mainfrom
theHamsta:onnx-optional

Conversation

@theHamsta
Copy link
Copy Markdown

@theHamsta theHamsta commented May 4, 2026

Name for feature: "onnx" or "onnx-support"
Make onnx support default?

@theHamsta theHamsta changed the title Onnx optional Make ONNX support optional May 4, 2026
@theHamsta theHamsta force-pushed the onnx-optional branch 3 times, most recently from 5262e31 to af13439 Compare May 4, 2026 09:10
Comment thread .github/workflows/ci.yml
jobs:
test:
name: Test on ${{ matrix.os }}
name: Test (${{ matrix.os }}, minimal=${{ matrix.minimal }})
Copy link
Copy Markdown
Author

@theHamsta theHamsta May 4, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made separate tags to have on env with and one without protoc (needing protoc in CI env is a pitfall for someone trying to use webnn-graph)

@theHamsta theHamsta force-pushed the onnx-optional branch 11 times, most recently from db01059 to 5ff7bbf Compare May 4, 2026 09:50
@theHamsta theHamsta requested a review from tarekziade May 4, 2026 09:52
@theHamsta
Copy link
Copy Markdown
Author

Preparation for rustnn/rustnn#89

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.

1 participant