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.

Sign up for GitHub

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

Jump to bottom

add Lemonfoxai #628

Merged
merged 7 commits into from
Oct 8, 2024
Merged

add Lemonfoxai #628

merged 7 commits into from
Oct 8, 2024

Conversation

keshavkrishna
Copy link
Contributor

@keshavkrishna keshavkrishna commented Sep 25, 2024 •
edited
Loading

Intergated LemonfoxAI

  • Integrated LemonfoxAI to the list of providers.

Supported routes:

  • /chat/completions
  • /image/generate
  • /audio/transcriptions

Related Issues:

@keshavkrishna keshavkrishna marked this pull request as ready for review September 26, 2024 18:40
narengogi
narengogi reviewed Sep 27, 2024
View reviewed changes
src/providers/index.ts Outdated Show resolved Hide resolved
narengogi
narengogi requested changes Sep 27, 2024
View reviewed changes
Copy link
Collaborator

@narengogi narengogi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me overall, made a few comments

narengogi
narengogi reviewed Oct 1, 2024
View reviewed changes
Copy link
Collaborator

@narengogi narengogi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, please delete the 'chatComplete.ts` file and its corresponding imports now that it is not being used

@VisargD
Copy link
Collaborator

VisargD commented Oct 1, 2024

Hey @keshavkrishna - Can you please address the last comment. I will merge post that

@keshavkrishna
Copy link
Contributor Author

It will be done by today EOD

@keshavkrishna
Copy link
Contributor Author

Looks good, please delete the 'chatComplete.ts` file and its corresponding imports now that it is not being used

I have removed the code which was not being used and its coresponding imports also, for stream complete we are still using LemonfoxAIChatCompleteStreamChunkTransform , so i have not deleted chatComplete.ts

@VisargD VisargD merged commit 5d247d9 into Portkey-AI:main Oct 8, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@narengogi narengogi narengogi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@keshavkrishna @VisargD @narengogi