-
Notifications
You must be signed in to change notification settings - Fork 289
Enhance benchmark #1531
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
Enhance benchmark #1531
Conversation
Signed-off-by: letonghan <letong.han@intel.com>
Dependency Review✅ No vulnerabilities or license issues found.Scanned Files |
for more information, see https://pre-commit.ci
…ples into enhance_benchmark
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some questions and minor edits.
Thank for your comments! Other minor spelling issues will be fixed soon. |
Signed-off-by: letonghan <letong.han@intel.com>
Signed-off-by: letonghan <letong.han@intel.com>
Signed-off-by: letonghan <letong.han@intel.com>
Signed-off-by: letonghan <letong.han@intel.com>
for more information, see https://pre-commit.ci
The poisson_load_shape.py file was located using relative path, refined to absolute path now so that outer files can call this function correctly. Found this issue in PR: opea-project/GenAIExamples#1531 Signed-off-by: letonghan <letong.han@intel.com>
The benchmark using |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* Fix relative path issue for poisson. The poisson_load_shape.py file was located using relative path, refined to absolute path now so that outer files can call this function correctly. Found this issue in PR: opea-project/GenAIExamples#1531 Signed-off-by: letonghan <letong.han@intel.com>
Description
Enhance the benchmark process:
benchmark.py
andbenchmark_chatqna.yaml
bench_target
process partIssues
n/a
Type of change
List the type of change like below. Please delete options that are not relevant.
Dependencies
None
Tests
Local tested on k8s