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

OpenAI Compatible API (together.xyz) gives status: 404 after upgrading mods. #206

Closed
ezynda3 opened this issue Jan 29, 2024 · 3 comments · Fixed by #213
Closed

OpenAI Compatible API (together.xyz) gives status: 404 after upgrading mods. #206

ezynda3 opened this issue Jan 29, 2024 · 3 comments · Fixed by #213

Comments

@ezynda3
Copy link

ezynda3 commented Jan 29, 2024

I recently added together's OpenAI compatible API to my mods config

together:
    api-key:
    api-key-env: TOGETHER_API_KEY
    base-url: https://api.together.xyz/v1
    models:
      mistralai/Mixtral-8x7B-Instruct-v0.1:
        aliases: ["mixtral"]
        max-input-chars: 12250
        fallback:
      snorkelai/Snorkel-Mistral-PairRM-DPO:
        aliases: ["snorkel"]
        max-input-chars: 12250
        fallback:

It was working just fine until I upgraded mods to v1.2.1.

Now whenever I try to run one of those models e.g. mods -m mixtral hello I get the following error.

   ERROR  There was a problem with the together API request.

  error, status code: 400, message: %!s(<nil>)
@ezynda3 ezynda3 changed the title OpenAI Compatible API (together.xyz) gives status: 404 after ugrading mods. OpenAI Compatible API (together.xyz) gives status: 404 after upgrading mods. Jan 29, 2024
@sedlund
Copy link

sedlund commented Feb 1, 2024

same.

e085c99 broke it. works in 2daa873

@caarlos0

togetherai api doesn't support a ResponseFormat so commenting this out:

mods/mods.go

Lines 402 to 404 in e085c99

ResponseFormat: &openai.ChatCompletionResponseFormat{
Type: responseType(cfg),
},

resolves.

It is omitempty

https://github.com/sashabaranov/go-openai/blob/6d9c3a6365643d02692ecc6f0b34a5fa3e7fea45/chat.go#L94

caarlos0 added a commit that referenced this issue Feb 6, 2024
closes #206

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
@caarlos0
Copy link
Member

caarlos0 commented Feb 6, 2024

can you try #213?

@sedlund
Copy link

sedlund commented Feb 6, 2024

can you try #213?

works for me! thanks.

caarlos0 added a commit that referenced this issue Feb 6, 2024
closes #206

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
shmup pushed a commit to shmup/mods that referenced this issue Feb 9, 2024
…celet#213)

closes charmbracelet#206

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
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 a pull request may close this issue.

3 participants