-
Notifications
You must be signed in to change notification settings - Fork 1.2k
feat: Implement FastAPI router system #4191
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
Open
leseb
wants to merge
27
commits into
llamastack:main
Choose a base branch
from
leseb:routeur
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
eb3cab1
feat: Implement FastAPI router system
leseb 2fe24a6
chore: move ListBatchesResponse to models.py
leseb 00e7ea6
fix: adopt FastAPI directly in llama-stack-api
leseb 30cab02
chore: refactor Batches protocol to use request models
leseb 2003042
chore: same as previous commit but for more fields
leseb 9595619
chore: remove empty dir
leseb f62c604
chore: update unit test to use previously created Class
leseb 23e7444
chore: rename routes.py to fastapi_routes.py
leseb 8a21d8d
chore: mv router_registry.py to fastapi_router_registry.py
leseb 95e9455
chore: removed impl_getter from router function
leseb 234eaf4
chore: remove impl_getter function
leseb 6f552e0
fix: mypy
leseb ac816a6
fix: move models.py to top-level init
leseb 03a3126
chore: more accurate route parcing
leseb 49005f1
fix: use hardcoded list and dictionary mapping for router registry
leseb 87e60bc
chore: move dep functions outside of create_router
leseb 4f08a62
chore: remove telemetry code for routers
leseb a6aaf18
chore: generate FastAPI dependency functions from Pydantic models to …
leseb 6d76a63
fix: mypy
leseb 3770963
Merge branch 'main' into routeur
leseb 9a2b4ef
chore: clarify function and log about which router
leseb b0b3034
chore: rm leftover
leseb 3dc5b5d
fix: more accurate type
leseb ead9e63
fix: no inline import
leseb f330c8e
chore: simplify route addition when calling inspect
leseb 9534293
Merge branch 'main' into routeur
leseb 98f202b
Merge branch 'main' into routeur
leseb File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.