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

Update onnxruntime-extension to remove OpenCV #891

Merged
merged 7 commits into from
Sep 26, 2024
Merged

Conversation

skyline75489
Copy link
Contributor

@skyline75489 skyline75489 commented Sep 13, 2024

Remove OpenCV to reduce the binary size and the overall compilation time.

The following data is from the GHA pipeline.

Build Time Before After
win-x64 9m 8s 5m 47s
linux-x64 3m 30s 1m 37s

Uncompressed CAPI Binary:

Binary Size Before After
win-x64 2,276 KB 1,321 KB
linux-x64 4,836 KB 3,124 KB

@natke
Copy link
Contributor

natke commented Sep 19, 2024

Pending this microsoft/onnxruntime-extensions#808

@skyline75489 skyline75489 marked this pull request as ready for review September 26, 2024 04:46
@skyline75489 skyline75489 changed the title Update onnxruntime-extension Update onnxruntime-extension to remove OpenCV Sep 26, 2024
@skyline75489 skyline75489 merged commit 1e4d289 into main Sep 26, 2024
9 of 13 checks passed
@skyline75489 skyline75489 deleted the jialli/ortx-rm-cv2 branch September 26, 2024 04:50
skyline75489 added a commit to skyline75489/onnxruntime-genai that referenced this pull request Sep 26, 2024
Remove OpenCV to reduce the binary size and the overall compilation
time.

The following data is from the GHA pipeline.

| Build Time | Before | After   |
|------------|--------|---------|
| win-x64    | 9m 8s  | 5m 47s  |
| linux-x64  | 3m 30s | 1m 37s  |

Uncompressed CAPI Binary:

| Binary Size | Before   | After    |
|-------------|----------|----------|
| win-x64     | 2,276 KB | 1,321 KB |
| linux-x64   | 4,836 KB | 3,124 KB |
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