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 vllm setup, json.data is an array and is accessed as such. Also a… #4497

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

NghiVo21
Copy link

@NghiVo21 NghiVo21 commented Mar 5, 2025

Description

  • add auto detect behaviour
  • fix bug with accessing the array instead of the object

Checklist

  • [] The relevant docs, if any, have been updated or created
  • [] The relevant tests, if any, have been updated or created

No change from what i can tell.

Testing instructions

Configure continue to use vllm, simply perform simple tasks with continue like edit or chat to encounter this bug.

…dd autodetect behaviour in case multiple model deployement is supported in the future
Copy link

netlify bot commented Mar 5, 2025

Deploy Preview for continuedev canceled.

Name Link
🔨 Latest commit 0290822
🔍 Latest deploy log https://app.netlify.com/sites/continuedev/deploys/67c8b08eeca6380008a22941

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