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

fix: fix open-benchmark template #1695

Merged
merged 1 commit into from
Mar 19, 2025
Merged

fix: fix open-benchmark template #1695

merged 1 commit into from
Mar 19, 2025

Conversation

SLR722
Copy link
Contributor

@SLR722 SLR722 commented Mar 19, 2025

What does this PR do?

open-benchmark templated is broken after the datasets api refactor due to 2 reasons

  • provider_id and provider_resource_id are no longer needed
  • the type in run.yaml will be resolved as dict

this PR is to fix the above 2 issues

Test

spin up a llama stack server successfully with llama stack run llama_stack/templates/open-benchmark/run.yaml

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label Mar 19, 2025
@SLR722 SLR722 changed the title [debug] fix open-benchmark template fix: [debug] fix open-benchmark template Mar 19, 2025
@SLR722 SLR722 changed the title fix: [debug] fix open-benchmark template fix: fix open-benchmark template Mar 19, 2025
@SLR722 SLR722 marked this pull request as ready for review March 19, 2025 18:24
@yanxi0830
Copy link
Contributor

Thanks!

@yanxi0830 yanxi0830 merged commit ab777ef into main Mar 19, 2025
15 of 16 checks passed
@yanxi0830 yanxi0830 deleted the fix_open_benchmark branch March 19, 2025 18:27
franciscojavierarceo pushed a commit to franciscojavierarceo/llama-stack that referenced this pull request Mar 22, 2025

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
## What does this PR do?
open-benchmark templated is broken after the datasets api refactor due
to 2 reasons
- provider_id and provider_resource_id are no longer needed 
- the type in run.yaml will be resolved as dict

this PR is to fix the above 2 issues 

## Test 
spin up a llama stack server successfully with llama stack run
`llama_stack/templates/open-benchmark/run.yaml`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Meta Open Source bot.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants