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

inference providers #2624

Merged
merged 37 commits into from
Jan 28, 2025
Merged

inference providers #2624

merged 37 commits into from
Jan 28, 2025

Conversation

julien-c
Copy link
Member

No description provided.

jeffboudier
jeffboudier reviewed Jan 28, 2025
View reviewed changes
jeffboudier
jeffboudier reviewed Jan 28, 2025
View reviewed changes
pcuenca
pcuenca reviewed Jan 28, 2025
View reviewed changes
pcuenca
pcuenca reviewed Jan 28, 2025
View reviewed changes
pcuenca
pcuenca reviewed Jan 28, 2025
View reviewed changes
pcuenca
pcuenca reviewed Jan 28, 2025
View reviewed changes
burtenshaw
burtenshaw reviewed Jan 28, 2025
View reviewed changes
Copy link
Collaborator

@burtenshaw burtenshaw left a comment

Choose a reason for hiding this comment

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

I made edits to the billing section to try and make it easier to understand.

merveenoyan
merveenoyan reviewed Jan 28, 2025
View reviewed changes
Copy link
Contributor

@merveenoyan merveenoyan left a comment

Choose a reason for hiding this comment

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

first pass

@julien-c
Update inference-providers.md
57ac8f5 
Co-authored-by: Merve Noyan <[email protected]>
(cherry picked from commit beab924)
@julien-c
added tiny context for direct HTTP calls
2bfa5ad 
cc @Vaibhavs10 @pcuenca @gary149

(cherry picked from commit e78fbf7)
pcuenca
pcuenca reviewed Jan 28, 2025
View reviewed changes
merveenoyan
merveenoyan reviewed Jan 28, 2025
View reviewed changes
Copy link
Contributor

@merveenoyan merveenoyan left a comment

Choose a reason for hiding this comment

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

Readability nits, no more comments 🤗

burtenshaw
burtenshaw reviewed Jan 28, 2025
View reviewed changes
burtenshaw
burtenshaw reviewed Jan 28, 2025
View reviewed changes
pcuenca
pcuenca reviewed Jan 28, 2025
View reviewed changes
cfahlgren1
cfahlgren1 reviewed Jan 28, 2025
View reviewed changes
Vaibhavs10
Vaibhavs10 approved these changes Jan 28, 2025
View reviewed changes
Copy link
Member

@Vaibhavs10 Vaibhavs10 left a comment

Choose a reason for hiding this comment

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

🚢🚢🚢

Wauplin
Wauplin reviewed Jan 28, 2025
View reviewed changes
merveenoyan
merveenoyan reviewed Jan 28, 2025
View reviewed changes
Wauplin
Wauplin reviewed Jan 28, 2025
View reviewed changes
@julien-c julien-c merged commit f40439b into main Jan 28, 2025
1 check passed
@julien-c julien-c deleted the inference-providers-on-the-hub branch January 28, 2025 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pcuenca pcuenca pcuenca left review comments

@Wauplin Wauplin Wauplin left review comments

@cfahlgren1 cfahlgren1 cfahlgren1 left review comments

@Vaibhavs10 Vaibhavs10 Vaibhavs10 approved these changes

@burtenshaw burtenshaw burtenshaw left review comments

@merveenoyan merveenoyan merveenoyan left review comments

@jeffboudier jeffboudier jeffboudier left review comments

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

Successfully merging this pull request may close these issues.
9 participants
@julien-c @pcuenca @Wauplin @cfahlgren1 @Vaibhavs10 @burtenshaw @merveenoyan @jeffboudier @hanouticelina