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

perf(api-rust): switch from image to opencv #21570

Merged
merged 1 commit into from
Dec 15, 2024
Merged

perf(api-rust): switch from image to opencv #21570

merged 1 commit into from
Dec 15, 2024

Conversation

hongbo-miao
Copy link
Owner

@hongbo-miao hongbo-miao commented Dec 15, 2024

For 1.2 MB image, before the change: 5606 ms:

2024-12-15T09:55:25.375172Z DEBUG request{method=POST uri=/classify-image version=HTTP/1.1}: tower_http::trace::on_request: started processing request
2024-12-15T09:55:30.981536Z DEBUG request{method=POST uri=/classify-image version=HTTP/1.1}: tower_http::trace::on_response: finished processing request latency=5606 ms status=200

After the change: 157 ms

2024-12-15T10:08:43.378690Z DEBUG request{method=POST uri=/classify-image version=HTTP/1.1}: tower_http::trace::on_request: started processing request
2024-12-15T10:08:43.536464Z DEBUG request{method=POST uri=/classify-image version=HTTP/1.1}: tower_http::trace::on_response: finished processing request latency=157 ms status=200

Copy link

CML (Local) Report

Params

Value
lr 0.001
train {'epochs': 2}

Copy link

Commit: dc34f0a
Data: default

Configuration Intent Classification Micro F1 Entity Recognition Micro F1 Response Selection Micro F1 Story Recognition Micro F1
config.yml 1.0000 (0.00) 1.0000 (0.00) no data 1.0000 (0.00)

None of the test stories failed - all good!

@hongbo-miao hongbo-miao merged commit 8633870 into main Dec 15, 2024
158 of 159 checks passed
@hongbo-miao hongbo-miao deleted the opencv branch December 15, 2024 12:07
Copy link

🎉 This PR is included in version 1.129.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant